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
next prev parent reply other threads:[~2014-10-26 16:54 UTC|newest]
Thread overview: 6+ 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 22:16 ` Andrew Lunn
2014-10-26 8:35 ` Ben Peddell
2014-10-26 17:42 ` Ben Peddell
2014-10-26 16:54 ` Arnd Bergmann [this message]
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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).