On Wed, Aug 03, 2022 at 07:27:50PM +0200, Heinrich Schuchardt wrote:
> On 8/3/22 18:26, Quentin Schulz wrote:
> > Hi Tom,
> > 
> > On 8/3/22 18:08, Tom Rini wrote:
> > > On Wed, Aug 03, 2022 at 04:39:52PM +0200, Heinrich Schuchardt wrote:
> > > 
> > > > Only five boards are left that need to be migrated to DM_GPIO. Set a
> > > > 2023.04 deadline.
> > > > 
> > > > * edminiv2_defconfig
> > > > * mx28evk_auart_console_defconfig
> > > > * mx28evk_nand_defconfig
> > > > * mx28evk_spi_defconfig
> > > > * warp_defconfig
> > > > 
> > > > Signed-off-by: Heinrich Schuchardt <heinrich.schucha...@canonical.com>
> > > 
> > > I see you've now ack'd Fabio's patch to remove those mx28 configs.
> > > Here's my patch to remove edminiv2:
> > > https://patchwork.ozlabs.org/project/uboot/patch/20220802113424.2988464-6-tr...@konsulko.com/
> > > 
> > > That leaves just "warp", which is missing a ton of conversions, so let
> > > me just post a removal patch instead and see where it goes.
> > > 
> > 
> > I guess this would also mean the removal of GPIO drivers that haven't
> > moved to DM?
> > 
> > # git grep -L UCLASS drivers/gpio/*.c
> > drivers/gpio/adi_gpio2.c
> > drivers/gpio/kona_gpio.c
> > drivers/gpio/kw_gpio.c
> > drivers/gpio/mpc83xx_gpio.c
> > drivers/gpio/mvgpio.c
> > drivers/gpio/pca953x.c
> > drivers/gpio/pca9698.c
> > drivers/gpio/sh_pfc.c
> > drivers/gpio/tca642x.c
> 
> I tried to remove the drivers but that leads to errors in
> https://source.denx.de/u-boot/custodians/u-boot-efi/-/pipelines/13021
> 
> Here are some examples:
> 
> mx6sabreauto
> +make[3]: *** No rule to make target 'drivers/gpio/pca953x.o

This is a slightly trickier case as there's pca953x_gpio.c that is
DM_GPIO, so the question is if we have cases where SPL_DM is off (SPL_DM
is not required) and so the platform cannot switch over to the DM
driver there, but needs to for full U-Boot.  Maybe we need a new symbol,
CONFIG_SPL_PCA953X that depends on !SPL_DM_GPIO.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to