Re: [PATCH v3 08/15] media: qcom: camss: Untangle if/else spaghetti in camss

From: Konrad Dybcio
Date: Sat Aug 26 2023 - 06:04:18 EST


On 23.08.2023 12:44, Bryan O'Donoghue wrote:
> I refuse to add another SoC to this convoluted if/else structure. By the
> time we added in a third SoC we ought to have transitioned these control
> flow strutures to switches.
>
> Adding in another Soc or two will make some of these if statements into
> five clause behemoths.
>
> Introduce switches in the obvious places to despaghetiify.
>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>
> ---
Acked-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>

Konrad