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

[Vikram Sharma]

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