Re: [PATCH 1/3] arm64: dts: qcom: sm8350: Remove second reg from pdc

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

 



Hi,

On 3/17/2021 11:38 PM, Bjorn Andersson wrote:
On Wed 17 Mar 09:02 CDT 2021, Marc Zyngier wrote:

On Wed, 17 Mar 2021 09:48:09 +0000,
Maulik Shah <mkshah@xxxxxxxxxxxxxx> wrote:
Hi Marc,

On 3/17/2021 2:47 PM, Marc Zyngier wrote:
On Wed, 17 Mar 2021 05:29:54 +0000,
Maulik Shah <mkshah@xxxxxxxxxxxxxx> wrote:
PDC interrupt controller driver do not use second reg. Remove it.
This is a DT file, not a driver. What the driver does is irrelevant.

The real question is: what does this range do?

Thanks,

	M.
This is to set interrupt type in SPI config for which there was a
change [1] but has not gone in for upstream PDC driver.

The second reg is not used in upstream PDC driver, probably when
posting downstream DT changes for sm8350/sm8250 it was carried in
device node as is.

As its not mentioned in bindigs as well, dtbs_check reports it as
additional reg when converted to yaml.
Then I'd rather you provide accurate documentation in the binding
rather than changing the DT files. Other operating systems may use it,
and it isn't unlikely that Linux could use the feature at some point.

I agree. Maulik, please update the DT binding to document this region as
well.
sure. updated in v2.


It also seems relevant to pursue getting [1] into the upstream Linux
kernel. Is this something that you use downstream Maulik?

Yes its used in downstream. We can pursue to get [1] in.

Thanks,
Maulik


Regards,
Bjorn

--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux