mbox series

[V5,0/8] dt-bindinds/dts: support i.MX8ULP

Message ID 20211120113454.785997-1-peng.fan@oss.nxp.com (mailing list archive)
Headers show
Series dt-bindinds/dts: support i.MX8ULP | expand

Message

Peng Fan (OSS) Nov. 20, 2021, 11:34 a.m. UTC
From: Peng Fan <peng.fan@nxp.com>


V5:
 only fix patch 8/8 "arm64: dts: imx8ulp: Add the basic dts for imx8ulp evk board"
  - Correct bus-width to 8 for eMMC
  - Drop pinctrl enet which no user
 Drop patch 1/9 in V4, since in merged in linux-next
 Add A-b/R-b tag

V4:
 https://lists.infradead.org/pipermail/linux-arm-kernel/2021-November/696020.html

I just pick-up Jacky's work, rebase and send out V4 based on
Jacky's V3 patchset.

If you prefer to split the patchset and send single patches, I could
resend one by one.

Note: the V3 has a gpio bindings patch, it has been separated and
sent to gpio list in a single one per Linus requested.

This patchset is to add i.MX8ULP in dt-bindings, introduce
basic i.MX8ULP dtsi and basic evk board support.

The detailed version changes are in each patch.

There is checkpatch error in patch 8, but that is for device tree macro
and we could use use "(xx)".

In V4, the changes are in patch 8, 9 when rebasing to linux-next/master
,fix build errors after the clk driver in tree and drop a few nodes
that not have bindings.

Tested with CONFIG_CLK_IMX8ULP and CONFIG_PINCTRL_IMX8ULP enabled.

V3:
 https://lore.kernel.org/linux-arm-kernel/20210625011355.3468586-6-ping.bai@nxp.com/T/

Jacky Bai (8):
  dt-bindings: i2c: imx-lpi2c: Add imx8ulp compatible string
  dt-bindings: serial: fsl-lpuart: Add imx8ulp compatible string
  dt-bindings: spi: fsl-lpspi: Add imx8ulp compatible string
  dt-bindings: timer: tpm-timer: Add imx8ulp compatible string
  dt-bindings: watchdog: imx7ulp-wdt: Add imx8ulp compatible string
  dt-bindings: arm: fsl: Add binding for imx8ulp evk
  arm64: dts: imx8ulp: Add the basic dtsi file for imx8ulp
  arm64: dts: imx8ulp: Add the basic dts for imx8ulp evk board

 .../devicetree/bindings/arm/fsl.yaml          |   6 +
 .../bindings/i2c/i2c-imx-lpi2c.yaml           |   4 +-
 .../bindings/serial/fsl-lpuart.yaml           |   4 +-
 .../bindings/spi/spi-fsl-lpspi.yaml           |  11 +-
 .../bindings/timer/nxp,tpm-timer.yaml         |   6 +-
 .../bindings/watchdog/fsl-imx7ulp-wdt.yaml    |   7 +-
 arch/arm64/boot/dts/freescale/Makefile        |   1 +
 arch/arm64/boot/dts/freescale/imx8ulp-evk.dts |  64 ++
 .../boot/dts/freescale/imx8ulp-pinfunc.h      | 978 ++++++++++++++++++
 arch/arm64/boot/dts/freescale/imx8ulp.dtsi    | 396 +++++++
 10 files changed, 1468 insertions(+), 9 deletions(-)
 create mode 100644 arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
 create mode 100755 arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h
 create mode 100644 arch/arm64/boot/dts/freescale/imx8ulp.dtsi

Comments

Mark Brown Nov. 23, 2021, midnight UTC | #1
On Sat, 20 Nov 2021 19:34:46 +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> 
> V5:
>  only fix patch 8/8 "arm64: dts: imx8ulp: Add the basic dts for imx8ulp evk board"
>   - Correct bus-width to 8 for eMMC
>   - Drop pinctrl enet which no user
>  Drop patch 1/9 in V4, since in merged in linux-next
>  Add A-b/R-b tag
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-next

Thanks!

[3/8] dt-bindings: spi: fsl-lpspi: Add imx8ulp compatible string
      commit: 49cd1eb37b487036f51bd57b591f7b5760a10e02

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a 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.

