On 14:46-20250107, Kevin Hilman wrote: > Nishanth Menon <nm@xxxxxx> writes: > > > On 15:07-20241205, Kevin Hilman wrote: > >> > >> On Mon, 02 Dec 2024 15:33:30 +0100, Romain Naour wrote: > >> > The ACSPCIE_PROXY_CTRL registers within the CTRL_MMR space of TI's J721e > >> > SoC are used to drive the reference clock to the PCIe Endpoint device via > >> > the PAD IO Buffers. Add the compatible for allowing the PCIe driver to > >> > obtain the regmap for the ACSPCIE_CTRL register within the System > >> > Controller device-tree node in order to enable the PAD IO Buffers. > >> > > >> > The Technical Reference Manual for J721e SoC with details of the > >> > ASCPCIE_CTRL registers is available at: > >> > https://www.ti.com/lit/zip/spruil1 > >> > > >> > [...] > >> > >> Applied, thanks! > >> > >> [1/2] dt-bindings: mfd: syscon: Add ti,j721e-acspcie-proxy-ctrl compatible > >> commit: d8efc0b428856137608ffcbb6994da6041c9fe2a > >> [2/2] arm64: dts: ti: k3-j721e-beagleboneai64: Enable ACSPCIE output for PCIe1 > >> commit: 1d5e14a20dc60b440c60bec8489acfd45cdf7508 > >> > >> Best regards, > >> -- > >> Kevin Hilman <khilman@xxxxxxxxxxxx> > >> > > This will need a bit of fixup - See along the lines of the following. > > Additionally, we should be a bit careful about the dependency of dts > > mix up from two trees. > > sorry, these should be going through your tree in the first place. They > are now dropped from my tree, please go ahead and take them along with > Andrews fixup. Sorry for complicating things. Romain, There is additional fixups needed, unfortunately as well: syscon yaml has two lists based on which dt-schema version you use.. your patch fixed one list, but missed the other as well. Could you integrate the fixes and resubmit, please? -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D Something of this form needs to be squashed. diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml index 348025870b0f7..dda489916bc54 100644 --- a/Documentation/devicetree/bindings/mfd/syscon.yaml +++ b/Documentation/devicetree/bindings/mfd/syscon.yaml @@ -216,6 +216,8 @@ properties: - ti,am62p-cpsw-mac-efuse - ti,am654-dss-oldi-io-ctrl - ti,am654-icssg-ctrl + - ti,j721e-acspcie-proxy-ctrl - ti,j784s4-pcie-ctrl - ti,keystone-pllctrl - const: syscon