mbox series

[U-Boot,V6,0/2] Update sabrelite and nitrogen6x boards to use distro boot support

Message ID 20180418150459.2083-1-guillaume.gardet@free.fr
Headers show
Series Update sabrelite and nitrogen6x boards to use distro boot support | expand

Message

Guillaume GARDET April 18, 2018, 3:04 p.m. UTC
This patch serie updates sabrelite and nitrogen6x boards to use distro boot support.
Sabrelite has been boot tested with boot.scr script and EFI/Grub2 on mmc0 and mmc1 slots.
Nitrogen6* boards have been build tested only.

Currently, only the Sabrelite has fdtfile defined.

Signed-off-by: Guillaume GARDET <guillaume.gardet@free.fr>
Cc: Troy Kisky <troy.kisky@boundarydevices.com>
Cc: Stefano Babic <sbabic@denx.de>
Cc: Fabio Estevam <fabio.estevam@nxp.com>
Cc: Gary Bisson <gary.bisson@boundarydevices.com>

Changes in V6:
  * added CONFIG_CMD_GPT to sabrelite defconfig
  * remove CONFIG_ARCH_MISC_INIT to sabrelite defconfig

Changes in V5:
  * remove obsolete code from include/configs/nitrogen6x.h
  
Changes in V4:
  * Remove imx6 soc definition
  * Change comment on not defined fdtfile (no more fallback)

Changes in V3:
  * Add imx6 soc definition
  * Also update nitrogen6x config, not only sabrelite
  * Split mx6qsabrelite_defconfig update to a separate patch
Changes in V2:
  * add mx6qsabrelite_defconfig update


Guillaume GARDET (2):
  imx6: Convert sabrelite and nitrogen6x boards to distro boot support
  imx6: sabrelite: update defconfig to use distro defaults

 configs/mx6qsabrelite_defconfig |  15 ++--
 include/configs/nitrogen6x.h    | 179 ++++++++++------------------------------
 2 files changed, 48 insertions(+), 146 deletions(-)

Comments

Guillaume GARDET May 16, 2018, 7:20 a.m. UTC | #1
Hi,

just a friendly reminder for those 2 patches.


Guillaume



Le 18/04/2018 à 17:04, Guillaume GARDET a écrit :
> This patch serie updates sabrelite and nitrogen6x boards to use distro boot support.
> Sabrelite has been boot tested with boot.scr script and EFI/Grub2 on mmc0 and mmc1 slots.
> Nitrogen6* boards have been build tested only.
>
> Currently, only the Sabrelite has fdtfile defined.
>
> Signed-off-by: Guillaume GARDET <guillaume.gardet@free.fr>
> Cc: Troy Kisky <troy.kisky@boundarydevices.com>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Fabio Estevam <fabio.estevam@nxp.com>
> Cc: Gary Bisson <gary.bisson@boundarydevices.com>
>
> Changes in V6:
>    * added CONFIG_CMD_GPT to sabrelite defconfig
>    * remove CONFIG_ARCH_MISC_INIT to sabrelite defconfig
>
> Changes in V5:
>    * remove obsolete code from include/configs/nitrogen6x.h
>    
> Changes in V4:
>    * Remove imx6 soc definition
>    * Change comment on not defined fdtfile (no more fallback)
>
> Changes in V3:
>    * Add imx6 soc definition
>    * Also update nitrogen6x config, not only sabrelite
>    * Split mx6qsabrelite_defconfig update to a separate patch
> Changes in V2:
>    * add mx6qsabrelite_defconfig update
>
>
> Guillaume GARDET (2):
>    imx6: Convert sabrelite and nitrogen6x boards to distro boot support
>    imx6: sabrelite: update defconfig to use distro defaults
>
>   configs/mx6qsabrelite_defconfig |  15 ++--
>   include/configs/nitrogen6x.h    | 179 ++++++++++------------------------------
>   2 files changed, 48 insertions(+), 146 deletions(-)
>
Stefano Babic May 17, 2018, 9:51 a.m. UTC | #2
Hi Guillaume,

