Hi Simon,

> -----Original Message-----
> From: Simon Glass <s...@chromium.org>
> Sent: Tuesday, 22 June, 2021 9:32 PM
> To: Chee, Tien Fong <tien.fong.c...@intel.com>
> Cc: Ley Foon Tan <lftan.li...@gmail.com>; Lim, Elly Siew Chin
> <elly.siew.chin....@intel.com>; ZY - u-boot <u-boot@lists.denx.de>; Marek
> Vasut <ma...@denx.de>; Tan, Ley Foon <ley.foon....@intel.com>; See, Chin
> Liang <chin.liang....@intel.com>; Simon Goldschmidt
> <simon.k.r.goldschm...@gmail.com>; Westergreen, Dalon
> <dalon.westergr...@intel.com>; Gan, Yau Wai <yau.wai....@intel.com>
> Subject: Re: [v2 04/17] arm: socfpga: Add handoff data support for Intel N5X
> device
> 
> Hi,
> 
> On Mon, 31 May 2021 at 05:39, Chee, Tien Fong <tien.fong.c...@intel.com>
> wrote:
> >
> > Hi Ley Foon,
> >
> > > -----Original Message-----
> > > From: Ley Foon Tan <lftan.li...@gmail.com>
> > > Sent: Friday, 14 May, 2021 5:13 PM
> > > To: Lim, Elly Siew Chin <elly.siew.chin....@intel.com>
> > > Cc: ZY - u-boot <u-boot@lists.denx.de>; Marek Vasut <ma...@denx.de>;
> > > Tan, Ley Foon <ley.foon....@intel.com>; See, Chin Liang
> > > <chin.liang....@intel.com>; Simon Goldschmidt
> > > <simon.k.r.goldschm...@gmail.com>; Chee, Tien Fong
> > > <tien.fong.c...@intel.com>; Westergreen, Dalon
> > > <dalon.westergr...@intel.com>; Simon Glass <s...@chromium.org>; Gan,
> > > Yau Wai <yau.wai....@intel.com>
> > > Subject: Re: [v2 04/17] arm: socfpga: Add handoff data support for
> > > Intel N5X device
> > >
> > > On Fri, Apr 30, 2021 at 3:39 PM Siew Chin Lim
> > > <elly.siew.chin....@intel.com>
> > > wrote:
> > > >
> > > > N5X support both HPS handoff data and DDR handoff data.
> > > > Existing HPS handoff functions are restructured to support both
> > > > existing devices and N5X device.
> > > >
> > > > Signed-off-by: Siew Chin Lim <elly.siew.chin....@intel.com>
> > > > Signed-off-by: Tien Fong Chee <tien.fong.c...@intel.com>
> > > >
> > > > ---
> > > > v2:
> > > > - Enabled auto detect the endianness from the magic word
> > > > - Merged and simplifying the big and little endian flow
> > > > ---
> > > >  .../mach-socfpga/include/mach/handoff_soc64.h |  38 +++++-
> > > > arch/arm/mach-socfpga/system_manager_soc64.c  |  18 +--
> > > >  arch/arm/mach-socfpga/wrap_handoff_soc64.c    | 126 +++++++++++++--
> > > ---
> > > >  3 files changed, 136 insertions(+), 46 deletions(-)
> 
> I'm not too clear on what is happening here, but have you looked at
> SPL_HANDOFF or BLOBLIST?

Yeah, but these are not for our use case here. Our handoff are generated from 
the tool, which having user settings. Our secure device manager will pass these 
handoff to SPL, this driver is  mainly for parsing our handoff format to 
consistent format in the table, which will be consumed and checking other 
drivers.

> 
> Regards,
> Simon

Regards,
Tien Fong

Reply via email to