[U-Boot] [PATCH 1/3] Makefile: Add a warning for boards that don't use CONFIG_BLK

Tom Rini trini at konsulko.com
Thu Nov 22 23:16:05 UTC 2018


On Thu, Nov 22, 2018 at 01:09:16PM -0700, Simon Glass wrote:

> The deadline for this has passed and we are starting to remove boards that
> do not use driver model for block devices.
> 
> Add a noisy Makefile warning.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> 
>  Makefile | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/Makefile b/Makefile
> index 552687db538..8bf422f6a85 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -917,6 +917,13 @@ ifeq ($(CONFIG_DM_I2C_COMPAT)$(CONFIG_SANDBOX),y)
>  	@echo "(possibly in a subsequent patch in your series)"
>  	@echo "before sending patches to the mailing list."
>  	@echo "===================================================="
> +endif
> +ifneq ($(CONFIG_BLK),y)
> +	@echo "===================== WARNING ======================"
> +	@echo "This board does not use CONFIG_BLK. Please update"
> +	@echo "the board to use CONFIG_BLK before the end of 2018."
> +	@echo "See doc/driver-model/MIGRATION.txt for more info."
> +	@echo "===================================================="
>  endif

The problem with this is that it doesn't catch most of the problem
combinations.  I'm still working on a series that highlights what the
problem cases we have left are.  But it's also not, in a lot of cases, a
board problem, but a lack of driver being updated problem so I suspect
we'll really be marking a bunch of stuff as depends on BROKEN.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20181122/61dea168/attachment.sig>


More information about the U-Boot mailing list