Re: [PATCH v1 1/2] media: dt-bindings: Add qcom,qcs8300-camss

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

 




On 3/19/2025 3:18 AM, Bryan O'Donoghue wrote:
On 18/03/2025 07:16, Krzysztof Kozlowski wrote:
On 18/03/2025 06:52, Vikram Sharma wrote:

On 2/23/2025 5:03 PM, Krzysztof Kozlowski wrote:
On Fri, Feb 14, 2025 at 03:17:46PM +0530, Vikram Sharma wrote:
+properties:
+  compatible:
+    const: qcom,qcs8300-camss
+
+  reg:
+    maxItems: 21
+
+  reg-names:
+    items:
+      - const: csid_wrapper
Why different order of entries than sm8550?

Hi Krzysztof,

Thanks for your review.
I did this change to address a comment from Bryan on another series.
https://lore.kernel.org/linux-arm-msm/e152ff78-caa5-493a-88da-96a6670eb2a2@xxxxxxxxxx/

Please suggest if I should keep the order same as sm8550?
If you chosen the same order as x1e80100 then it is fine, but that file
is not merged so it is your responsibility to track any differences and
be sure whatever you send is always in sync with x1e.

Best regards,
Krzysztof

My mistake, I hadn't realised/remembered what we merged for 8550.

Vikram, please follow latest committed example @ 8550.

Thanks Bryan for conclusion on this. Will keep reference dts as 8550.


---
bod

Best regards,
Vikram




[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