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

[Sascha Hauer <s.hauer@pengutronix.de>]

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?

I don't want to give out the rule "Only use unmodified dts in barebox"
and then use dts files from Linux-next. If we only want to use
unmodified dts files we have to use the ones from Linux mainline.
Otherwise we are doomed when a pull request doesn't make it in the
Kernel and then the dts files are changed significantly before the
next merge window, as just happens with Shawns dts changes for 3.14.

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