Re: [PATCH v3 0/6] PCI: J7200/J721E PCIe bindings
From: Kishon Vijay Abraham I <hidden>
Date: 2021-01-04 15:32:17
Also in:
linux-devicetree, lkml
Nishanth, On 04/01/21 8:21 pm, Nishanth Menon wrote:
On 18:52-20210104, Kishon Vijay Abraham I wrote:quoted
Nishanth, On 04/01/21 6:46 pm, Nishanth Menon wrote:quoted
On 18:40-20210104, Kishon Vijay Abraham I wrote:quoted
Nishanth, On 04/01/21 6:29 pm, Nishanth Menon wrote:quoted
On 17:52-20210104, Kishon Vijay Abraham I wrote:quoted
Patch series adds DT nodes in order to get PCIe working in J7200. Also includes couple of fixes for J721e. v1 of the patch series can be found @ [1] v2 of the patch series can be found @ [2] Changes from v2: 1) Moved serdes_refclk node out of interconnect node and also replaced "_" with "-" Changes from v1: 1) Include only the device tree patches here (the binding patch is sent separately) 2) Include couple of patches that fixes J721E DTS. [1] -> http://lore.kernel.org/r/20201102101154.13598-1-kishon@ti.com (local) [2] -> http://lore.kernel.org/r/20201210130747.25436-1-kishon@ti.com (local) Kishon Vijay Abraham I (6): arm64: dts: ti: k3-j721e-main: Fix supported max outbound regions arm64: dts: ti: k3-j721e-main: Remove "syscon" nodes added for pcieX_ctrl arm64: dts: ti: k3-j7200-main: Add SERDES and WIZ device tree node arm64: dts: ti: k3-j7200-main: Add PCIe device tree node arm64: dts: ti: k3-j7200-common-proc-board: Enable SERDES0 arm64: dts: ti: k3-j7200-common-proc-board: Enable PCIe .../dts/ti/k3-j7200-common-proc-board.dts | 38 ++++++ arch/arm64/boot/dts/ti/k3-j7200-main.dtsi | 111 ++++++++++++++++++ arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 52 ++------ 3 files changed, 157 insertions(+), 44 deletions(-)A bit confused on the dependency here. is there something merged into next-20210104 that makes this series ready for pickup? is there a way I can get a immutable tag for driver fixups to pull so that my dts next is not broken for PCIe (I am assuming looking at the series that this is probably not a backward compatible series?)?There are no driver changes for the basic J7200 PCIe support and the DT bindings are already merged [1]. There are few errata fixes applicable for J721E which has to be removed for J7200 but that depends on other patches to be merged [1] but that doesn't impact j7200 functionality. [1] -> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml#n19 [2] -> http://lore.kernel.org/r/20201230120515.2348-1-nadeem@cadence.com (local)So, Dropping stuff like "cdns,max-outbound-regions" (change from 16 to 32) will work on older kernels? Could you do a quick sanity check on the couple of "fixes" patches in this thread is not breaking functionality introduced in the older stable kernels?Okay, the driver changes are done such that it works with both old DT and new DT however the newer DT will not work with old kernel. So I think I should drop the "Fixes" tag in the DT patches.If there is a specific stable kernel version you might like to use, you could use that as well for those stable tags (see [1])
It's not a fix that might bother people (as it doesn't impact existing functionality), so don't think it's a candidate for stable releases. Thanks Kishon
[1] https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html
_______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel