diff mbox series

[v2] arm: dts: agilex: Enable QSPI

Message ID 20200331004525.77662-1-ley.foon.tan@intel.com
State Accepted
Commit df8e15af2bed62a5a93c5783ec9e32b9029bb010
Headers show
Series [v2] arm: dts: agilex: Enable QSPI | expand

Commit Message

Tan, Ley Foon March 31, 2020, 12:45 a.m. UTC
Enable QSPI for Agilex SoC devkit.

Signed-off-by: Ley Foon Tan <ley.foon.tan at intel.com>

---
v2:
- Fixed missing ";".
---
 arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Marek Vasut March 31, 2020, 12:52 a.m. UTC | #1
On 3/31/20 2:45 AM, Ley Foon Tan wrote:
> Enable QSPI for Agilex SoC devkit.
> 
> Signed-off-by: Ley Foon Tan <ley.foon.tan at intel.com>
> 
> ---
> v2:
> - Fixed missing ";".
> ---
>  arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi b/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi
> index 1908be4b8b27..c0be18c2e4ed 100644
> --- a/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi
> +++ b/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi
> @@ -37,3 +37,7 @@
>  	u-boot,dm-pre-reloc;
>  };
>  
> +&qspi {
> +	status = "okay";
> +};
> +

There's still an extra newline at the end, I'll fix it when applying,
but checkpatch should warn about it.
Tan, Ley Foon March 31, 2020, 1:04 a.m. UTC | #2
> -----Original Message-----
> From: Marek Vasut <marex at denx.de>
> Sent: Tuesday, March 31, 2020 8:52 AM
> To: Tan, Ley Foon <ley.foon.tan at intel.com>; u-boot at lists.denx.de
> Cc: Ley Foon Tan <lftan.linux at gmail.com>; See, Chin Liang
> <chin.liang.see at intel.com>; Simon Goldschmidt
> <simon.k.r.goldschmidt at gmail.com>
> Subject: Re: [PATCH v2] arm: dts: agilex: Enable QSPI
> 
> On 3/31/20 2:45 AM, Ley Foon Tan wrote:
> > Enable QSPI for Agilex SoC devkit.
> >
> > Signed-off-by: Ley Foon Tan <ley.foon.tan at intel.com>
> >
> > ---
> > v2:
> > - Fixed missing ";".
> > ---
> >  arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi
> > b/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi
> > index 1908be4b8b27..c0be18c2e4ed 100644
> > --- a/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi
> > +++ b/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi
> > @@ -37,3 +37,7 @@
> >  	u-boot,dm-pre-reloc;
> >  };
> >
> > +&qspi {
> > +	status = "okay";
> > +};
> > +
> 
> There's still an extra newline at the end, I'll fix it when applying, but
> checkpatch should warn about it.
Okay, thanks.

BTW, checkpatch no warning for this.

Regards
Ley Foon
diff mbox series

Patch

diff --git a/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi b/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi
index 1908be4b8b27..c0be18c2e4ed 100644
--- a/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi
+++ b/arch/arm/dts/socfpga_agilex_socdk-u-boot.dtsi
@@ -37,3 +37,7 @@ 
 	u-boot,dm-pre-reloc;
 };
 
+&qspi {
+	status = "okay";
+};
+