Hi Gaurav
On Sat, Mar 19, 2022 at 10:47 AM Michael Nazzareno Trimarchi
<mich...@amarulasolutions.com> wrote:
>
> HI
>
> Please send me a link to apply your series
>
> Michael
>
> On Thu, Mar 17, 2022 at 1:10 PM Gaurav Jain <gaurav.j...@nxp.com> wrote:
> >
> > Hello Michael
> >
> > A gentle reminder!!
> > As you said earlier, can you test the mamoj board with proposed solution to
> > fix SPL size issue.
> >
> > Regards
> > Gaurav Jain
> >
> > > -----Original Message-----
> > > From: Gaurav Jain
> > > Sent: Monday, March 14, 2022 11:18 AM
> > > To: Michael Nazzareno Trimarchi <mich...@amarulasolutions.com>; Stefano
> > > Babic <sba...@denx.de>
> > > Cc: Tom Rini <tr...@konsulko.com>; Fabio Estevam <feste...@gmail.com>;
> > > Michael Walle <mich...@walle.cc>; Varun Sethi <v.se...@nxp.com>; Adrian
> > > Alonso <adrian.alo...@nxp.com>; Alison Wang <alison.w...@nxp.com>;
> > > Andy Tang <andy.t...@nxp.com>; Franck Lenormand
> > > <franck.lenorm...@nxp.com>; Horia Geanta <horia.gea...@nxp.com>; Ji
> > > Luo <ji....@nxp.com>; ma...@denx.de; Meenakshi Aggarwal
> > > <meenakshi.aggar...@nxp.com>; Mingkai Hu <mingkai...@nxp.com>;
> > > olte...@gmail.com; Pankaj Gupta <pankaj.gu...@nxp.com>; Peng Fan
> > > <peng....@nxp.com>; Pramod Kumar <pramod.kuma...@nxp.com>;
> > > Priyanka Jain <priyanka.j...@nxp.com>; Rajesh Bhagat
> > > <rajesh.bha...@nxp.com>; Sahil Malhotra <sahil.malho...@nxp.com>;
> > > Shengzhou Liu <shengzhou....@nxp.com>; Silvano Di Ninno
> > > <silvano.dini...@nxp.com>; s...@chromium.org; u-boot@lists.denx.de; dl-
> > > uboot-imx <uboot-...@nxp.com>; Wasim Khan <wasim.k...@nxp.com>;
> > > Ye Li <ye...@nxp.com>; Raffaele RECALCATI <raffaele.recalc...@bticino.it>;
> > > Jagan Teki <ja...@amarulasolutions.com>
> > > Subject: RE: [EXT] Re: [PATCH v8 01/15] crypto/fsl: Add support for CAAM
> > > Job
> > > ring driver model
> > >
> > > Hi Michael
> > >
> > > > -----Original Message-----
> > > > From: Michael Nazzareno Trimarchi <mich...@amarulasolutions.com>
> > > > Sent: Tuesday, March 8, 2022 7:05 PM
> > > > To: Stefano Babic <sba...@denx.de>
> > > > Cc: Tom Rini <tr...@konsulko.com>; Fabio Estevam
> > > <feste...@gmail.com>;
> > > > Gaurav Jain <gaurav.j...@nxp.com>; Michael Walle <mich...@walle.cc>;
> > > > Varun Sethi <v.se...@nxp.com>; Adrian Alonso <adrian.alo...@nxp.com>;
> > > > Alison Wang <alison.w...@nxp.com>; Andy Tang <andy.t...@nxp.com>;
> > > > Franck Lenormand <franck.lenorm...@nxp.com>; Horia Geanta
> > > > <horia.gea...@nxp.com>; Ji Luo <ji....@nxp.com>; ma...@denx.de;
> > > > Meenakshi Aggarwal <meenakshi.aggar...@nxp.com>; Mingkai Hu
> > > > <mingkai...@nxp.com>; olte...@gmail.com; Pankaj Gupta
> > > > <pankaj.gu...@nxp.com>; Peng Fan <peng....@nxp.com>; Pramod Kumar
> > > > <pramod.kuma...@nxp.com>; Priyanka Jain <priyanka.j...@nxp.com>;
> > > > Rajesh Bhagat <rajesh.bha...@nxp.com>; Sahil Malhotra
> > > > <sahil.malho...@nxp.com>; Shengzhou Liu <shengzhou....@nxp.com>;
> > > > Silvano Di Ninno <silvano.dini...@nxp.com>; s...@chromium.org; u-
> > > > b...@lists.denx.de; dl-uboot-imx <uboot-...@nxp.com>; Wasim Khan
> > > > <wasim.k...@nxp.com>; Ye Li <ye...@nxp.com>; Raffaele RECALCATI
> > > > <raffaele.recalc...@bticino.it>; Jagan Teki
> > > > <ja...@amarulasolutions.com>
> > > > Subject: Re: [EXT] Re: [PATCH v8 01/15] crypto/fsl: Add support for
> > > > CAAM Job ring driver model
> > > >
> > > > Caution: EXT Email
> > > >
> > > > Hi
> > > >
> > > > On Tue, Mar 8, 2022 at 2:28 PM Stefano Babic <sba...@denx.de> wrote:
> > > > >
> > > > > Hi Tom,
> > > > >
> > > > > On 08.03.22 13:48, Tom Rini wrote:
> > > > > > On Tue, Mar 08, 2022 at 08:12:27AM -0300, Fabio Estevam wrote:
> > > > > >> On Tue, Mar 8, 2022 at 8:10 AM Gaurav Jain <gaurav.j...@nxp.com>
> > > > wrote:
> > > > > >>
> > > > > >>> I further checked on your concern and propose the below change
> > > > > >>> to
> > > > stop building caam driver in SPL for imx6dl_mamoj.
> > > > > >>>
> > > > > >>> --- a/configs/imx6dl_mamoj_defconfig
> > > > > >>> +++ b/configs/imx6dl_mamoj_defconfig
> > > > > >>> @@ -61,3 +61,4 @@ CONFIG_USB_GADGET_MANUFACTURER="FSL"
> > > > > >>> CONFIG_USB_GADGET_VENDOR_NUM=0x0525
> > > > > >>> CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5
> > > > > >>> CONFIG_CI_UDC=y
> > > > > >>> +CONFIG_SPL_CRYPTO=n
> > > > > >>
> > > > > >> No, this is not how defconfig works.
> > > > > >>
> > > > > >> You should set:
> > > > > >>
> > > > > >> # CONFIG_SPL_CRYPTO is not set
> > > > > >
> > > > > > But more, WHY is this the right answer? You're disabling
> > > > > > functionality and the board maintainers aren't even on the CC list
> > > > > > for this thread it looks like.
> > > > >
> > > > > This was my blocking point, too, for this series. There is no answer
> > > > > about the board.
> > > > >
> > > > > I can just imagine (as for other MX6DL) that the required
> > > > > functionality is secure boot, then CONFIG_IMX_HAB is enough. But
> > > > > well, just the board maintainers can say. Added Raffaele and Jagan
> > > > > as reported maintainers for this board.
> > > > >
> > > > > > Why are you not fixing the board so it still links? What makes
> > > > > > this board a problem that other (I assume) imx6dl boards are not?
> > > > >
> > > > > Well, this board has, compared to other MX6DL, a lot of SPL_ option
> > > > > that are bloating the SPL size. And then does not match anymore with
> > > > > the internal RAm, while other MX6DL boards are not so affected by
> > > > > the increased size because they have no SPL at all or SPL with less
> > > > > options and then smaller. We do not need to fix the link, but reduce
> > > > > the size, and nobody else than the maintainers can tell us if
> > > > > removing some feature is ok.
> > > > >
> > > >
> > > > I will test this week, and try to find a board here to test
> > > >
> > > > * Jagan * Do you have a momoj?
> > >
> > > Have you tested mamoj board?
> > >
git diff
diff --git a/arch/arm/mach-imx/mx6/Kconfig b/arch/arm/mach-imx/mx6/Kconfig
index 98df4d4e42..b665ec887e 100644
--- a/arch/arm/mach-imx/mx6/Kconfig
+++ b/arch/arm/mach-imx/mx6/Kconfig
@@ -309,7 +309,6 @@ config TARGET_MX6DL_MAMOJ
select PINCTRL
select SPL
select SPL_DM if SPL
- select SPL_GPIO if SPL
select SPL_LIBCOMMON_SUPPORT if SPL
select SPL_LIBDISK_SUPPORT if SPL
select SPL_LIBGENERIC_SUPPORT if SPL
I strongly suggest to drop this and check again for now. I don't have
access to board and I have asked Jagan
Michael
> > > Regards
> > > Gaurav
> > > >
> > > > Michael
> > > >
> > > > > Regards,
> > > > > Stefano
> > > > >
> > > > > --
> > > > >
> > > >
> > > ================================================================
> > > > =====
> > > > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
> > > > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell,
> > > > > Germany
> > > > > Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email:
> > > > > sba...@denx.de
> > > > >
> > > >
> > > ================================================================
> > > > =====
> > > >
> > > >
> > > >
> > > > --
> > > > Michael Nazzareno Trimarchi
> > > > Co-Founder & Chief Executive Officer
> > > > M. +39 347 913 2170
> > > > mich...@amarulasolutions.com
> > > > __________________________________
> > > >
> > > > Amarula Solutions BV
> > > > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL T. +31 (0)85 111 9172
> > > > i...@amarulasolutions.com
> > > >
> > > https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.a
> > > > marulasolutions.com%2F&data=04%7C01%7Cgaurav.jain%40nxp.com%
> > > >
> > > 7C843e57f75346449f13ae08da010867b6%7C686ea1d3bc2b4c6fa92cd99c5c3
> > > >
> > > 01635%7C0%7C0%7C637823432854617357%7CUnknown%7CTWFpbGZsb3d8
> > > >
> > > eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3
> > > >
> > > D%7C3000&sdata=f025HmlIM8FuTEYspDjwOevTT1k7JRZeIiS%2FpTH2hc
> > > > Y%3D&reserved=0
>
>
>
> --
> Michael Nazzareno Trimarchi
> Co-Founder & Chief Executive Officer
> M. +39 347 913 2170
> mich...@amarulasolutions.com
> __________________________________
>
> Amarula Solutions BV
> Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> T. +31 (0)85 111 9172
> i...@amarulasolutions.com
> www.amarulasolutions.com
--
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
mich...@amarulasolutions.com
__________________________________
Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
i...@amarulasolutions.com
www.amarulasolutions.com
