From: olof@lixom.net (Olof Johansson)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] ARM: mvebu: dt for v4.10 (#1)
Date: Fri, 18 Nov 2016 17:58:38 -0800 [thread overview]
Message-ID: <20161119015838.GG2543@localhost> (raw)
In-Reply-To: <87poltbwt2.fsf@free-electrons.com>
Hi,
On Thu, Nov 17, 2016 at 10:50:33PM +0100, Gregory CLEMENT wrote:
> Hi,
>
> Here is the first pull request for dt for mvebu for v4.10.
>
> I hope being able to send a second part very soon with the series
> removing all the DT warning.
>
> Gregory
>
> The following changes since commit 1001354ca34179f3db924eb66672442a173147dc:
>
> Linux 4.9-rc1 (2016-10-15 12:17:50 -0700)
>
> are available in the git repository at:
>
> git://git.infradead.org/linux-mvebu.git tags/mvebu-dt-4.10-1
>
> for you to fetch changes up to cf20c489de6fcef88405d4febef7a078d2053b9e:
>
> ARM: dt: orion5x: convert ls-chl to FDT (2016-11-07 17:23:34 +0100)
>
> ----------------------------------------------------------------
> mvebu fixes for 4.10 (part 1)
>
> Most of the commit are pinmux and i2c fix for netgear NASes
> Fix on a wrong comment about PLL frequency
> Bigger commit: conversion of on otion5x based board to the device tree
>
> ----------------------------------------------------------------
> Ashley Hughes (1):
> ARM: dt: orion5x: convert ls-chl to FDT
This is a great conversion, but I'd like to see the code handled a
little differently.
First of all, there's no longer a need to have a config option for
MACH_LINKSTATION_LSCHL, as long as ARCH_ORION5X_DT is enabled you'll
be fine. So you can remove that Kconfig entry alltogether.
Also, there's no need to make the DT addition and the legacy platform
removal in one commit. It's common that we build up the DT support to the point
that it's at parity, and then remove the legacy board. That way we also don't
entangle DT commits with non-DT commits, which can sometimes be a bit of a pain
(in particular for those who maintain a copy of the DT subdir in another git
repo).
So, mind respinning with this fixed? Thanks!
-Olof
next prev parent reply other threads:[~2016-11-19 1:58 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-17 21:50 [GIT PULL] ARM: mvebu: dt for v4.10 (#1) Gregory CLEMENT
2016-11-19 1:58 ` Olof Johansson [this message]
2016-11-19 8:25 ` Gregory CLEMENT
-- strict thread matches above, loose matches on Subject: below --
2016-11-19 8:33 Gregory CLEMENT
2016-11-25 23:47 ` Arnd Bergmann
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=20161119015838.GG2543@localhost \
--to=olof@lixom.net \
--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).