mbox series

[v3,0/6] PCI: J7200/J721E PCIe bindings

Message ID 20210104122232.24071-1-kishon@ti.com
Headers show
Series PCI: J7200/J721E PCIe bindings | expand

Message

Kishon Vijay Abraham I Jan. 4, 2021, 12:22 p.m. UTC
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
[2] -> http://lore.kernel.org/r/20201210130747.25436-1-kishon@ti.com

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(-)

-- 
2.17.1

Comments

Nishanth Menon Jan. 4, 2021, 12:59 p.m. UTC | #1
On 17:52-20210104, Kishon Vijay Abraham I wrote:
> 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

> [2] -> http://lore.kernel.org/r/20201210130747.25436-1-kishon@ti.com

> 

> 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?)?

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
Kishon Vijay Abraham I Jan. 4, 2021, 1:10 p.m. UTC | #2
Nishanth,

On 04/01/21 6:29 pm, Nishanth Menon wrote:
> On 17:52-20210104, Kishon Vijay Abraham I wrote:

>> 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

>> [2] -> http://lore.kernel.org/r/20201210130747.25436-1-kishon@ti.com

>>

>> 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

Thanks
Kishon
Nishanth Menon Jan. 4, 2021, 1:16 p.m. UTC | #3
On 18:40-20210104, Kishon Vijay Abraham I wrote:
> Nishanth,

> 

> On 04/01/21 6:29 pm, Nishanth Menon wrote:

> > On 17:52-20210104, Kishon Vijay Abraham I wrote:

> >> 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

> >> [2] -> http://lore.kernel.org/r/20201210130747.25436-1-kishon@ti.com

> >>

> >> 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


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?

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
Kishon Vijay Abraham I Jan. 4, 2021, 1:22 p.m. UTC | #4
Nishanth,

On 04/01/21 6:46 pm, Nishanth Menon wrote:
> On 18:40-20210104, Kishon Vijay Abraham I wrote:

>> Nishanth,

>>

>> On 04/01/21 6:29 pm, Nishanth Menon wrote:

>>> On 17:52-20210104, Kishon Vijay Abraham I wrote:

>>>> 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

>>>> [2] -> http://lore.kernel.org/r/20201210130747.25436-1-kishon@ti.com

>>>>

>>>> 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

> 

> 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.

Thanks
Kishon
Nishanth Menon Jan. 4, 2021, 2:51 p.m. UTC | #5
On 18:52-20210104, Kishon Vijay Abraham I wrote:
> Nishanth,

> 

> On 04/01/21 6:46 pm, Nishanth Menon wrote:

> > On 18:40-20210104, Kishon Vijay Abraham I wrote:

> >> Nishanth,

> >>

> >> On 04/01/21 6:29 pm, Nishanth Menon wrote:

> >>> On 17:52-20210104, Kishon Vijay Abraham I wrote:

> >>>> 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

> >>>> [2] -> http://lore.kernel.org/r/20201210130747.25436-1-kishon@ti.com

> >>>>

> >>>> 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

> > 

> > 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])


[1] https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
Kishon Vijay Abraham I Jan. 4, 2021, 3:31 p.m. UTC | #6
Nishanth,

On 04/01/21 8:21 pm, Nishanth Menon wrote:
> On 18:52-20210104, Kishon Vijay Abraham I wrote:

>> Nishanth,

>>

>> On 04/01/21 6:46 pm, Nishanth Menon wrote:

>>> On 18:40-20210104, Kishon Vijay Abraham I wrote:

>>>> Nishanth,

>>>>

>>>> On 04/01/21 6:29 pm, Nishanth Menon wrote:

>>>>> On 17:52-20210104, Kishon Vijay Abraham I wrote:

>>>>>> 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

>>>>>> [2] -> http://lore.kernel.org/r/20201210130747.25436-1-kishon@ti.com

>>>>>>

>>>>>> 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

>>>

>>> 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

>
Nishanth Menon Jan. 4, 2021, 3:52 p.m. UTC | #7
On 21:01-20210104, Kishon Vijay Abraham I wrote:
> Nishanth,

> 

> On 04/01/21 8:21 pm, Nishanth Menon wrote:

> > On 18:52-20210104, Kishon Vijay Abraham I wrote:

> >> Nishanth,

> >>

> >> On 04/01/21 6:46 pm, Nishanth Menon wrote:

> >>> On 18:40-20210104, Kishon Vijay Abraham I wrote:

> >>>> Nishanth,

> >>>>

> >>>> On 04/01/21 6:29 pm, Nishanth Menon wrote:

> >>>>> On 17:52-20210104, Kishon Vijay Abraham I wrote:

> >>>>>> 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

> >>>>>> [2] -> http://lore.kernel.org/r/20201210130747.25436-1-kishon@ti.com

> >>>>>>

> >>>>>> 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

> >>>

> >>> 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.



OK. please repost without the fixes tag.
-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D