Message ID | 20220318035420.15058-16-samuel@sholland.org |
---|---|
State | Changes Requested |
Delegated to: | Andre Przywara |
Headers | show |
Series | sunxi: Add and use a pinctrl driver | expand |
On Thu, 17 Mar 2022 22:54:12 -0500 Samuel Holland <samuel@sholland.org> wrote: > This is now handled automatically by the clock and pinctrl drivers. > > SPL still calls this function because it needes the non-DM code. > > Signed-off-by: Samuel Holland <samuel@sholland.org> Looks good, but needs some testing to see if really cover all the clocks already. Anyway: Reviewed-by: Andre Przywara <andre.przywara@arm.com> Cheers, Andre > --- > > (no changes since v1) > > board/sunxi/board.c | 8 -------- > 1 file changed, 8 deletions(-) > > diff --git a/board/sunxi/board.c b/board/sunxi/board.c > index 05249d47eb..77ceefdb46 100644 > --- a/board/sunxi/board.c > +++ b/board/sunxi/board.c > @@ -241,14 +241,6 @@ int board_init(void) > } > } > > -#if CONFIG_IS_ENABLED(DM_I2C) > - /* > - * Temporary workaround for enabling I2C clocks until proper sunxi DM > - * clk, reset and pinctrl drivers land. > - */ > - i2c_init_board(); > -#endif > - > #ifdef CONFIG_DM_MMC > /* > * Temporary workaround for enabling MMC clocks until a sunxi DM
On 3/31/22 6:20 PM, Andre Przywara wrote: > On Thu, 17 Mar 2022 22:54:12 -0500 > Samuel Holland <samuel@sholland.org> wrote: > >> This is now handled automatically by the clock and pinctrl drivers. >> >> SPL still calls this function because it needes the non-DM code. >> >> Signed-off-by: Samuel Holland <samuel@sholland.org> > > Looks good, but needs some testing to see if really cover all the > clocks already. This is a good point. A31 and A23/A33 are still using the legacy PRCM MFD/clocks binding, so we will need drivers for those before we can merge this (and the P2WI/RSB patches). Regards, Samuel > Anyway: > Reviewed-by: Andre Przywara <andre.przywara@arm.com> > > Cheers, > Andre > >> --- >> >> (no changes since v1) >> >> board/sunxi/board.c | 8 -------- >> 1 file changed, 8 deletions(-) >> >> diff --git a/board/sunxi/board.c b/board/sunxi/board.c >> index 05249d47eb..77ceefdb46 100644 >> --- a/board/sunxi/board.c >> +++ b/board/sunxi/board.c >> @@ -241,14 +241,6 @@ int board_init(void) >> } >> } >> >> -#if CONFIG_IS_ENABLED(DM_I2C) >> - /* >> - * Temporary workaround for enabling I2C clocks until proper sunxi DM >> - * clk, reset and pinctrl drivers land. >> - */ >> - i2c_init_board(); >> -#endif >> - >> #ifdef CONFIG_DM_MMC >> /* >> * Temporary workaround for enabling MMC clocks until a sunxi DM >
diff --git a/board/sunxi/board.c b/board/sunxi/board.c index 05249d47eb..77ceefdb46 100644 --- a/board/sunxi/board.c +++ b/board/sunxi/board.c @@ -241,14 +241,6 @@ int board_init(void) } } -#if CONFIG_IS_ENABLED(DM_I2C) - /* - * Temporary workaround for enabling I2C clocks until proper sunxi DM - * clk, reset and pinctrl drivers land. - */ - i2c_init_board(); -#endif - #ifdef CONFIG_DM_MMC /* * Temporary workaround for enabling MMC clocks until a sunxi DM
This is now handled automatically by the clock and pinctrl drivers. SPL still calls this function because it needes the non-DM code. Signed-off-by: Samuel Holland <samuel@sholland.org> --- (no changes since v1) board/sunxi/board.c | 8 -------- 1 file changed, 8 deletions(-)