Re: [PATCH 2/7] ARM: i.MX53: Add sata to devicetree

From: Sascha Hauer <s.hauer@pengutronix.de>
To: Alexander Shiyan <shc_work@mail.ru>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 2/7] ARM: i.MX53: Add sata to devicetree
Date: Mon, 10 Feb 2014 09:33:58 +0100	[thread overview]
Message-ID: <20140210083358.GH17250@pengutronix.de> (raw)
In-Reply-To: <1391869601.90335932@f192.i.mail.ru>

On Sat, Feb 08, 2014 at 06:26:41PM +0400, Alexander Shiyan wrote:
> Суббота,  8 февраля 2014, 15:16 +01:00 от Eric Bénard <eric@eukrea.com>:
> > Signed-off-by: Eric Bénard <eric@eukrea.com>
> > ---
> > arch/arm/dts/imx53.dtsi | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> > 
> > diff --git a/arch/arm/dts/imx53.dtsi b/arch/arm/dts/imx53.dtsi
> > index 678547c..91659b8 100644
> > --- a/arch/arm/dts/imx53.dtsi
> > +++ b/arch/arm/dts/imx53.dtsi
> > @@ -75,6 +75,15 @@
> > interrupt-parent = <&tzic>;
> > ranges;
> > 
> > +		sata: sata@10000000 {
> > +			compatible = "fsl,imx53-ahci";
> > +			reg = <0x10000000 0x1000>;
> > +			interrupts = <28>;
> > +			clocks = <&clks 173>, <&clks 5>, <&clks 188>;
> > +			clock-names = "sata", "ahb", "per";
> > +			status = "disabled";
> > +		};
> 
> Sascha, we stop changing DTS of the kernel or not?
> What is the ultimate solution?

What do you suggest for the case above? A imx53-bb.dtsi included from
imx53.dtsi? I think disallowing ahci support for barebox if it's not
upstream in Linux is not a good option.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox