[PATCH] ARM: exynos: remove unused code
Kukjin Kim
kgene.kim at samsung.com
Fri Jun 14 22:24:38 EDT 2013
Olof Johansson wrote:
>
> On Fri, Jun 14, 2013 at 6:17 AM, Arnd Bergmann <arnd at arndb.de> wrote:
> > After exynos has been converted to DT-only booting by removing
> > most a lot of the ATAGS code, I had a closer look of what is
> > left. It turns out that a number of unused files are left in
> > the tree that can be removed along with about another quarter
> > of the remaining code in mach-exynos.
> >
> > Specifically this includes:
> >
> > * all board files
> > * all setup-*.c files
> > * dev-audio.c and dev-uart.c
> > * all code testing for of_have_populated_dt
> > * the eint_irqchip
> > * almost all the mach/irqs.h constants (the remaining ones
> > are likely bugs)
> > * Most physical address constants. The remaining ones are
> > either bugs or used for the iotable. Both can probably
> > get removed in the long run.
> > * regs-usb-phy.h, which was already unused before
> > * A lot of code in plat-samsung/ can be made conditional
> > on CONFIG_ATAGS, in particular now unused device
> > definitions.
> >
> > Signed-off-by: Arnd Bergmann <arnd at arndb.de>
>
> Acked-by: Olof Johansson <olof at lixom.net>
>
> > Hi Kukjin,
> >
> > Could you check the validity of this patch please?
> >
> > I think we should apply it on top of your series. I've generated
> > the patch using 'git format-patch -D' for brevity, so be careful
> > to actually delete the unused files if you want to apply it to
> > a local branch on your side.
>
> Great. This is the piece that I wanted to see in the non-DT-removal
> branch before I pulled it.
>
Yeah, I think so. But as Arnd suggested, I'm looking at Tomasz's patches and
it will be sorted out in this weekend.
Thanks,
- Kukjin
More information about the linux-arm-kernel
mailing list