[PATCH v1] sysreset: socfpga: agilex: Enable sysreset support

[Tan, Ley Foon]

> -----Original Message-----
> From: Ang, Chee Hong <chee.hong.ang at intel.com>
> Sent: Wednesday, August 5, 2020 6:08 PM
> To: Tan, Ley Foon <ley.foon.tan at intel.com>; u-boot at lists.denx.de
> Cc: Marek Vasut <marex at denx.de>; Simon Goldschmidt
> <simon.k.r.goldschmidt at gmail.com>; Tom Rini <trini at konsulko.com>; See,
> Chin Liang <chin.liang.see at intel.com>; Chee, Tien Fong
> <tien.fong.chee at intel.com>; Lim, Elly Siew Chin
> <elly.siew.chin.lim at intel.com>
> Subject: RE: [PATCH v1] sysreset: socfpga: agilex: Enable sysreset support
> 
> > > -----Original Message-----
> > > From: Ang, Chee Hong <chee.hong.ang at intel.com>
> > > Sent: Wednesday, August 5, 2020 5:54 PM
> > > To: u-boot at lists.denx.de
> > > Cc: Marek Vasut <marex at denx.de>; Simon Goldschmidt
> > > <simon.k.r.goldschmidt at gmail.com>; Tom Rini <trini at konsulko.com>;
> > > See, Chin Liang <chin.liang.see at intel.com>; Tan, Ley Foon
> > > <ley.foon.tan at intel.com>; Ang, Chee Hong <chee.hong.ang at intel.com>;
> > > Chee, Tien Fong <tien.fong.chee at intel.com>; Lim, Elly Siew Chin
> > > <elly.siew.chin.lim at intel.com>
> > > Subject: [PATCH v1] sysreset: socfpga: agilex: Enable sysreset
> > > support
> > >
> > > Enable sysreset support for Agilex platform.
> > >
> > > Signed-off-by: Chee Hong Ang <chee.hong.ang at intel.com>
> > > ---
> > >  arch/arm/Kconfig         | 2 +-
> > >  drivers/sysreset/Kconfig | 6 +++---
> > >  2 files changed, 4 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index
> > > 6b8a32c38d..105b5f08a9 100644
> > > --- a/arch/arm/Kconfig
> > > +++ b/arch/arm/Kconfig
> > > @@ -991,7 +991,7 @@ config ARCH_SOCFPGA
> > >  	select SYS_THUMB_BUILD if TARGET_SOCFPGA_GEN5 ||
> > > TARGET_SOCFPGA_ARRIA10
> > >  	select SYSRESET
> > >  	select SYSRESET_SOCFPGA if TARGET_SOCFPGA_GEN5 ||
> > > TARGET_SOCFPGA_ARRIA10
> > > -	select SYSRESET_SOCFPGA_S10 if TARGET_SOCFPGA_STRATIX10
> > > +	select SYSRESET_SOCFPGA_S10 if TARGET_SOCFPGA_STRATIX10 ||
> > > +TARGET_SOCFPGA_AGILEX
> > >  	imply CMD_DM
> > >  	imply CMD_MTDPARTS
> > >  	imply CRC32_VERIFY
> > > diff --git a/drivers/sysreset/Kconfig b/drivers/sysreset/Kconfig
> > > index
> > > 6ebc90e1d3..d886d1c933 100644
> > > --- a/drivers/sysreset/Kconfig
> > > +++ b/drivers/sysreset/Kconfig
> > > @@ -80,11 +80,11 @@ config SYSRESET_SOCFPGA
> > >  	  (Cyclone 5, Arria 5 and Arria 10).
> > >
> > >  config SYSRESET_SOCFPGA_S10
> >
> > This can change to _SOC64.
> Do I need to change the source file from
> drivers/sysreset/sysreset_socfpga_s10.c to
> drivers/sysreset/sysreset_socfpga_soc64.c as well ?
> drivers/sysreset/makefile has this:
> obj-$(CONFIG_SYSRESET_SOCFPGA_S10) += sysreset_socfpga_s10.o
> 

Yes, I think it is better. Rename first , then add Agilex support.

Regards
Ley Foon