Thanks,
Mark
Peng Fan Nov. 25, 2021, 9:11 a.m. UTC | #2
Hi Shawn,

> Subject: [PATCH V5 0/8] dt-bindinds/dts: support i.MX8ULP
> 
> From: Peng Fan <peng.fan@nxp.com>
> 
> 
> V5:
>  only fix patch 8/8 "arm64: dts: imx8ulp: Add the basic dts for imx8ulp evk
> board"
>   - Correct bus-width to 8 for eMMC
>   - Drop pinctrl enet which no user
>  Drop patch 1/9 in V4, since in merged in linux-next  Add A-b/R-b tag
> 
> V4:
> 
> https://lists.infradead.org/pipermail/linux-arm-kernel/2021-November/6960
> 20.html
> 
> I just pick-up Jacky's work, rebase and send out V4 based on Jacky's V3
> patchset.
> 
> If you prefer to split the patchset and send single patches, I could resend one
> by one.
> 
> Note: the V3 has a gpio bindings patch, it has been separated and sent to gpio
> list in a single one per Linus requested.
> 
> This patchset is to add i.MX8ULP in dt-bindings, introduce basic i.MX8ULP dtsi
> and basic evk board support.
> 
> The detailed version changes are in each patch.
> 
> There is checkpatch error in patch 8, but that is for device tree macro and we
> could use use "(xx)".
> 
> In V4, the changes are in patch 8, 9 when rebasing to linux-next/master ,fix
> build errors after the clk driver in tree and drop a few nodes that not have
> bindings.
> 
> Tested with CONFIG_CLK_IMX8ULP and CONFIG_PINCTRL_IMX8ULP
> enabled.
> 
> V3:
> 
> https://lore.kernel.org/linux-arm-kernel/20210625011355.3468586-6-ping.b
> ai@nxp.com/T/
> 
> Jacky Bai (8):
>   dt-bindings: i2c: imx-lpi2c: Add imx8ulp compatible string
>   dt-bindings: serial: fsl-lpuart: Add imx8ulp compatible string
>   dt-bindings: spi: fsl-lpspi: Add imx8ulp compatible string
>   dt-bindings: timer: tpm-timer: Add imx8ulp compatible string
>   dt-bindings: watchdog: imx7ulp-wdt: Add imx8ulp compatible string
>   dt-bindings: arm: fsl: Add binding for imx8ulp evk
>   arm64: dts: imx8ulp: Add the basic dtsi file for imx8ulp
>   arm64: dts: imx8ulp: Add the basic dts for imx8ulp evk board

For the several dt-bindings, patch 3 has been taken by SPI maintainer, 

All patches have got R-b or A-b tag.

would you pick others bindings or they should go through subsystem
maintainer's tree?

Thanks,
Peng.

> 
>  .../devicetree/bindings/arm/fsl.yaml          |   6 +
>  .../bindings/i2c/i2c-imx-lpi2c.yaml           |   4 +-
>  .../bindings/serial/fsl-lpuart.yaml           |   4 +-
>  .../bindings/spi/spi-fsl-lpspi.yaml           |  11 +-
>  .../bindings/timer/nxp,tpm-timer.yaml         |   6 +-
>  .../bindings/watchdog/fsl-imx7ulp-wdt.yaml    |   7 +-
>  arch/arm64/boot/dts/freescale/Makefile        |   1 +
>  arch/arm64/boot/dts/freescale/imx8ulp-evk.dts |  64 ++
>  .../boot/dts/freescale/imx8ulp-pinfunc.h      | 978 ++++++++++++++++++
>  arch/arm64/boot/dts/freescale/imx8ulp.dtsi    | 396 +++++++
>  10 files changed, 1468 insertions(+), 9 deletions(-)  create mode 100644
> arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
>  create mode 100755 arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> 
> --
> 2.25.1
Peng Fan Nov. 25, 2021, 9:19 a.m. UTC | #3
> Subject: RE: [PATCH V5 0/8] dt-bindinds/dts: support i.MX8ULP

Seems there is conflict after Abel's patchset in next tree, I'll rebase.

Thanks,
Peng.

> 
> Hi Shawn,
> 
> > Subject: [PATCH V5 0/8] dt-bindinds/dts: support i.MX8ULP
> >
> > From: Peng Fan <peng.fan@nxp.com>
> >
> >
> > V5:
> >  only fix patch 8/8 "arm64: dts: imx8ulp: Add the basic dts for
> > imx8ulp evk board"
> >   - Correct bus-width to 8 for eMMC
> >   - Drop pinctrl enet which no user
> >  Drop patch 1/9 in V4, since in merged in linux-next  Add A-b/R-b tag
> >
> > V4:
> >
> > https://lists.infradead.org/pipermail/linux-arm-kernel/2021-November/6
> > 960
> > 20.html
> >
> > I just pick-up Jacky's work, rebase and send out V4 based on Jacky's
> > V3 patchset.
> >
> > If you prefer to split the patchset and send single patches, I could
> > resend one by one.
> >
> > Note: the V3 has a gpio bindings patch, it has been separated and sent
> > to gpio list in a single one per Linus requested.
> >
> > This patchset is to add i.MX8ULP in dt-bindings, introduce basic
> > i.MX8ULP dtsi and basic evk board support.
> >
> > The detailed version changes are in each patch.
> >
> > There is checkpatch error in patch 8, but that is for device tree
> > macro and we could use use "(xx)".
> >
> > In V4, the changes are in patch 8, 9 when rebasing to
> > linux-next/master ,fix build errors after the clk driver in tree and
> > drop a few nodes that not have bindings.
> >
> > Tested with CONFIG_CLK_IMX8ULP and CONFIG_PINCTRL_IMX8ULP
> enabled.
> >
> > V3:
> >
> > https://lore.kernel.org/linux-arm-kernel/20210625011355.3468586-6-ping
> > .b
> > ai@nxp.com/T/
> >
> > Jacky Bai (8):
> >   dt-bindings: i2c: imx-lpi2c: Add imx8ulp compatible string
> >   dt-bindings: serial: fsl-lpuart: Add imx8ulp compatible string
> >   dt-bindings: spi: fsl-lpspi: Add imx8ulp compatible string
> >   dt-bindings: timer: tpm-timer: Add imx8ulp compatible string
> >   dt-bindings: watchdog: imx7ulp-wdt: Add imx8ulp compatible string
> >   dt-bindings: arm: fsl: Add binding for imx8ulp evk
> >   arm64: dts: imx8ulp: Add the basic dtsi file for imx8ulp
> >   arm64: dts: imx8ulp: Add the basic dts for imx8ulp evk board
> 
> For the several dt-bindings, patch 3 has been taken by SPI maintainer,
> 
> All patches have got R-b or A-b tag.
> 
> would you pick others bindings or they should go through subsystem
> maintainer's tree?
> 
> Thanks,
> Peng.
> 
> >
> >  .../devicetree/bindings/arm/fsl.yaml          |   6 +
> >  .../bindings/i2c/i2c-imx-lpi2c.yaml           |   4 +-
> >  .../bindings/serial/fsl-lpuart.yaml           |   4 +-
> >  .../bindings/spi/spi-fsl-lpspi.yaml           |  11 +-
> >  .../bindings/timer/nxp,tpm-timer.yaml         |   6 +-
> >  .../bindings/watchdog/fsl-imx7ulp-wdt.yaml    |   7 +-
> >  arch/arm64/boot/dts/freescale/Makefile        |   1 +
> >  arch/arm64/boot/dts/freescale/imx8ulp-evk.dts |  64 ++
> >  .../boot/dts/freescale/imx8ulp-pinfunc.h      | 978
> ++++++++++++++++++
> >  arch/arm64/boot/dts/freescale/imx8ulp.dtsi    | 396 +++++++
> >  10 files changed, 1468 insertions(+), 9 deletions(-)  create mode
> > 100644 arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
> >  create mode 100755 arch/arm64/boot/dts/freescale/imx8ulp-pinfunc.h
> >  create mode 100644 arch/arm64/boot/dts/freescale/imx8ulp.dtsi
> >
> > --
> > 2.25.1
Wolfram Sang Nov. 25, 2021, 9:32 a.m. UTC | #4
> Seems there is conflict after Abel's patchset in next tree, I'll rebase.

True also for the I2C patch. A rebase would be great!