[U-Boot] [RFC PATCH v1 2/2] arm: ls1021aqds: Enable driver model lpuart serial driver

[Huan Wang]

Hi, York,

> On Tue, Feb 9, 2016 at 8:57 AM, York Sun <york.sun at nxp.com> wrote:
> > Convert ls1021aqds_nor_lpuart and ls1021aqds_ddr4_nor_lpuart to driver
> > model suport. Enable lpuart port driver.
> >
> > Signed-off-by: York Sun <york.sun at nxp.com>
> > CC: Alison Wang <alison.wang at nxp.com>
> > CC: Bin Meng <bmeng.cn at gmail.com>
> >
> > ---
> >
> >  arch/arm/dts/Makefile                        |    1 +
> >  arch/arm/dts/ls1021a-qds-lpuart.dts          |   16 ++++++++++++++++
> >  configs/ls1021aqds_ddr4_nor_lpuart_defconfig |    4 ++++
> >  configs/ls1021aqds_nor_lpuart_defconfig      |    4 ++++
> >  4 files changed, 25 insertions(+)
> >  create mode 100644 arch/arm/dts/ls1021a-qds-lpuart.dts
> >
> > diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile index
> > ef58aec..c27b80c 100644
> > --- a/arch/arm/dts/Makefile
> > +++ b/arch/arm/dts/Makefile
> > @@ -95,6 +95,7 @@ dtb-$(CONFIG_TARGET_BEAGLE_X15) +=
> > am57xx-beagle-x15.dtb
> >  dtb-$(CONFIG_TARGET_STV0991) += stv0991.dtb
> >
> >  dtb-$(CONFIG_LS102XA) += ls1021a-qds-duart.dtb \
> > +       ls1021a-qds-lpuart.dtb \
> >         ls1021a-twr-duart.dtb ls1021a-twr-lpuart.dtb
> >  dtb-$(CONFIG_FSL_LSCH3) += fsl-ls2080a-qds.dtb \
> >         fsl-ls2080a-rdb.dtb
> > diff --git a/arch/arm/dts/ls1021a-qds-lpuart.dts
> > b/arch/arm/dts/ls1021a-qds-lpuart.dts
> > new file mode 100644
> > index 0000000..1d16ffd
> > --- /dev/null
> > +++ b/arch/arm/dts/ls1021a-qds-lpuart.dts
> > @@ -0,0 +1,16 @@
> > +/*
> > + * Freescale ls1021a QDS board common device tree source
> > + *
> > + * Copyright 2013-2015 Freescale Semiconductor, Inc.
> > + *
> > + * SPDX-License-Identifier:    GPL-2.0+
> > + */
> > +
> > +/dts-v1/;
> > +#include "ls1021a-qds.dtsi"
> > +
> > +/ {
> > +       chosen {
> > +               stdout-path = &lpuart0;
> > +       };
> > +};
> > diff --git a/configs/ls1021aqds_ddr4_nor_lpuart_defconfig
> > b/configs/ls1021aqds_ddr4_nor_lpuart_defconfig
> > index 44b2a0d..ffac6dd 100644
> > --- a/configs/ls1021aqds_ddr4_nor_lpuart_defconfig
> > +++ b/configs/ls1021aqds_ddr4_nor_lpuart_defconfig
> > @@ -1,7 +1,11 @@
> >  CONFIG_ARM=y
> >  CONFIG_TARGET_LS1021AQDS=y
> >  CONFIG_SYS_EXTRA_OPTIONS="SYS_FSL_DDR4,LPUART"
> > +CONFIG_DM_SERIAL=y
> > +CONFIG_DEFAULT_DEVICE_TREE="ls1021a-qds-duart"
> 
> I believe this should be "ls1021a-qds-lpuart"?
> 
> >  # CONFIG_CMD_SETEXPR is not set
> > +CONFIG_OF_CONTROL=y
> > +CONFIG_DM=y
> >  CONFIG_NETDEVICES=y
> >  CONFIG_E1000=y
> >  CONFIG_FSL_LPUART=y
> > diff --git a/configs/ls1021aqds_nor_lpuart_defconfig
> > b/configs/ls1021aqds_nor_lpuart_defconfig
> > index 1186af2..e81448d 100644
> > --- a/configs/ls1021aqds_nor_lpuart_defconfig
> > +++ b/configs/ls1021aqds_nor_lpuart_defconfig
> > @@ -1,7 +1,11 @@
> >  CONFIG_ARM=y
> >  CONFIG_TARGET_LS1021AQDS=y
> > +CONFIG_DM_SERIAL=y
> > +CONFIG_DEFAULT_DEVICE_TREE="ls1021a-qds-lpuart"
> >  CONFIG_SYS_EXTRA_OPTIONS="LPUART"
> >  # CONFIG_CMD_SETEXPR is not set
> > +CONFIG_OF_CONTROL=y
> > +CONFIG_DM=y
> >  CONFIG_NETDEVICES=y
> >  CONFIG_E1000=y
> >  CONFIG_FSL_LPUART=y
> > --
> 
[Alison Wang] I reviewed this set, and have no other comments except Bin Meng's.

BTW, I tested this set on LS1021AQDS board with that little modification about the comment.


Best Regards,
Alison Wang