[PATCH v3] board: gateworks: venice: enable DM_SERIAL
Michael Nazzareno Trimarchi
michael at amarulasolutions.com
Fri Apr 15 07:43:22 CEST 2022
HI Adam
On Thu, Apr 14, 2022 at 8:44 PM Adam Ford <aford173 at gmail.com> wrote:
>
> On Thu, Apr 14, 2022 at 1:34 PM Tim Harvey <tharvey at gateworks.com> wrote:
> >
> > On Thu, Apr 14, 2022 at 11:02 AM Michael Nazzareno Trimarchi
> > <michael at amarulasolutions.com> wrote:
> > >
> > > HI Tim
> > >
> > > On Thu, Apr 14, 2022 at 7:53 PM Tim Harvey <tharvey at gateworks.com> wrote:
> > > >
> > > > Enable DM_SERIAL for both U_Boot and the SPL. The uart2 and its pinmux
> > > > are already marked with u-boot,dm-spl but we need to move the call to
> > > > preloader_console_init() after spl_early_init() to avoid a board hang
> > > > as dm can't be used until after spl_early_init().
> > > >
> > > > Remove the manual config of the UART pinmux now that it is no longer
> > > > needed.
> > > >
> > > > Signed-off-by: Tim Harvey <tharvey at gateworks.com>
> > > > Cc: Michael Nazzareno Trimarchi <michael at amarulasolutions.com>
> > > > ---
> > > > v3: enable DM_SERIAL for SPL as well per Michael's suggestion
> > > > v2: rebase on imx/master
> > > > ---
> > > > board/gateworks/venice/spl.c | 17 ++---------------
> > > > configs/imx8mm_venice_defconfig | 1 +
> > > > configs/imx8mn_venice_defconfig | 1 +
> > > > 3 files changed, 4 insertions(+), 15 deletions(-)
> > > >
> > > > diff --git a/board/gateworks/venice/spl.c b/board/gateworks/venice/spl.c
> > > > index b56e1b607d58..59a6a29f70d8 100644
> > > > --- a/board/gateworks/venice/spl.c
> > > > +++ b/board/gateworks/venice/spl.c
> > > > @@ -87,25 +87,14 @@ static void spl_dram_init(int size)
> > > > ddr_init(dram_timing);
> > > > }
> > > >
> > > > -#define UART_PAD_CTRL (PAD_CTL_DSE6 | PAD_CTL_FSEL1)
> > > > #define WDOG_PAD_CTRL (PAD_CTL_DSE6 | PAD_CTL_ODE | PAD_CTL_PUE | PAD_CTL_PE)
> > > >
> > > > #ifdef CONFIG_IMX8MM
> > > > -static iomux_v3_cfg_t const uart_pads[] = {
> > > > - IMX8MM_PAD_UART2_RXD_UART2_RX | MUX_PAD_CTRL(UART_PAD_CTRL),
> > > > - IMX8MM_PAD_UART2_TXD_UART2_TX | MUX_PAD_CTRL(UART_PAD_CTRL),
> > > > -};
> > > > -
> > >
> > >
> > > What about the clock? I'm a bit surprised that it works without the
> > > clock enabled.
> > >
> >
> > Michael,
> >
> > The clock hasn't changed. Like Peng's series for converting
> > imx8mm-evk, imx8mn-evk, and imx8mp-evk to DM_SERIAL, the
> > board_init_f() function still calls 'init_uart_clk(1)' for UART2
> > (index is 0-based UART) and that is still required. I'm not sure why
> > the dm IMX driver can't do that but it doesn't currently and that
> > could always be an imx8mm general cleanup later.
>
> I think that's because the clock driver doesn't have the uart clocks
> in it. I was investigating the hanging in SPL with DM_SERIAL and
> noticed from the clk dump that the uart clocks were not present, so I
> added them to the clock driver, but I didn't have success. I also
> tried moving the preloader_console_init and switching to early spl
> init instead of a regular one. Because I wasn't successful, I didn't
> do anything with those patches, but I can post them to the mailing
> list if there is interest.
I have sent mine but I was thinking of doing the same. What I found
that is really
strange
commit b41d4b83f0203be79a3bfa1e4bde316355c7f2a0
Author: Sean Anderson <seanga2 at gmail.com>
Date: Sun Feb 2 13:15:17 2020 -0500
serial: Set baudrate on boot
Currently, the baud rate is never set on boot. This works ok when a previous
bootloader has configured the baudrate properly, or when the
baudrate is set to
a reasonable default in the serial driver's probe(). However, when
this is not
the case, we could be using a different baud rate than what was configured.
Signed-off-by: Sean Anderson <seanga2 at gmail.com>
Where I don't know if the probe is called later and we can not set or
get the clock from there. imx7d
enable all the clock for clock serial in clk init
Michael
>
> adam
> >
> > Are your questions because you are trying to implement DM_SERIAL for
> > one of your boards and having issues? If so, please submit your patch
> > as an RFC so we can try to help.
> >
> > Best Regards,
> >
> > Tim
> >
> > > Michael
> > >
> > > > static iomux_v3_cfg_t const wdog_pads[] = {
> > > > IMX8MM_PAD_GPIO1_IO02_WDOG1_WDOG_B | MUX_PAD_CTRL(WDOG_PAD_CTRL),
> > > > };
> > > > #endif
> > > > #ifdef CONFIG_IMX8MN
> > > > -static const iomux_v3_cfg_t uart_pads[] = {
> > > > - IMX8MN_PAD_UART2_RXD__UART2_DCE_RX | MUX_PAD_CTRL(UART_PAD_CTRL),
> > > > - IMX8MN_PAD_UART2_TXD__UART2_DCE_TX | MUX_PAD_CTRL(UART_PAD_CTRL),
> > > > -};
> > > > -
> > > > static const iomux_v3_cfg_t wdog_pads[] = {
> > > > IMX8MN_PAD_GPIO1_IO02__WDOG1_WDOG_B | MUX_PAD_CTRL(WDOG_PAD_CTRL),
> > > > };
> > > > @@ -119,8 +108,6 @@ int board_early_init_f(void)
> > > >
> > > > set_wdog_reset(wdog);
> > > >
> > > > - imx_iomux_v3_setup_multiple_pads(uart_pads, ARRAY_SIZE(uart_pads));
> > > > -
> > > > return 0;
> > > > }
> > > >
> > > > @@ -232,8 +219,6 @@ void board_init_f(ulong dummy)
> > > >
> > > > timer_init();
> > > >
> > > > - preloader_console_init();
> > > > -
> > > > /* Clear the BSS. */
> > > > memset(__bss_start, 0, __bss_end - __bss_start);
> > > >
> > > > @@ -243,6 +228,8 @@ void board_init_f(ulong dummy)
> > > > hang();
> > > > }
> > > >
> > > > + preloader_console_init();
> > > > +
> > > > ret = uclass_get_device_by_name(UCLASS_CLK,
> > > > "clock-controller at 30380000",
> > > > &dev);
> > > > diff --git a/configs/imx8mm_venice_defconfig b/configs/imx8mm_venice_defconfig
> > > > index dd61ec9b70fb..1ccbe9970a6c 100644
> > > > --- a/configs/imx8mm_venice_defconfig
> > > > +++ b/configs/imx8mm_venice_defconfig
> > > > @@ -110,6 +110,7 @@ CONFIG_SPL_DM_PMIC_MP5416=y
> > > > CONFIG_DM_REGULATOR=y
> > > > CONFIG_DM_REGULATOR_FIXED=y
> > > > CONFIG_DM_REGULATOR_GPIO=y
> > > > +CONFIG_DM_SERIAL=y
> > > > CONFIG_MXC_UART=y
> > > > CONFIG_SYSRESET=y
> > > > CONFIG_SPL_SYSRESET=y
> > > > diff --git a/configs/imx8mn_venice_defconfig b/configs/imx8mn_venice_defconfig
> > > > index c3a96a378553..ff926dac0e18 100644
> > > > --- a/configs/imx8mn_venice_defconfig
> > > > +++ b/configs/imx8mn_venice_defconfig
> > > > @@ -108,6 +108,7 @@ CONFIG_SPL_DM_PMIC_MP5416=y
> > > > CONFIG_DM_REGULATOR=y
> > > > CONFIG_DM_REGULATOR_FIXED=y
> > > > CONFIG_DM_REGULATOR_GPIO=y
> > > > +CONFIG_DM_SERIAL=y
> > > > CONFIG_MXC_UART=y
> > > > CONFIG_SYSRESET=y
> > > > CONFIG_SPL_SYSRESET=y
> > > > --
> > > > 2.17.1
> > > >
> > >
> > >
> > > --
> > > Michael Nazzareno Trimarchi
> > > Co-Founder & Chief Executive Officer
> > > M. +39 347 913 2170
> > > michael at amarulasolutions.com
> > > __________________________________
> > >
> > > Amarula Solutions BV
> > > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> > > T. +31 (0)85 111 9172
> > > info at amarulasolutions.com
> > > www.amarulasolutions.com
--
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael at amarulasolutions.com
__________________________________
Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info at amarulasolutions.com
www.amarulasolutions.com
More information about the U-Boot
mailing list