From: nicolas.ferre@atmel.com (Nicolas Ferre)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] at91: cleanup for 3.16 #1
Date: Mon, 19 May 2014 17:10:32 +0200 [thread overview]
Message-ID: <537A1EE8.6030303@atmel.com> (raw)
In-Reply-To: <20140516233141.GC6423@quad.lixom.net>
On 17/05/2014 01:31, Olof Johansson :
> On Fri, May 16, 2014 at 04:26:35PM -0700, Olof Johansson wrote:
>> On Wed, May 07, 2014 at 07:39:35PM +0200, Nicolas Ferre wrote:
>>> On 07/05/2014 19:34, Nicolas Ferre :
>>>> Arnd, Olof, Kevin,
>>>>
>>>> This is the first cleanup pull-request for 3.16. It is pretty big because it
>>>> integrates the work from Boris about CCF and Alexandre about IIO/ADC. I
>>>> integrated them in this cleanup topic because they both touch the core at91
>>>> code, the clk and IIO drivers as well as the DT. The concerned maintainers
>>>> added their tags.
>>>>
>>>> The patch by Linus is a move of at91 specific GPIO definitions out of the
>>>> include/mach directory which is an step towards single zImage.
>>>>
>>>> Thanks, best regards,
>>>>
>>>> The following changes since commit 89ca3b881987f5a4be4c5dbaa7f0df12bbdde2fd:
>>>>
>>>> Linux 3.15-rc4 (2014-05-04 18:14:42 -0700)
>>>>
>>>> are available in the git repository at:
>>>>
>>>> git://github.com/at91linux/linux-at91.git tags/at91-cleanup
>>>
>>> There is a little conflict with at91-3.16-dt that you already pulled in
>>> arm-soc: here is the branch that resolves it:
>>>
>>> https://github.com/at91linux/linux-at91/commits/at91-3.16-resolved
>>
>> That resolution looks odd. Why is one clock under clocks { } and two of them
>> are at the top level? Shouldn't they all be under the clocks subnode?
>>
>> I've merged in now with your resolution, but I think this needs revisiting.
With information from Alexandre, I have the feeling that we should
remove the "clocks" container altogether. I propose to clean completely
this part in 3.17 as it will not change the system but only its
representation in DT...
Still, we will fix the ugliest parts like this one:
http://code.bulix.org/by22lb-86239
> Oh, and also: The branch was named cleanup, but it really contains mostly new
> driver and new contents, so I merged it under next/soc instead. In the future,
> for releases when you have more contents, please split it up a bit more (dt
> updates separately, driver updates separately) if it can be done without too
> much hassle. For now it's not a huge deal to do it this way though.
Yes. The issue with these series that switch from one description to
another is that if we split the driver and dt parts it would add extra
steps which are not very easy to keep in sync, implement and are kind of
useless.
I have more series to come for 3.16 that I named "cleanup" as well. I
will double check to see if I can split them up more...
Thanks, bye,
--
Nicolas Ferre
next prev parent reply other threads:[~2014-05-19 15:10 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-07 17:34 [GIT PULL] at91: cleanup for 3.16 #1 Nicolas Ferre
2014-05-07 17:39 ` Nicolas Ferre
2014-05-16 23:26 ` Olof Johansson
2014-05-16 23:31 ` Olof Johansson
2014-05-19 15:10 ` Nicolas Ferre [this message]
2014-05-20 4:28 ` Olof Johansson
2014-05-17 4:22 ` Alexandre Belloni
2014-05-20 4:25 ` Olof Johansson
2014-05-20 4:32 ` Olof Johansson
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=537A1EE8.6030303@atmel.com \
--to=nicolas.ferre@atmel.com \
--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).