From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv0] arm: mvebu: add .dts file for Synology DS213j
Date: Sun, 26 Oct 2014 17:54:18 +0100 [thread overview]
Message-ID: <17414879.SyK2RFgsXQ@wuerfel> (raw)
In-Reply-To: <20141025221611.GA14620@lunn.ch>
On Sunday 26 October 2014 00:16:11 Andrew Lunn wrote:
> Hi Arnaud
>
> Nice to see another device supported.
>
> > - When I bought the NAS, it had an old Marvell bootloader. After install
> > of latest DSM version, the u-boot was a new one w/ I had to change the
> > 'ranges' in the .dts to the ones below to have the kernel boot. I guess
> > new DS213j hardware will be shipped with new u-boot version at some
> > point and this will be fine; Otherwise, users will either have to
> > upgrade their DSM or manually s/1f/d0/ in .dts.
>
> Marvell have made a real mess here. Since there seems to be two
> different base addresses in use, i suggest you put a fat comment on
> the top of the .dts file about this issue. Give users a clue what to
> try if the kernel does not do anything at all.
>
Maybe it's better to have multiple dtb files in this case.
It should be trivial to have one .dts file that just overrides the
one address.
Arnd
WARNING: multiple messages have this Message-ID (diff)
From: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Cc: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>,
Arnaud Ebalard <arno-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org>,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
Ben Peddell <klightspeed-aslSrjg9ejhWX4hkXwHRhw@public.gmane.org>,
Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org>,
Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>,
Ian Campbell
<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
Rob Herring <rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>,
Simon Guinot <sguinot-D+JDLXUtGQkAvxtiuMwx3w@public.gmane.org>,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Gregory Clement
<gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
Russell King <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
Sebastian Hesselbarth
<sebastian.hesselbarth-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Subject: Re: [PATCHv0] arm: mvebu: add .dts file for Synology DS213j
Date: Sun, 26 Oct 2014 17:54:18 +0100 [thread overview]
Message-ID: <17414879.SyK2RFgsXQ@wuerfel> (raw)
In-Reply-To: <20141025221611.GA14620-g2DYL2Zd6BY@public.gmane.org>
On Sunday 26 October 2014 00:16:11 Andrew Lunn wrote:
> Hi Arnaud
>
> Nice to see another device supported.
>
> > - When I bought the NAS, it had an old Marvell bootloader. After install
> > of latest DSM version, the u-boot was a new one w/ I had to change the
> > 'ranges' in the .dts to the ones below to have the kernel boot. I guess
> > new DS213j hardware will be shipped with new u-boot version at some
> > point and this will be fine; Otherwise, users will either have to
> > upgrade their DSM or manually s/1f/d0/ in .dts.
>
> Marvell have made a real mess here. Since there seems to be two
> different base addresses in use, i suggest you put a fat comment on
> the top of the .dts file about this issue. Give users a clue what to
> try if the kernel does not do anything at all.
>
Maybe it's better to have multiple dtb files in this case.
It should be trivial to have one .dts file that just overrides the
one address.
Arnd
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2014-10-26 16:54 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-25 20:42 [PATCHv0] arm: mvebu: add .dts file for Synology DS213j Arnaud Ebalard
2014-10-25 20:42 ` Arnaud Ebalard
2014-10-25 22:16 ` Andrew Lunn
2014-10-25 22:16 ` Andrew Lunn
2014-10-26 8:35 ` Ben Peddell
2014-10-26 8:35 ` Ben Peddell
2014-10-26 17:42 ` Ben Peddell
2014-10-26 17:42 ` Ben Peddell
2014-10-26 16:54 ` Arnd Bergmann [this message]
2014-10-26 16:54 ` Arnd Bergmann
2014-10-25 22:18 ` Andrew Lunn
2014-10-25 22:18 ` Andrew Lunn
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=17414879.SyK2RFgsXQ@wuerfel \
--to=arnd@arndb.de \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.