Re: [PATCH] arm64: dts: qcom: sm8550: Add support for camss

[Bryan O'Donoghue]

On 16/05/2025 11:32, Wenmeng Liu wrote:
> On 2025/5/16 18:15, Bryan O'Donoghue wrote:
> > On 16/05/2025 08:34, Wenmeng Liu wrote:
> > > > This should be 689 yes
> > > >
> > > > ---
> > > > bod
> > >
> > > Hi Bryan,Konrad,
> > >
> > > I confirmed that the value is 688 instead of 689. The documentation 
> > > incorrectly listed it as 689. To CC linux-media, I have resent the 
> > > patch:
> > > https://lore.kernel.org/linux-arm-msm/20250516072707.388332-1- 
> > > quic_wenmliu@xxxxxxxxxxx/
> >
> > Do you mean the documentation in the kernel or the documentation 
> > inside of qcom ?
> >
> > I checked the internal silicon definition, I think Konrad did also.
> >
> > Which documentation do you mean here ?
> >
> > ---
> > bod
>
> I mean is the ipcat documentation.
> And if configured to 689, the interrupt will fail to request:
> irq：irq: type mismatch, failed to map hwirq-721 for interrupt- 
> controller@17100000!
>
> According the mobile configuration, this value should be 688.
>
> Thanks,
> Wenmeng

OK, thank you for clarification.

---
bod