[U-Boot] [PATCH] dm: core: Allow simple-bus driver in SPL

[U-Boot] [PATCH] dm: core: Allow simple-bus driver in SPL

[Marek Vasut]

There's no reason why simple-bus driver can not be used in SPL,
in fact it is necessary to get SoCFPGA SPL probe the cadence
SPI driver. So drop the restriction.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Simon Glass <sjg@chromium.org>
---
 drivers/core/Makefile | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/core/Makefile b/drivers/core/Makefile
index 5c2ead8..87b7c98 100644
--- a/drivers/core/Makefile
+++ b/drivers/core/Makefile
@@ -5,9 +5,7 @@
 #
 
 obj-y	+= device.o lists.o root.o uclass.o util.o
-ifndef CONFIG_SPL_BUILD
 obj-$(CONFIG_OF_CONTROL) += simple-bus.o
-endif
 obj-$(CONFIG_DM_DEVICE_REMOVE)	+= device-remove.o
 obj-$(CONFIG_DM)	+= dump.o
 obj-$(CONFIG_OF_CONTROL)	+= regmap.o
-- 
2.1.4

[U-Boot] [PATCH] dm: core: Allow simple-bus driver in SPL

[Simon Glass]

Hi Marek,

On 27 July 2015 at 14:31, Marek Vasut <marex@denx.de> wrote:
> There's no reason why simple-bus driver can not be used in SPL,
> in fact it is necessary to get SoCFPGA SPL probe the cadence
> SPI driver. So drop the restriction.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Simon Glass <sjg@chromium.org>
> ---
>  drivers/core/Makefile | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/core/Makefile b/drivers/core/Makefile
> index 5c2ead8..87b7c98 100644
> --- a/drivers/core/Makefile
> +++ b/drivers/core/Makefile
> @@ -5,9 +5,7 @@
>  #
>
>  obj-y  += device.o lists.o root.o uclass.o util.o
> -ifndef CONFIG_SPL_BUILD
>  obj-$(CONFIG_OF_CONTROL) += simple-bus.o
> -endif
>  obj-$(CONFIG_DM_DEVICE_REMOVE) += device-remove.o
>  obj-$(CONFIG_DM)       += dump.o
>  obj-$(CONFIG_OF_CONTROL)       += regmap.o
> --
> 2.1.4
>

Can you please instead add a new Kconfigs CONFIG_SIMPLE_BUS and
CONFIG_SPL_SIMPLE_BUS? They can be default on. If we force all SPLs to
have this, it does add bloat.

Regards,
Simon

[U-Boot] [PATCH] dm: core: Allow simple-bus driver in SPL

[Marek Vasut]

On Sunday, August 02, 2015 at 11:28:10 PM, Simon Glass wrote:
> Hi Marek,
> 
> On 27 July 2015 at 14:31, Marek Vasut <marex@denx.de> wrote:
> > There's no reason why simple-bus driver can not be used in SPL,
> > in fact it is necessary to get SoCFPGA SPL probe the cadence
> > SPI driver. So drop the restriction.
> > 
> > Signed-off-by: Marek Vasut <marex@denx.de>
> > Cc: Simon Glass <sjg@chromium.org>
> > ---
> > 
> >  drivers/core/Makefile | 2 --
> >  1 file changed, 2 deletions(-)
> > 
> > diff --git a/drivers/core/Makefile b/drivers/core/Makefile
> > index 5c2ead8..87b7c98 100644
> > --- a/drivers/core/Makefile
> > +++ b/drivers/core/Makefile
> > @@ -5,9 +5,7 @@
> > 
> >  #
> >  
> >  obj-y  += device.o lists.o root.o uclass.o util.o
> > 
> > -ifndef CONFIG_SPL_BUILD
> > 
> >  obj-$(CONFIG_OF_CONTROL) += simple-bus.o
> > 
> > -endif
> > 
> >  obj-$(CONFIG_DM_DEVICE_REMOVE) += device-remove.o
> >  obj-$(CONFIG_DM)       += dump.o
> >  obj-$(CONFIG_OF_CONTROL)       += regmap.o
> > 
> > --
> > 2.1.4
> 
> Can you please instead add a new Kconfigs CONFIG_SIMPLE_BUS and
> CONFIG_SPL_SIMPLE_BUS? They can be default on. If we force all SPLs to
> have this, it does add bloat.

Done

Best regards,
Marek Vasut