* [PATCH] ARM: dts: imx: share pad macro names between imx6q and imx6dl
[not found] <1373522316-12319-1-git-send-email-shawn.guo@linaro.org>
@ 2013-07-12 6:49 ` Sascha Hauer
2013-07-12 8:33 ` Shawn Guo
0 siblings, 1 reply; 2+ messages in thread
From: Sascha Hauer @ 2013-07-12 6:49 UTC (permalink / raw)
To: linux-arm-kernel
Hi Shawn,
Nice patch.
On Thu, Jul 11, 2013 at 01:58:36PM +0800, Shawn Guo wrote:
> The imx6q and imx6dl are two pin-to-pin compatible SoCs. The same board
> design can work with either chip plugged into the socket, e.g. sabresd
> and sabreauto boards.
>
> We currently define pin groups in imx6q.dtsi and imx6dl.dtsi
> respectively because the pad macro names are different between two
> chips. This brings a maintenance burden on having the same label point
> to the same pin group defined in two places.
>
> The patch drops prefix MX6Q_ and MX6DL_ from those pad macro names.
Can we use another prefix instead of removing it completely? Maybe
MX6QDL_?
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH] ARM: dts: imx: share pad macro names between imx6q and imx6dl
2013-07-12 6:49 ` [PATCH] ARM: dts: imx: share pad macro names between imx6q and imx6dl Sascha Hauer
@ 2013-07-12 8:33 ` Shawn Guo
0 siblings, 0 replies; 2+ messages in thread
From: Shawn Guo @ 2013-07-12 8:33 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jul 12, 2013 at 08:49:57AM +0200, Sascha Hauer wrote:
> Hi Shawn,
>
> Nice patch.
>
> On Thu, Jul 11, 2013 at 01:58:36PM +0800, Shawn Guo wrote:
> > The imx6q and imx6dl are two pin-to-pin compatible SoCs. The same board
> > design can work with either chip plugged into the socket, e.g. sabresd
> > and sabreauto boards.
> >
> > We currently define pin groups in imx6q.dtsi and imx6dl.dtsi
> > respectively because the pad macro names are different between two
> > chips. This brings a maintenance burden on having the same label point
> > to the same pin group defined in two places.
> >
> > The patch drops prefix MX6Q_ and MX6DL_ from those pad macro names.
>
> Can we use another prefix instead of removing it completely? Maybe
> MX6QDL_?
>
Sounds good. I changed it on my branch, and will not resend due to the
patch size.
Shawn
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-07-12 8:33 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1373522316-12319-1-git-send-email-shawn.guo@linaro.org>
2013-07-12 6:49 ` [PATCH] ARM: dts: imx: share pad macro names between imx6q and imx6dl Sascha Hauer
2013-07-12 8:33 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).