On 18/04/2018 17:04, Guillaume GARDET wrote:
> This patch serie updates sabrelite and nitrogen6x boards to use distro boot support.
> Sabrelite has been boot tested with boot.scr script and EFI/Grub2 on mmc0 and mmc1 slots.
> Nitrogen6* boards have been build tested only.
> 
> Currently, only the Sabrelite has fdtfile defined.
> 

Fine, but Troy is the maintainer for this board and I have not yet seen
if he agrees to switch the board to the distro environment. I would like
to have his ACK before pushing this.

Best regards,
Stefano Babic

> Signed-off-by: Guillaume GARDET <guillaume.gardet@free.fr>
> Cc: Troy Kisky <troy.kisky@boundarydevices.com>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Fabio Estevam <fabio.estevam@nxp.com>
> Cc: Gary Bisson <gary.bisson@boundarydevices.com>
> 
> Changes in V6:
>   * added CONFIG_CMD_GPT to sabrelite defconfig
>   * remove CONFIG_ARCH_MISC_INIT to sabrelite defconfig
> 
> Changes in V5:
>   * remove obsolete code from include/configs/nitrogen6x.h
>   
> Changes in V4:
>   * Remove imx6 soc definition
>   * Change comment on not defined fdtfile (no more fallback)
> 
> Changes in V3:
>   * Add imx6 soc definition
>   * Also update nitrogen6x config, not only sabrelite
>   * Split mx6qsabrelite_defconfig update to a separate patch
> Changes in V2:
>   * add mx6qsabrelite_defconfig update
> 
> 
> Guillaume GARDET (2):
>   imx6: Convert sabrelite and nitrogen6x boards to distro boot support
>   imx6: sabrelite: update defconfig to use distro defaults
> 
>  configs/mx6qsabrelite_defconfig |  15 ++--
>  include/configs/nitrogen6x.h    | 179 ++++++++++------------------------------
>  2 files changed, 48 insertions(+), 146 deletions(-)
>
Troy Kisky May 17, 2018, 3:45 p.m. UTC | #3
On 5/17/2018 2:51 AM, Stefano Babic wrote:
> Hi Guillaume,
> 
> On 18/04/2018 17:04, Guillaume GARDET wrote:
>> This patch serie updates sabrelite and nitrogen6x boards to use distro boot support.
>> Sabrelite has been boot tested with boot.scr script and EFI/Grub2 on mmc0 and mmc1 slots.
>> Nitrogen6* boards have been build tested only.
>>
>> Currently, only the Sabrelite has fdtfile defined.
>>
> 
> Fine, but Troy is the maintainer for this board and I have not yet seen
> if he agrees to switch the board to the distro environment. I would like
> to have his ACK before pushing this.
> 
> Best regards,
> Stefano Babic

Sorry, for the delay

Acked-by: Troy Kisky <troy.kisky@boundarydevices.com>
Stefano Babic May 17, 2018, 3:46 p.m. UTC | #4
On 17/05/2018 17:45, Troy Kisky wrote:
> On 5/17/2018 2:51 AM, Stefano Babic wrote:
>> Hi Guillaume,
>>
>> On 18/04/2018 17:04, Guillaume GARDET wrote:
>>> This patch serie updates sabrelite and nitrogen6x boards to use distro boot support.
>>> Sabrelite has been boot tested with boot.scr script and EFI/Grub2 on mmc0 and mmc1 slots.
>>> Nitrogen6* boards have been build tested only.
>>>
>>> Currently, only the Sabrelite has fdtfile defined.
>>>
>>
>> Fine, but Troy is the maintainer for this board and I have not yet seen
>> if he agrees to switch the board to the distro environment. I would like
>> to have his ACK before pushing this.
>>
>> Best regards,
>> Stefano Babic
> 
> Sorry, for the delay

No worry.
> 
> Acked-by: Troy Kisky <troy.kisky@boundarydevices.com>
> 

I apply the patches to u-boot-imx, thanks !

Best regards,
Stefano