Re: [GIT PULL] at91: cleanup for 3.16 #1

[Nicolas Ferre]

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/