[U-Boot] [PATCH v7 4/6] drivers: Enable FPGA driver build on SPL

Marek Vasut marex at denx.de
Sat Jun 3 11:29:09 UTC 2017


On 05/29/2017 06:00 AM, tien.fong.chee at intel.com wrote:
> From: Tien Fong Chee <tien.fong.chee at intel.com>
> 
> Enable FPGA driver build for SPL because FPGA driver is needed for SPL
> to configure and getting DDR up before loading U-boot into DDR and
> booting from there.
> 
> Signed-off-by: Tien Fong Chee <tien.fong.chee at intel.com>

The thing I fail to understand is why you're sending the patches in
seemingly random order. You add/change piece of code, then enable
something and then add the relevant code you're enabling. I really do
not understand that.

The patches should have some logical order -- change existing code, add
new code, enable stuff.

Anyway, please collect the Acks, fix the nit and resubmit.

> ---
>  drivers/Makefile | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/Makefile b/drivers/Makefile
> index 64c39d3..4478212 100644
> --- a/drivers/Makefile
> +++ b/drivers/Makefile
> @@ -48,6 +48,7 @@ obj-$(CONFIG_OMAP_USB_PHY) += usb/phy/
>  obj-$(CONFIG_SPL_SATA_SUPPORT) += block/
>  obj-$(CONFIG_SPL_USB_HOST_SUPPORT) += block/
>  obj-$(CONFIG_SPL_MMC_SUPPORT) += block/
> +obj-$(CONFIG_SPL_FPGA_SUPPORT) += fpga/
>  endif
>  
>  ifdef CONFIG_TPL_BUILD
> 


-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list