mbox series

[U-Boot,0/3] Support distro boot in pico-imx7d BL33 case

Message ID 1563262978-6855-1-git-send-email-jun.nie@linaro.org
Headers show
Series Support distro boot in pico-imx7d BL33 case | expand

Message

Jun Nie July 16, 2019, 7:42 a.m. UTC 
Support distro boot in pico-imx7d BL33 case with changing the enviroment
variables. While the other two patches are for polishing clock code and
for feature enablement.

Jun Nie (3):
  pico-imx7d: add config to enable CAAM
  pico-imx7d: Support distro boot for FIT image case
  pico-imx7d: polish uart clock id definition

 arch/arm/include/asm/arch-mx7/clock.h | 18 +----------------
 configs/pico-imx7d_bl33_defconfig     |  1 +
 include/configs/pico-imx7d.h          | 37 +++++++++++------------------------
 3 files changed, 13 insertions(+), 43 deletions(-)

Comments

Jun Nie July 26, 2019, 2:16 p.m. UTC | #1 
Jun Nie <jun.nie@linaro.org> 于2019年7月16日周二 下午3:43写道：
>
> Support distro boot in pico-imx7d BL33 case with changing the enviroment
> variables. While the other two patches are for polishing clock code and
> for feature enablement.
>

Hi,

any comments?

Thanks!
Jun
Jun Nie Aug. 8, 2019, 4:04 a.m. UTC | #2 
Jun Nie <jun.nie@linaro.org> 于2019年7月16日周二 下午3:43写道：
>
> Support distro boot in pico-imx7d BL33 case with changing the enviroment
> variables. While the other two patches are for polishing clock code and
> for feature enablement.
>
> Jun Nie (3):
>   pico-imx7d: add config to enable CAAM
>   pico-imx7d: Support distro boot for FIT image case
>   pico-imx7d: polish uart clock id definition
>
>  arch/arm/include/asm/arch-mx7/clock.h | 18 +----------------
>  configs/pico-imx7d_bl33_defconfig     |  1 +
>  include/configs/pico-imx7d.h          | 37 +++++++++++------------------------
>  3 files changed, 13 insertions(+), 43 deletions(-)
>
> --

Hi Vanessa,

Do you have any comments on these 3 patches? Or we can merge it now? Thanks!

Best Regards
Jun
Jun Nie Sept. 2, 2019, 8:09 a.m. UTC | #3 
Jun Nie <jun.nie@linaro.org> 于2019年8月8日周四 下午12:04写道：
>
> Jun Nie <jun.nie@linaro.org> 于2019年7月16日周二 下午3:43写道：
> >
> > Support distro boot in pico-imx7d BL33 case with changing the enviroment
> > variables. While the other two patches are for polishing clock code and
> > for feature enablement.
> >
> > Jun Nie (3):
> >   pico-imx7d: add config to enable CAAM
> >   pico-imx7d: Support distro boot for FIT image case
> >   pico-imx7d: polish uart clock id definition
> >
> >  arch/arm/include/asm/arch-mx7/clock.h | 18 +----------------
> >  configs/pico-imx7d_bl33_defconfig     |  1 +
> >  include/configs/pico-imx7d.h          | 37 +++++++++++------------------------
> >  3 files changed, 13 insertions(+), 43 deletions(-)
> >
> > --
>
> Hi Vanessa,
>
> Do you have any comments on these 3 patches? Or we can merge it now? Thanks!
>
> Best Regards
> Jun

Hi,

Does anyone has comments on these patches? Maybe we can merge them as
no comments for a long time. Thanks!

Regards,
Jun
Peng Fan Sept. 3, 2019, 1:39 a.m. UTC | #4 
Hi Jun,

> Subject: Re: [PATCH 0/3] Support distro boot in pico-imx7d BL33 case
> 
> Jun Nie <jun.nie@linaro.org> 于2019年8月8日周四 下午12:04写道：
> >
> > Jun Nie <jun.nie@linaro.org> 于2019年7月16日周二 下午3:43写道：
> > >
> > > Support distro boot in pico-imx7d BL33 case with changing the
> > > enviroment variables. While the other two patches are for polishing
> > > clock code and for feature enablement.
> > >
> > > Jun Nie (3):
> > >   pico-imx7d: add config to enable CAAM
> > >   pico-imx7d: Support distro boot for FIT image case
> > >   pico-imx7d: polish uart clock id definition
> > >
> > >  arch/arm/include/asm/arch-mx7/clock.h | 18 +----------------
> > >  configs/pico-imx7d_bl33_defconfig     |  1 +
> > >  include/configs/pico-imx7d.h          | 37
> +++++++++++------------------------
> > >  3 files changed, 13 insertions(+), 43 deletions(-)
> > >
> > > --
> >
> > Hi Vanessa,
> >
> > Do you have any comments on these 3 patches? Or we can merge it now?
> Thanks!
> >
> > Best Regards
> > Jun
> 
> Hi,
> 
> Does anyone has comments on these patches? Maybe we can merge them as
> no comments for a long time. Thanks!

I could merge this patchset into nxp-imx and send pull request to Stefano, if you
are fine with that.

Regards,
Peng. 

> 
> Regards,
> Jun
Jun Nie Sept. 3, 2019, 6:54 a.m. UTC | #5 
Peng Fan <peng.fan@nxp.com> 于2019年9月3日周二 上午9:39写道：
>
> Hi Jun,
>
> > Subject: Re: [PATCH 0/3] Support distro boot in pico-imx7d BL33 case
> >
> > Jun Nie <jun.nie@linaro.org> 于2019年8月8日周四 下午12:04写道：
> > >
> > > Jun Nie <jun.nie@linaro.org> 于2019年7月16日周二 下午3:43写道：
> > > >
> > > > Support distro boot in pico-imx7d BL33 case with changing the
> > > > enviroment variables. While the other two patches are for polishing
> > > > clock code and for feature enablement.
> > > >
> > > > Jun Nie (3):
> > > >   pico-imx7d: add config to enable CAAM
> > > >   pico-imx7d: Support distro boot for FIT image case
> > > >   pico-imx7d: polish uart clock id definition
> > > >
> > > >  arch/arm/include/asm/arch-mx7/clock.h | 18 +----------------
> > > >  configs/pico-imx7d_bl33_defconfig     |  1 +
> > > >  include/configs/pico-imx7d.h          | 37
> > +++++++++++------------------------
> > > >  3 files changed, 13 insertions(+), 43 deletions(-)
> > > >
> > > > --
> > >
> > > Hi Vanessa,
> > >
> > > Do you have any comments on these 3 patches? Or we can merge it now?
> > Thanks!
> > >
> > > Best Regards
> > > Jun
> >
> > Hi,
> >
> > Does anyone has comments on these patches? Maybe we can merge them as
> > no comments for a long time. Thanks!
>
> I could merge this patchset into nxp-imx and send pull request to Stefano, if you
> are fine with that.
>
> Regards,
> Peng.

Hi Peng,

I am fine. Thank you!

Jun
>
> >
> > Regards,
> > Jun
Jun Nie Sept. 9, 2019, 1:47 p.m. UTC | #6 
Peng Fan <peng.fan@nxp.com> 于2019年9月3日周二 上午9:39写道：
>
> Hi Jun,
>
> > Subject: Re: [PATCH 0/3] Support distro boot in pico-imx7d BL33 case
> >
> > Jun Nie <jun.nie@linaro.org> 于2019年8月8日周四 下午12:04写道：
> > >
> > > Jun Nie <jun.nie@linaro.org> 于2019年7月16日周二 下午3:43写道：
> > > >
> > > > Support distro boot in pico-imx7d BL33 case with changing the
> > > > enviroment variables. While the other two patches are for polishing
> > > > clock code and for feature enablement.
> > > >
> > > > Jun Nie (3):
> > > >   pico-imx7d: add config to enable CAAM
> > > >   pico-imx7d: Support distro boot for FIT image case
> > > >   pico-imx7d: polish uart clock id definition
> > > >
> > > >  arch/arm/include/asm/arch-mx7/clock.h | 18 +----------------
> > > >  configs/pico-imx7d_bl33_defconfig     |  1 +
> > > >  include/configs/pico-imx7d.h          | 37
> > +++++++++++------------------------
> > > >  3 files changed, 13 insertions(+), 43 deletions(-)
> > > >
> > > > --
> > >
> > > Hi Vanessa,
> > >
> > > Do you have any comments on these 3 patches? Or we can merge it now?
> > Thanks!
> > >
> > > Best Regards
> > > Jun
> >
> > Hi,
> >
> > Does anyone has comments on these patches? Maybe we can merge them as
> > no comments for a long time. Thanks!
>
> I could merge this patchset into nxp-imx and send pull request to Stefano, if you
> are fine with that.
>
> Regards,
> Peng.

Peng,

It is appreciated to help on this. Any comments from Stefan? Thanks!

Regards,
Jun
>
> >
> > Regards,
> > Jun
Peng Fan Sept. 11, 2019, 6:02 a.m. UTC | #7 
> -----Original Message-----
> From: Jun Nie <jun.nie@linaro.org>
> Sent: 2019年9月9日 21:47
> To: Peng Fan <peng.fan@nxp.com>
> Cc: Stefano Babic <sbabic@denx.de>; Fabio Estevam <festevam@gmail.com>;
> Otavio Salvador <otavio@ossystems.com.br>; Vanessa Maegima
> <vanessa.maegima@nxp.com>; Bryan O'Donoghue
> <bryan.odonoghue@linaro.org>; U-Boot Mailing List <u-boot@lists.denx.de>
> Subject: Re: [PATCH 0/3] Support distro boot in pico-imx7d BL33 case
> 
> Peng Fan <peng.fan@nxp.com> 于2019年9月3日周二 上午9:39写道：
> >
> > Hi Jun,
> >
> > > Subject: Re: [PATCH 0/3] Support distro boot in pico-imx7d BL33 case
> > >
> > > Jun Nie <jun.nie@linaro.org> 于2019年8月8日周四 下午12:04写道：
> > > >
> > > > Jun Nie <jun.nie@linaro.org> 于2019年7月16日周二 下午3:43写
> 道：
> > > > >
> > > > > Support distro boot in pico-imx7d BL33 case with changing the
> > > > > enviroment variables. While the other two patches are for
> > > > > polishing clock code and for feature enablement.
> > > > >
> > > > > Jun Nie (3):
> > > > >   pico-imx7d: add config to enable CAAM
> > > > >   pico-imx7d: Support distro boot for FIT image case
> > > > >   pico-imx7d: polish uart clock id definition
> > > > >
> > > > >  arch/arm/include/asm/arch-mx7/clock.h | 18 +----------------
> > > > >  configs/pico-imx7d_bl33_defconfig     |  1 +
> > > > >  include/configs/pico-imx7d.h          | 37
> > > +++++++++++------------------------
> > > > >  3 files changed, 13 insertions(+), 43 deletions(-)
> > > > >
> > > > > --
> > > >
> > > > Hi Vanessa,
> > > >
> > > > Do you have any comments on these 3 patches? Or we can merge it
> now?
> > > Thanks!
> > > >
> > > > Best Regards
> > > > Jun
> > >
> > > Hi,
> > >
> > > Does anyone has comments on these patches? Maybe we can merge them
> > > as no comments for a long time. Thanks!
> >
> > I could merge this patchset into nxp-imx and send pull request to
> > Stefano, if you are fine with that.
> >
> > Regards,
> > Peng.
> 
> Peng,
> 
> It is appreciated to help on this. Any comments from Stefan? Thanks!

CI build failed:
https://travis-ci.org/MrVan/u-boot/jobs/583445551
I need drop your patchset.

Thanks,
Peng.

> 
> Regards,
> Jun
> >
> > >
> > > Regards,
> > > Jun
Jun Nie Sept. 12, 2019, 10:40 a.m. UTC | #8 
Peng Fan <peng.fan@nxp.com> 于2019年9月11日周三 下午2:02写道：
>
>
>
> > -----Original Message-----
> > From: Jun Nie <jun.nie@linaro.org>
> > Sent: 2019年9月9日 21:47
> > To: Peng Fan <peng.fan@nxp.com>
> > Cc: Stefano Babic <sbabic@denx.de>; Fabio Estevam <festevam@gmail.com>;
> > Otavio Salvador <otavio@ossystems.com.br>; Vanessa Maegima
> > <vanessa.maegima@nxp.com>; Bryan O'Donoghue
> > <bryan.odonoghue@linaro.org>; U-Boot Mailing List <u-boot@lists.denx.de>
> > Subject: Re: [PATCH 0/3] Support distro boot in pico-imx7d BL33 case
> >
> > Peng Fan <peng.fan@nxp.com> 于2019年9月3日周二 上午9:39写道：
> > >
> > > Hi Jun,
> > >
> > > > Subject: Re: [PATCH 0/3] Support distro boot in pico-imx7d BL33 case
> > > >
> > > > Jun Nie <jun.nie@linaro.org> 于2019年8月8日周四 下午12:04写道：
> > > > >
> > > > > Jun Nie <jun.nie@linaro.org> 于2019年7月16日周二 下午3:43写
> > 道：
> > > > > >
> > > > > > Support distro boot in pico-imx7d BL33 case with changing the
> > > > > > enviroment variables. While the other two patches are for
> > > > > > polishing clock code and for feature enablement.
> > > > > >
> > > > > > Jun Nie (3):
> > > > > >   pico-imx7d: add config to enable CAAM
> > > > > >   pico-imx7d: Support distro boot for FIT image case
> > > > > >   pico-imx7d: polish uart clock id definition
> > > > > >
> > > > > >  arch/arm/include/asm/arch-mx7/clock.h | 18 +----------------
> > > > > >  configs/pico-imx7d_bl33_defconfig     |  1 +
> > > > > >  include/configs/pico-imx7d.h          | 37
> > > > +++++++++++------------------------
> > > > > >  3 files changed, 13 insertions(+), 43 deletions(-)
> > > > > >
> > > > > > --
> > > > >
> > > > > Hi Vanessa,
> > > > >
> > > > > Do you have any comments on these 3 patches? Or we can merge it
> > now?
> > > > Thanks!
> > > > >
> > > > > Best Regards
> > > > > Jun
> > > >
> > > > Hi,
> > > >
> > > > Does anyone has comments on these patches? Maybe we can merge them
> > > > as no comments for a long time. Thanks!
> > >
> > > I could merge this patchset into nxp-imx and send pull request to
> > > Stefano, if you are fine with that.
> > >
> > > Regards,
> > > Peng.
> >
> > Peng,
> >
> > It is appreciated to help on this. Any comments from Stefan? Thanks!
>
> CI build failed:
> https://travis-ci.org/MrVan/u-boot/jobs/583445551
> I need drop your patchset.
>
> Thanks,
> Peng.

Thanks for the result. Can you help to only drop the patch for
polishing clock? The three patches are not tightly coupled actually.

Regards,
Jun

>
> >
> > Regards,
> > Jun
> > >
> > > >
> > > > Regards,
> > > > Jun