All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shawn Guo <shawnguo@kernel.org>
To: Michael Walle <michael@walle.cc>
Cc: "Y.b. Lu" <yangbo.lu@nxp.com>,
	Vladimir Oltean <vladimir.oltean@nxp.com>,
	Leo Li <leoyang.li@nxp.com>, Rob Herring <robh+dt@kernel.org>,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
	Adrian Hunter <adrian.hunter@intel.com>,
	Ulf Hansson <ulf.hansson@linaro.org>,
	linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org,
	Ashish Kumar <ashish.kumar@nxp.com>
Subject: Re: [PATCH] arm64: dts: ls1028a: make the eMMC and SD card controllers use fixed indices
Date: Mon, 30 Nov 2020 22:29:19 +0800	[thread overview]
Message-ID: <20201130142918.GJ4072@dragon> (raw)
In-Reply-To: <31db48954bdf02fc0af73871043fc76b@walle.cc>

On Wed, Nov 25, 2020 at 09:25:23AM +0100, Michael Walle wrote:
> We are going cirlces here. I guess Shawn (as the soc maintainer) has to
> step in and decide if a common soc include should contain aliases for
> nodes which are disabled. That is what it boils down to.
> 
> All other arguments against having aliases in the common include can be
> found in this thread.
> 
> > Distros, bootloaders, and users' cases using fixed index before could
> > avoid issues, and been used as they were.
> 
> Nobody argue against having these alias. We are arguing against having
> them in the common soc include.

342ab37ecaf8 ("arm64: dts: freescale: use fixed index mmcN for
layerscape") is dropped from my tree.

Shawn

WARNING: multiple messages have this Message-ID (diff)
From: Shawn Guo <shawnguo@kernel.org>
To: Michael Walle <michael@walle.cc>
Cc: devicetree@vger.kernel.org, Ulf Hansson <ulf.hansson@linaro.org>,
	linux-kernel@vger.kernel.org, Ashish Kumar <ashish.kumar@nxp.com>,
	Vladimir Oltean <vladimir.oltean@nxp.com>,
	linux-mmc@vger.kernel.org,
	Adrian Hunter <adrian.hunter@intel.com>,
	Leo Li <leoyang.li@nxp.com>, Rob Herring <robh+dt@kernel.org>,
	"Y.b. Lu" <yangbo.lu@nxp.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] arm64: dts: ls1028a: make the eMMC and SD card controllers use fixed indices
Date: Mon, 30 Nov 2020 22:29:19 +0800	[thread overview]
Message-ID: <20201130142918.GJ4072@dragon> (raw)
In-Reply-To: <31db48954bdf02fc0af73871043fc76b@walle.cc>

On Wed, Nov 25, 2020 at 09:25:23AM +0100, Michael Walle wrote:
> We are going cirlces here. I guess Shawn (as the soc maintainer) has to
> step in and decide if a common soc include should contain aliases for
> nodes which are disabled. That is what it boils down to.
> 
> All other arguments against having aliases in the common include can be
> found in this thread.
> 
> > Distros, bootloaders, and users' cases using fixed index before could
> > avoid issues, and been used as they were.
> 
> Nobody argue against having these alias. We are arguing against having
> them in the common soc include.

342ab37ecaf8 ("arm64: dts: freescale: use fixed index mmcN for
layerscape") is dropped from my tree.

Shawn

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2020-11-30 14:30 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-19 15:50 [PATCH] arm64: dts: ls1028a: make the eMMC and SD card controllers use fixed indices Vladimir Oltean
2020-11-19 15:50 ` Vladimir Oltean
2020-11-19 16:08 ` Michael Walle
2020-11-19 16:08   ` Michael Walle
2020-11-19 16:32   ` Vladimir Oltean
2020-11-19 16:32     ` Vladimir Oltean
2020-11-20  2:04 ` Y.b. Lu
2020-11-20  2:04   ` Y.b. Lu
2020-11-20  9:30   ` Vladimir Oltean
2020-11-20  9:30     ` Vladimir Oltean
2020-11-23 11:09     ` Michael Walle
2020-11-23 11:09       ` Michael Walle
2020-11-24  7:41     ` Y.b. Lu
2020-11-24  7:41       ` Y.b. Lu
2020-11-24  8:03       ` Michael Walle
2020-11-24  8:03         ` Michael Walle
2020-11-24  8:47         ` Y.b. Lu
2020-11-24  8:47           ` Y.b. Lu
2020-11-24  8:55           ` Michael Walle
2020-11-24  8:55             ` Michael Walle
2020-11-24  9:02             ` Y.b. Lu
2020-11-24  9:02               ` Y.b. Lu
2020-11-24  9:08               ` Michael Walle
2020-11-24  9:08                 ` Michael Walle
2020-11-24  9:22                 ` Y.b. Lu
2020-11-24  9:22                   ` Y.b. Lu
2020-11-24  9:43                   ` Michael Walle
2020-11-24  9:43                     ` Michael Walle
2020-11-24 10:22                     ` Y.b. Lu
2020-11-24 10:22                       ` Y.b. Lu
2020-11-24 10:31               ` Vladimir Oltean
2020-11-24 10:31                 ` Vladimir Oltean
2020-11-24 11:15                 ` Y.b. Lu
2020-11-24 11:15                   ` Y.b. Lu
2020-11-24 11:28                   ` Vladimir Oltean
2020-11-24 11:28                     ` Vladimir Oltean
2020-11-25  2:59                     ` Y.b. Lu
2020-11-25  2:59                       ` Y.b. Lu
2020-11-25  8:25                       ` Michael Walle
2020-11-25  8:25                         ` Michael Walle
2020-11-30 14:29                         ` Shawn Guo [this message]
2020-11-30 14:29                           ` Shawn Guo
2020-12-01  2:38                           ` Y.b. Lu
2020-12-01  2:38                             ` Y.b. Lu

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=20201130142918.GJ4072@dragon \
    --to=shawnguo@kernel.org \
    --cc=adrian.hunter@intel.com \
    --cc=ashish.kumar@nxp.com \
    --cc=devicetree@vger.kernel.org \
    --cc=leoyang.li@nxp.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=michael@walle.cc \
    --cc=robh+dt@kernel.org \
    --cc=ulf.hansson@linaro.org \
    --cc=vladimir.oltean@nxp.com \
    --cc=yangbo.lu@nxp.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.