[U-Boot] [PATCH v2 2/6] i.MX6: Change SYS_SOC from mx6 to imx6

Lothar Waßmann LW at KARO-electronics.de
Mon Mar 26 07:13:40 UTC 2018


Hi,

On Sun, 25 Mar 2018 15:30:08 -0300 Fabio Estevam wrote:
> On Sun, Mar 25, 2018 at 3:22 PM, Jagan Teki <jagan at amarulasolutions.com> wrote:
> > Now, MX6 has proper CONFIG names so rename the SYS_SOC to
> > imx6 from mx6 and also change related files.
> >
> > Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>
> > ---
> >  .travis.yml                                                |  4 ++--
> >  arch/arm/Kconfig                                           |  2 +-
> >  arch/arm/Makefile                                          |  4 ++--
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/clock.h       |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/crm_regs.h    |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/gpio.h        |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/imx-rdc.h     |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/imx-regs.h    |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/iomux.h       |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/litesom.h     |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6-ddr.h     |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6-pins.h    |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6_plugin.S  |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6dl-ddr.h   |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6dl_pins.h  |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6q-ddr.h    |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6q_pins.h   |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6sl-ddr.h   |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6sl_pins.h  |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6sll_pins.h |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6sx-ddr.h   |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6sx_pins.h  |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6sx_rdc.h   |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6ul-ddr.h   |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6ul_pins.h  |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mx6ull_pins.h |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/mxc_hdmi.h    |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/opos6ul.h     |  0
> >  arch/arm/include/asm/{arch-mx6 => arch-imx6}/sys_proto.h   |  0
> >  arch/arm/mach-imx/Makefile                                 | 10 +++++-----
> >  arch/arm/mach-imx/{mx6 => imx6}/Kconfig                    |  2 +-
> >  arch/arm/mach-imx/{mx6 => imx6}/Makefile                   |  0
> >  arch/arm/mach-imx/{mx6 => imx6}/clock.c                    |  0
> >  arch/arm/mach-imx/{mx6 => imx6}/ddr.c                      |  0
> >  arch/arm/mach-imx/{mx6 => imx6}/litesom.c                  |  0
> >  arch/arm/mach-imx/{mx6 => imx6}/mp.c                       |  0
> >  arch/arm/mach-imx/{mx6 => imx6}/opos6ul.c                  |  0
> >  arch/arm/mach-imx/{mx6 => imx6}/soc.c                      |  0
> >  board/advantech/dms-ba16/Kconfig                           |  2 +-
> >  board/ge/bx50v3/Kconfig                                    |  2 +-
> >  board/liebherr/display5/Kconfig                            |  2 +-
> >  board/logicpd/imx6/mx6q_2x_MT41K512M16HA.cfg               |  6 +++---
> >  board/technexion/pico-imx6ul/Kconfig                       |  2 +-
> >  board/toradex/apalis_imx6/Kconfig                          |  2 +-
> >  board/toradex/colibri_imx6/Kconfig                         |  2 +-
> >  drivers/watchdog/Makefile                                  |  2 +-
> >  46 files changed, 21 insertions(+), 21 deletions(-)
> 
> Sorry, but I still don't like this patch at all as I mentioned in the
> previous version.
>
+1

There is no danger confusing the mx* files with any other SoC family,
so renaming them to imx* is unnecessary churn.


Lothar Waßmann


More information about the U-Boot mailing list