Re: [PATCH 1/6] media: dt-bindings: Document SC8280XP/SM8350 Venus
From: Bryan O'Donoghue
Date: Mon Aug 07 2023 - 15:05:27 EST
On 07/08/2023 19:55, Konrad Dybcio wrote:
On 7.08.2023 20:49, Bryan O'Donoghue wrote:
On 07/08/2023 19:45, Konrad Dybcio wrote:
That can be taken care of with match data.
Konrad
Well perhaps.
I'm just sticking my oar in, to elucidate.
The compat sub-nodes aren't just a random choice with no logic. They exist to select between what you assign the blocks to be, encoder, decoder or any admixture thereof.
A functionality we want to maintain.
Surely something like a modparam would be more suitable here?
Konrad
Hmm.
Well from earlier in the thread the question "why do we have these
compat strings" is because we can have any combination of
encoder/decoder assigned.
If there's a cogent argument _still_ to be made to transition to some
new way of assignment then fine so long as we don't break that basic
flexibility.
Though my own €0.02 is that a module parameter is more of a PITA than a
compat string.
OTOH I could make the argument, that the high probability is most people
- probably all, just instantiate a single encoder and decoder and aren't
aware of or using the inbuilt flexibility.
@stan probably has the right idea what to do.
---
bod