Messages in this thread | | | Date | Sat, 26 Aug 2023 12:03:32 +0200 | Subject | Re: [PATCH v3 08/15] media: qcom: camss: Untangle if/else spaghetti in camss | From | Konrad Dybcio <> |
| |
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@linaro.org> > --- Acked-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Konrad
| |