* [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).