public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC] should we remove soc subnode in all berlin SoC's dts if SOC_BUS is selected
Date: Fri, 26 Sep 2014 14:32:29 +0200	[thread overview]
Message-ID: <1821058.sf1gKKFj8T@wuerfel> (raw)
In-Reply-To: <20140926193945.3bb94276@xhacker>

On Friday 26 September 2014 19:39:45 Jisheng Zhang wrote:
> We want to export machine name, soc family, chip revision and chipid to
> userspace via. the SOC BUS interface, i.e call soc_device_register()
> 
> It works perfectly, but there's one side effect: every device nodes will have
> an additional level. For example:
> 
> /sys/devices/soc.0/xxx.sdhci will be changed to
> 
> /sys/devices/soc0/soc.0/xxx.sdhci
> 
> This seems a bit ugly. I have one solution: remove "soc" subnodes in all berlin's
> dts, then we will have 
> 
> /sys/devices/soc0/xxx.sdhci
> 
> Is there any better solution? 
> 

As of a recent discussion, we decided that the soc_device_register is now
purely informational and not meant to be used as the root for all the
other devices.

The structure in the DTS is fine and will continue to do the right thing.

Originally the intention was that soc_device_register would be used to
register the soc device  that corresponds to your /soc device node in DT,
and then you'd populate the other devices below, but we never implemented
a nice way to call that, so you don't need to bother with it.

Instead, you will now have /sys/devices/soc0/ for the soc_device you register
manually, and /sys/devices/soc.0/ to correspond with the DT node and continues
to have the child devices. This is still a bit confusing, but should be
consistent across all (new) platforms.

	Arnd

  reply	other threads:[~2014-09-26 12:32 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-26 11:39 [RFC] should we remove soc subnode in all berlin SoC's dts if SOC_BUS is selected Jisheng Zhang
2014-09-26 12:32 ` Arnd Bergmann [this message]
2014-09-27  3:19   ` Jisheng Zhang

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=1821058.sf1gKKFj8T@wuerfel \
    --to=arnd@arndb.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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