Re: [PATCH] ARM: exynos: remove unused code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 14, 2013 at 6:17 AM, Arnd Bergmann <arnd@xxxxxxxx> 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@xxxxxxxx>

Acked-by: Olof Johansson <olof@xxxxxxxxx>

> 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.


-Olof
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux