From: "Lothar Waßmann" <LW@KARO-electronics.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 2/6] i.MX6: Change SYS_SOC from mx6 to imx6
Date: Mon, 26 Mar 2018 09:13:40 +0200 [thread overview]
Message-ID: <20180326091340.5188ca0f@karo-electronics.de> (raw)
In-Reply-To: <CAOMZO5C4pc+ObmygzgKf4QNhBw6Mh+Cbhkocf6LMk5BHO78HSw@mail.gmail.com>
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@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@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
next prev parent reply other threads:[~2018-03-26 7:13 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-25 18:22 [U-Boot] [PATCH v2 0/6] ARM: i.MX6: Add proper CONFIG names Jagan Teki
2018-03-25 18:22 ` [U-Boot] [PATCH v2 1/6] i.MX6: Add proper CONFIG_SOC_ names for MX6 architectures Jagan Teki
2018-03-25 18:22 ` [U-Boot] [PATCH v2 2/6] i.MX6: Change SYS_SOC from mx6 to imx6 Jagan Teki
2018-03-25 18:30 ` Fabio Estevam
2018-03-26 7:13 ` Lothar Waßmann [this message]
2018-03-25 18:22 ` [U-Boot] [PATCH v2 3/6] ARM: dts: i.MX6QDL: U-Boot specific dts for u-boot, dm-spl Jagan Teki
2018-03-25 18:22 ` [U-Boot] [PATCH v2 4/6] ARM: dts: imx6ul-isiot: Move usdhc2 into dtsi Jagan Teki
2018-03-25 18:22 ` [U-Boot] [PATCH v2 5/6] ARM: dts: i.MX6UL: U-Boot specific dts for u-boot, dm-spl Jagan Teki
2018-03-25 18:22 ` [U-Boot] [PATCH v2 6/6] ARM: i.MX6: dts: Build dtb based on SOC type Jagan Teki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180326091340.5188ca0f@karo-electronics.de \
--to=lw@karo-electronics.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox