Re: [PATCH v4 4/6] media: qcom: camss: Add sc8280xp resource details

[Konrad Dybcio]

On 11/9/23 16:13, Bryan O'Donoghue wrote:
> On 09/11/2023 13:44, Konrad Dybcio wrote:
> > > +        .clock_rate = { { 0 },
> > > +                { 0 },
> > > +                { 19200000, 80000000, 80000000, 80000000, 80000000},
> > > +                { 19200000, 150000000, 266666667, 320000000, 400000000, 480000000 },
> > > +                { 400000000, 558000000, 637000000, 760000000 },
> > > +                { 0 }, },
> > Not the case here!
>
> I agree with you in principle, the checking for the frequency shouldn't rely on if (freq[x]) however in this case - we are doing aggregate initialisation of a fixed size array and the compiler should save us from ourselves.
Oh right, I didn't think about the sizing of it and partial init..

Konrad