| Message ID | 20190605163434.23173-1-s-anna@ti.com |
|---|---|
| Headers | show |
| Series | Add MCU SRAM nodes for TI K3 SoCs | expand |
On 6/5/19 12:34 PM, Suman Anna wrote: > Hi Tero, > > The following series adds the DT nodes for the MCU SRAM present within > the MCU domain for both AM65x and J721E SoCs. The first 2 patches enable > the MCU SRAM node to be probed properly on AM65x SoCs, and the third > patch the equivalent for J721E SoCs. The third patch depends on the > base J721E support patch series from Nishanth [1]. > > The last patch is not related to MCU SRAM, but adds the ranges for the > R5F cluster that is also present in the MCU domain in preparation for > the R5F nodes. > Why not have this last patch as part of the series adding the R5F nodes? Andrew > regards > Suman > > [1] https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=121189 > > Suman Anna (4): > arm64: dts: ti: k3-am65: Add MCU SRAM ranges in interconnect nodes > arm64: dts: ti: k3-am65-mcu: Add the MCU RAM node > arm64: dts: ti: k3-j721e: Add the MCU SRAM node > arm64: dts: ti: k3-am65: Add R5F ranges in interconnect nodes > > arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi | 8 ++++++++ > arch/arm64/boot/dts/ti/k3-am65.dtsi | 6 ++++++ > arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 8 ++++++++ > 3 files changed, 22 insertions(+) >
On 6/5/19 11:53 AM, Andrew F. Davis wrote: > On 6/5/19 12:34 PM, Suman Anna wrote: >> Hi Tero, >> >> The following series adds the DT nodes for the MCU SRAM present within >> the MCU domain for both AM65x and J721E SoCs. The first 2 patches enable >> the MCU SRAM node to be probed properly on AM65x SoCs, and the third >> patch the equivalent for J721E SoCs. The third patch depends on the >> base J721E support patch series from Nishanth [1]. >> >> The last patch is not related to MCU SRAM, but adds the ranges for the >> R5F cluster that is also present in the MCU domain in preparation for >> the R5F nodes. >> > > Why not have this last patch as part of the series adding the R5F nodes? The patch can be applied independently as is, no reason to wait until I post those. regards Suman > > Andrew > >> regards >> Suman >> >> [1] >> https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=121189 >> >> Suman Anna (4): >> arm64: dts: ti: k3-am65: Add MCU SRAM ranges in interconnect nodes >> arm64: dts: ti: k3-am65-mcu: Add the MCU RAM node >> arm64: dts: ti: k3-j721e: Add the MCU SRAM node >> arm64: dts: ti: k3-am65: Add R5F ranges in interconnect nodes >> >> arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi | 8 ++++++++ >> arch/arm64/boot/dts/ti/k3-am65.dtsi | 6 ++++++ >> arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 8 ++++++++ >> 3 files changed, 22 insertions(+) >>
On 6/5/19 1:01 PM, Suman Anna wrote: > On 6/5/19 11:53 AM, Andrew F. Davis wrote: >> On 6/5/19 12:34 PM, Suman Anna wrote: >>> Hi Tero, >>> >>> The following series adds the DT nodes for the MCU SRAM present within >>> the MCU domain for both AM65x and J721E SoCs. The first 2 patches enable >>> the MCU SRAM node to be probed properly on AM65x SoCs, and the third >>> patch the equivalent for J721E SoCs. The third patch depends on the >>> base J721E support patch series from Nishanth [1]. >>> >>> The last patch is not related to MCU SRAM, but adds the ranges for the >>> R5F cluster that is also present in the MCU domain in preparation for >>> the R5F nodes. >>> >> >> Why not have this last patch as part of the series adding the R5F nodes? > > The patch can be applied independently as is, no reason to wait until I > post those. > No reason not to wait either.. plus it would keep related changes in the same series. Anyway, I don't have any problem with this, was just asking as I was not sure if I was missing some hidden dependency between the nodes. Andrew > regards > Suman > >> >> Andrew >> >>> regards >>> Suman >>> >>> [1] >>> https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=121189 >>> >>> Suman Anna (4): >>> arm64: dts: ti: k3-am65: Add MCU SRAM ranges in interconnect nodes >>> arm64: dts: ti: k3-am65-mcu: Add the MCU RAM node >>> arm64: dts: ti: k3-j721e: Add the MCU SRAM node >>> arm64: dts: ti: k3-am65: Add R5F ranges in interconnect nodes >>> >>> arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi | 8 ++++++++ >>> arch/arm64/boot/dts/ti/k3-am65.dtsi | 6 ++++++ >>> arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 8 ++++++++ >>> 3 files changed, 22 insertions(+) >>> >
On 05/06/2019 19:34, Suman Anna wrote: > Hi Tero, > > The following series adds the DT nodes for the MCU SRAM present within > the MCU domain for both AM65x and J721E SoCs. The first 2 patches enable > the MCU SRAM node to be probed properly on AM65x SoCs, and the third > patch the equivalent for J721E SoCs. The third patch depends on the > base J721E support patch series from Nishanth [1]. > > The last patch is not related to MCU SRAM, but adds the ranges for the > R5F cluster that is also present in the MCU domain in preparation for > the R5F nodes. > > regards > Suman > > [1] https://patchwork.kernel.org/project/linux-arm-kernel/list/?series=121189 Queuing patches 1,2 and 4 towards 5.3 thanks. Patch #3 is waiting until I can sort out j7 dependencies, which should happen later this week. -Tero > > Suman Anna (4): > arm64: dts: ti: k3-am65: Add MCU SRAM ranges in interconnect nodes > arm64: dts: ti: k3-am65-mcu: Add the MCU RAM node > arm64: dts: ti: k3-j721e: Add the MCU SRAM node > arm64: dts: ti: k3-am65: Add R5F ranges in interconnect nodes > > arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi | 8 ++++++++ > arch/arm64/boot/dts/ti/k3-am65.dtsi | 6 ++++++ > arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 8 ++++++++ > 3 files changed, 22 insertions(+) > -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki