mbox series

[00/12] TI J7x Disable Incomplete DT Nodes

Message ID 20221020160305.18711-1-afd@ti.com (mailing list archive)
Headers show
Series TI J7x Disable Incomplete DT Nodes | expand

Message

Andrew Davis Oct. 20, 2022, 4:02 p.m. UTC
Hello all,

Same story as for AM64x[0] and AM62x[1], this time for J7x.

This one was a little more involved that the last 2 rounds as some IP
(like UART) are pinned out using the default pinmux. Perhaps we should be
explicit here and have pinmux nodes that simply restate the defaults. I
can go solve that in a follow up series if we think that is what we
want to do.

I have the last round for AM65x ready to post, but that involves a few
boards that I do not have (Simatic IOT2050) but would like to do some
additional re-work with, so holding off until after I get that sorted.

Thanks,
Andrew

[0] https://www.spinics.net/lists/arm-kernel/msg1018532.html
[1] https://www.spinics.net/lists/arm-kernel/msg1018864.html

Andrew Davis (12):
  arm64: dts: ti: k3-j721e: Enable UART nodes at the board level
  arm64: dts: ti: k3-j721e: Enable I2C nodes at the board level
  arm64: dts: ti: k3-j721e: Enable MCASP nodes at the board level
  arm64: dts: ti: k3-j721e: Enable MCAN nodes at the board level
  arm64: dts: ti: k3-j7200: Enable UART nodes at the board level
  arm64: dts: ti: k3-j7200: Enable I2C nodes at the board level
  arm64: dts: ti: k3-j721s2: Enable UART nodes at the board level
  arm64: dts: ti: k3-j721e: Enable Mailbox nodes at the board level
  arm64: dts: ti: k3-j7200: Enable Mailbox nodes at the board level
  arm64: dts: ti: k3-j721s2: Enable Mailbox nodes at the board level
  arm64: dts: ti: k3-j721s2: Enable MCAN nodes at the board level
  arm64: dts: ti: k3-j721s2: Enable I2C nodes at the board level

 .../dts/ti/k3-j7200-common-proc-board.dts     |  48 ++----
 arch/arm64/boot/dts/ti/k3-j7200-main.dtsi     |  29 ++++
 .../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi      |   5 +
 arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi   |  42 +----
 .../dts/ti/k3-j721e-common-proc-board.dts     | 141 +++-------------
 arch/arm64/boot/dts/ti/k3-j721e-main.dtsi     |  55 ++++++
 .../boot/dts/ti/k3-j721e-mcu-wakeup.dtsi      |   7 +
 arch/arm64/boot/dts/ti/k3-j721e-sk.dts        | 158 ++----------------
 arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi   |  33 +---
 .../dts/ti/k3-j721s2-common-proc-board.dts    | 132 +--------------
 arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi    |  58 +++++++
 .../boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi     |   7 +
 arch/arm64/boot/dts/ti/k3-j721s2-som-p0.dtsi  |  98 +----------
 13 files changed, 231 insertions(+), 582 deletions(-)

Comments

Bryan Brattlof Oct. 20, 2022, 10:16 p.m. UTC | #1
On October 20, 2022 thus sayeth Andrew Davis:
> Hello all,
> 
> Same story as for AM64x[0] and AM62x[1], this time for J7x.
> 
> This one was a little more involved that the last 2 rounds as some IP
> (like UART) are pinned out using the default pinmux. Perhaps we should be
> explicit here and have pinmux nodes that simply restate the defaults. I
> can go solve that in a follow up series if we think that is what we
> want to do.
> 
> I have the last round for AM65x ready to post, but that involves a few
> boards that I do not have (Simatic IOT2050) but would like to do some
> additional re-work with, so holding off until after I get that sorted.
> 
> Thanks,
> Andrew
> 
> [0] https://www.spinics.net/lists/arm-kernel/msg1018532.html
> [1] https://www.spinics.net/lists/arm-kernel/msg1018864.html
> 
> Andrew Davis (12):
>   arm64: dts: ti: k3-j721e: Enable UART nodes at the board level
>   arm64: dts: ti: k3-j721e: Enable I2C nodes at the board level
>   arm64: dts: ti: k3-j721e: Enable MCASP nodes at the board level
>   arm64: dts: ti: k3-j721e: Enable MCAN nodes at the board level
>   arm64: dts: ti: k3-j7200: Enable UART nodes at the board level
>   arm64: dts: ti: k3-j7200: Enable I2C nodes at the board level
>   arm64: dts: ti: k3-j721s2: Enable UART nodes at the board level
>   arm64: dts: ti: k3-j721e: Enable Mailbox nodes at the board level
>   arm64: dts: ti: k3-j7200: Enable Mailbox nodes at the board level
>   arm64: dts: ti: k3-j721s2: Enable Mailbox nodes at the board level
>   arm64: dts: ti: k3-j721s2: Enable MCAN nodes at the board level
>   arm64: dts: ti: k3-j721s2: Enable I2C nodes at the board level
> 
>  .../dts/ti/k3-j7200-common-proc-board.dts     |  48 ++----
>  arch/arm64/boot/dts/ti/k3-j7200-main.dtsi     |  29 ++++
>  .../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi      |   5 +
>  arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi   |  42 +----
>  .../dts/ti/k3-j721e-common-proc-board.dts     | 141 +++-------------
>  arch/arm64/boot/dts/ti/k3-j721e-main.dtsi     |  55 ++++++
>  .../boot/dts/ti/k3-j721e-mcu-wakeup.dtsi      |   7 +
>  arch/arm64/boot/dts/ti/k3-j721e-sk.dts        | 158 ++----------------
>  arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi   |  33 +---
>  .../dts/ti/k3-j721s2-common-proc-board.dts    | 132 +--------------
>  arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi    |  58 +++++++
>  .../boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi     |   7 +
>  arch/arm64/boot/dts/ti/k3-j721s2-som-p0.dtsi  |  98 +----------
>  13 files changed, 231 insertions(+), 582 deletions(-)
> 

*whew* these Jacinto boards have a lot of stuff on them :) but it LGTM

Reviewed-by: Bryan Brattlof <bb@ti.com>

~Bryan
Vignesh Raghavendra Oct. 28, 2022, 1:20 p.m. UTC | #2
On 20/10/22 9:32 pm, Andrew Davis wrote:
> Andrew Davis (12):
>   arm64: dts: ti: k3-j721e: Enable UART nodes at the board level
>   arm64: dts: ti: k3-j721e: Enable I2C nodes at the board level
>   arm64: dts: ti: k3-j721e: Enable MCASP nodes at the board level
>   arm64: dts: ti: k3-j721e: Enable MCAN nodes at the board level
>   arm64: dts: ti: k3-j7200: Enable UART nodes at the board level
>   arm64: dts: ti: k3-j7200: Enable I2C nodes at the board level
>   arm64: dts: ti: k3-j721s2: Enable UART nodes at the board level
>   arm64: dts: ti: k3-j721e: Enable Mailbox nodes at the board level
>   arm64: dts: ti: k3-j7200: Enable Mailbox nodes at the board level
>   arm64: dts: ti: k3-j721s2: Enable Mailbox nodes at the board level
>   arm64: dts: ti: k3-j721s2: Enable MCAN nodes at the board level
>   arm64: dts: ti: k3-j721s2: Enable I2C nodes at the board level
> 
>  .../dts/ti/k3-j7200-common-proc-board.dts     |  48 ++----
>  arch/arm64/boot/dts/ti/k3-j7200-main.dtsi     |  29 ++++
>  .../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi      |   5 +
>  arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi   |  42 +----
>  .../dts/ti/k3-j721e-common-proc-board.dts     | 141 +++-------------
>  arch/arm64/boot/dts/ti/k3-j721e-main.dtsi     |  55 ++++++
>  .../boot/dts/ti/k3-j721e-mcu-wakeup.dtsi      |   7 +
>  arch/arm64/boot/dts/ti/k3-j721e-sk.dts        | 158 ++----------------
>  arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi   |  33 +---
>  .../dts/ti/k3-j721s2-common-proc-board.dts    | 132 +--------------
>  arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi    |  58 +++++++
>  .../boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi     |   7 +
>  arch/arm64/boot/dts/ti/k3-j721s2-som-p0.dtsi  |  98 +----------
>  13 files changed, 231 insertions(+), 582 deletions(-)

Acked-by: Vignesh Raghavendra <vigneshr@ti.com>
Nishanth Menon Oct. 28, 2022, 1:31 p.m. UTC | #3
Hi Andrew Davis,

On Thu, 20 Oct 2022 11:02:53 -0500, Andrew Davis wrote:
> Same story as for AM64x[0] and AM62x[1], this time for J7x.
> 
> This one was a little more involved that the last 2 rounds as some IP
> (like UART) are pinned out using the default pinmux. Perhaps we should be
> explicit here and have pinmux nodes that simply restate the defaults. I
> can go solve that in a follow up series if we think that is what we
> want to do.
> 
> [...]

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[01/12] arm64: dts: ti: k3-j721e: Enable UART nodes at the board level
        commit: fe17e20fde14c6cbaf1cae97aef370b6f4c73518
[02/12] arm64: dts: ti: k3-j721e: Enable I2C nodes at the board level
        commit: 282c4ad3b8c1da99095e0edbbe617cdf4dcb138b
[03/12] arm64: dts: ti: k3-j721e: Enable MCASP nodes at the board level
        commit: 256596ad158c80c8e4104a6310a411ec8829f9eb
[04/12] arm64: dts: ti: k3-j721e: Enable MCAN nodes at the board level
        commit: 39e7758b94e31071a87489364c723afe92849ae7
[05/12] arm64: dts: ti: k3-j7200: Enable UART nodes at the board level
        commit: dae322f8a7ef11a06bfb61bada4ebbcac219783e
[06/12] arm64: dts: ti: k3-j7200: Enable I2C nodes at the board level
        commit: a9ed915cd55c883fed72e50b45988fa4305496a7
[07/12] arm64: dts: ti: k3-j721s2: Enable UART nodes at the board level
        commit: 0e63f35a9c6acf1d74bd693c492f8955bb16c89a
[08/12] arm64: dts: ti: k3-j721e: Enable Mailbox nodes at the board level
        commit: 7e48b665100ee84bdaa163dece6d40351077f664
[09/12] arm64: dts: ti: k3-j7200: Enable Mailbox nodes at the board level
        commit: 74f0f58d3122c3e2cc313dd6311b9742dbfd54e0
[10/12] arm64: dts: ti: k3-j721s2: Enable Mailbox nodes at the board level
        commit: 3fb0402f65959edc9a14842dc36828d2c7109ae2
[11/12] arm64: dts: ti: k3-j721s2: Enable MCAN nodes at the board level
        commit: 06639b8ae0e9be0cf0195209e6b69f4e254687b0
[12/12] arm64: dts: ti: k3-j721s2: Enable I2C nodes at the board level
        commit: 0aef5131380628e2a9bfcbbcfed5d6477c8f9651

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] git://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git