pinctrl: mvebu: numerous fixes, cleanups and improvements
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest0 of 34 patches 0 repliesNew
Patches in v1 No applies recorded yet
None of the 34 patches the cover letter announces are in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCH 00/34] pinctrl: mvebu: numerous fixes, cleanups and improvements
Andrew,
Quoted 4 lines · Andrew Lunn
Quoted 2 lines
I wish you good luck when reviewing this. It was boring to do, so I guess it is going to be horribly boring to review :-)
Not so far :-)
Great :-)
Quoted 5 lines
So i mailed two specific comments, but i think there might be a generic problem. You appear to be modifying pinctrl without making corresponding changes to .dtsi files? Or have i just missed them? I would of expected the .dtsi file changes to be in the same patch, or there are going to be bisect issues.
In fact, for most of the changes there should be zero impact on the .dtsi or .dts files. Most of the changes are about the subnames (which don't appear in DT) or adding new functions.
However, you indeed pointed two cases I missed. So I guess I'll have to review the entire series to double check what is the impact on the DT. But before doing that, it'd be great to have your point of on the patches that actually break the DT backward compatibility. If there are to be rejected anyway, I'd rather not spend more time on them.
Thanks a lot!
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com