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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





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



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux