ARM: shmobile: Rework include path V2
Get series with b4
b4 am [email protected]Versions newest first
- v2 Latest0 of 8 patches 0 repliesNew
Patches in v2 No applies recorded yet
None of the 8 patches the cover letter announces are in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.
Cover letter [PATCH v2 00/08] ARM: shmobile: Rework include path V2
Quoted 9 lines · Magnus Damm
Please note that these are only lightly compile tested with a handful of defconfigs, but any potential issue should be trivial to fix.
Also, there are certain bits of
arch/arm/mach-shmobile/include/mach/get included from places likearch/arm/boot/compressed/(booting legacy kernel directly from reset vector) and also from SoC specific legacy code underdrivers/pinctrl. My proposal is to keep those users as-is until we phase out legacy board support (2-3 kernels?), let me know if you would like to get rid of "include/mach" sooner!
If it's only about arch/arm/boot/compressed, you can just move the header files to the same directory, as nothing else includes them.
I've played with a full multiplatform conversion of a few platforms including shmobile in the past, and all I needed for the legacy pinctrl drivers was to duplicate the irq_pin() definition, which is probably a reasonable hack for code that we know is going away anyway, just add a comment for both definitions explaining that they have to be kept in sync. This one is less obvious than the boot header.
One general comment: please use 'git format-patch -M' or set the 'diff.renames' git option for sending patches for reviews that rename files. No need to send this patch set again.
Arnd