[RFC PATCH 1/1] Makefile: Expand legacy (non-DM) driver warnings

Niel Fourie lusus at denx.de
Mon Mar 30 18:07:20 CEST 2020

Hi Simon

On 3/28/20 9:05 PM, Simon Glass wrote:
> Hi Niel,
> On Wed, 25 Mar 2020 at 07:47, Niel Fourie <lusus at denx.de> wrote:
>> Expand warnings printed by Makefile after compile when legacy
>> drivers are in use. These include:
>> - CONFIG_POWER, also without CONFIG_DM_PMIC
>> - Absence of CONFIG_RAM and CONFIG_SPL_RAM
>> Also replaced existing CONFIG_DM_SPI warning for consistency.
>> Removed CONFIG_BLK requirement for CONFIG_DM_USB, as all USB
>> devices not block devices.
>> Signed-off-by: Niel Fourie <lusus at denx.de>
>> Cc: Simon Glass <sjg at chromium.org>
>> ---
>>   Makefile | 73 +++++++++++++++++++++++++++++++++++++++++++++++++++-----
>>   1 file changed, 67 insertions(+), 6 deletions(-)
> Could we add instructions on what should be done? It seems a little
> unclear to me.

Yes, sure. I am still a little uncertain on how to correctly create a 
sensible RFC patch.

My question is basically: How useful would you consider having more of 
these warnings in the Makefile, if at all? I am the least certain of the 
last one with CONFIG_RAM and CONFIG_SPL_RAM.

For background, I explored the driver model and then I had a look at how 
much legacy there was still around. I found some further defines and 
conditions which could be turned into legacy warnings in the Makefile 
for some more visibility. Any even further such warnings would mostly 
involve checking the defines for the individual legacy drivers, which I 
do not really consider viable.

Testing of these warnings, especially automated, would be a challenge. I 
have a heavily butchered Sandbox build which triggers most of the legacy 
warnings in the Makefile, but I would not want to inflict it on anybody 

Best regard,
Niel Fourie

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-21 Fax: +49-8142-66989-80  Email: lusus at denx.de

More information about the U-Boot mailing list