ARM: dts: imx6qp-sabreauto: enable sata on imx6qp sabreauto board

Commit Message

Hongxing Zhu July 3, 2020, 8:18 a.m. UTC

Enable SATA on iMX6QP SABREAUTO board.

Signed-off-by: Richard Zhu <[e-mail]>
---
 arch/arm/boot/dts/imx6qp-sabreauto.dts | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Fabio Estevam July 8, 2020, 5:41 p.m. UTC | #1

Hi Richard,

In the Subject the board name appears twice, which is redundant.

It could be made simpler like this:

ARM: dts: imx6qp-sabreauto: enable sata


On Fri, Jul 3, 2020 at 5:23 AM Richard Zhu <[e-mail]> wrote:
>
> Enable SATA on iMX6QP SABREAUTO board.
>
> Signed-off-by: Richard Zhu <[e-mail]>

Reviewed-by: Fabio Estevam <[e-mail]>

Hongxing Zhu July 10, 2020, 6:55 a.m. UTC | #2

> -----Original Message-----
> From: Fabio Estevam <[e-mail]>
> Sent: 2020年7月9日 1:41
> To: Richard Zhu <[e-mail]>
> Cc: Shawn Guo <[e-mail]>; dl-linux-imx <[e-mail]>;
> moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE
> <[e-mail]>; linux-kernel
> <[e-mail]>
> Subject: [EXT] Re: [PATCH] ARM: dts: imx6qp-sabreauto: enable sata on
> imx6qp sabreauto board
> 
> Caution: EXT Email
> 
> Hi Richard,
> 
> In the Subject the board name appears twice, which is redundant.
> 
> It could be made simpler like this:
> 
> ARM: dts: imx6qp-sabreauto: enable sata
> 
Hi Fabio:
Thanks for your review. Okay, simple subject would be used later.

Best Regards
Richard Zhu

> On Fri, Jul 3, 2020 at 5:23 AM Richard Zhu <[e-mail]> wrote:
> >
> > Enable SATA on iMX6QP SABREAUTO board.
> >
> > Signed-off-by: Richard Zhu <[e-mail]>
> 
> Reviewed-by: Fabio Estevam <[e-mail]>

Patch

diff --git a/arch/arm/boot/dts/imx6qp-sabreauto.dts b/arch/arm/boot/dts/imx6qp-sabreauto.dts
index d4caeeb0af70..639d9dd35377 100644
--- a/arch/arm/boot/dts/imx6qp-sabreauto.dts
+++ b/arch/arm/boot/dts/imx6qp-sabreauto.dts
@@ -50,6 +50,10 @@ 
 	status = "disabled";
 };
 
+&sata {
+	status = "okay";
+};
+
 &vgen3_reg {
 	regulator-always-on;
 };