[U-Boot] [PATCH 4/5] efi_loader: gop: fixes for CONFIG_DM_VIDEO without CONFIG_LCD

[Simon Glass]

Hi Alex,

On 18 July 2017 at 07:47, Alexander Graf <agraf at suse.de> wrote:
> On 07/18/2017 04:00 PM, Simon Glass wrote:
>>
>> Hi,
>>
>> On 12 July 2017 at 05:52, Alexander Graf <agraf at suse.de> wrote:
>>>
>>>
>>> On 25.06.17 01:05, Rob Clark wrote:
>>>>
>>>> Signed-off-by: Rob Clark <robdclark at gmail.com>
>>>> Cc: Alexander Graf <agraf at suse.de>
>>>
>>>
>>> Looks reasonable to me, but could probably use a commit message ;). Also
>>> please make sure to CC Simon on all things DM.
>>>
>> Can we drop the CONFIG_LCD support entirely? This is legacy code at
>> this point. What boards use it?
>
>
> Sounds like someone would first need to convert a bunch of boards :).
>
> $ for i in $(grep CONFIG_LCD configs/* | cut -d : -f 1); do grep -q DM_VIDEO
> $i || echo $i; done
> configs/at91sam9261ek_dataflash_cs0_defconfig
> configs/at91sam9261ek_dataflash_cs3_defconfig
> configs/at91sam9261ek_nandflash_defconfig
> configs/at91sam9263ek_dataflash_cs0_defconfig
> configs/at91sam9263ek_dataflash_defconfig
> configs/at91sam9263ek_nandflash_defconfig
> configs/at91sam9263ek_norflash_boot_defconfig
> configs/at91sam9263ek_norflash_defconfig
> configs/at91sam9g10ek_dataflash_cs0_defconfig
> configs/at91sam9g10ek_dataflash_cs3_defconfig
> configs/at91sam9g10ek_nandflash_defconfig
> configs/at91sam9m10g45ek_mmc_defconfig
> configs/at91sam9m10g45ek_nandflash_defconfig
> configs/at91sam9n12ek_mmc_defconfig
> configs/at91sam9n12ek_nandflash_defconfig
> configs/at91sam9n12ek_spiflash_defconfig
> configs/at91sam9rlek_dataflash_defconfig
> configs/at91sam9rlek_mmc_defconfig
> configs/at91sam9rlek_nandflash_defconfig
> configs/at91sam9x5ek_dataflash_defconfig
> configs/at91sam9x5ek_mmc_defconfig
> configs/at91sam9x5ek_nandflash_defconfig
> configs/at91sam9x5ek_spiflash_defconfig
> configs/brppt1_mmc_defconfig
> configs/brppt1_nand_defconfig
> configs/brppt1_spi_defconfig
> configs/brxre1_defconfig
> configs/cm_t3517_defconfig
> configs/cm_t35_defconfig
> configs/picosam9g45_defconfig
> configs/pm9261_defconfig
> configs/pm9263_defconfig
> configs/sama5d36ek_cmp_mmc_defconfig
> configs/sama5d36ek_cmp_nandflash_defconfig
> configs/sama5d36ek_cmp_spiflash_defconfig
> configs/sama5d3xek_mmc_defconfig
> configs/sama5d3xek_nandflash_defconfig
> configs/sama5d3xek_spiflash_defconfig
> configs/sama5d4ek_mmc_defconfig
> configs/sama5d4ek_nandflash_defconfig
> configs/sama5d4ek_spiflash_defconfig
> configs/zipitz2_defconfig

Not really. I suspect none of those uses EFI_LOADER

There is video driver for atmel which is most of the boards in that
list, but we can disable EFI_LOADER until they are converted.

We should avoid adding new features to legacy code paths as it makes
DM conversion harder and less likely to complete.

Regards,
Simon