public inbox for linuxppc-dev@ozlabs.org 
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb•de>
To: Scott Wood <scottwood@freescale•com>
Cc: linuxppc-dev@ozlabs•org
Subject: Re: [PATCH] Add of_platform_device_scan().
Date: Wed, 4 Oct 2006 18:37:46 +0200	[thread overview]
Message-ID: <200610041837.47317.arnd@arndb.de> (raw)
In-Reply-To: <4523E214.5060404@freescale.com>

On Wednesday 04 October 2006 18:32, Scott Wood wrote:
> What I'd really like (long-term, of course) is if platform_device and=20
> of_device were merged, with device tree support (or at least a means of=20
> passing on properties that *could* come from a device tree without=20
> special glue code that knows about each property) in arch-neutral code;=20
> the mechanism for discovering devices ideally shouldn't depend on the=20
> CPU's instruction set.

My guess is that this won't happen, because other architectures
normally don't describe their platform devices in a way that is
anywhere near what we have on powerpc.

Benh suggested moving some of the functionality of of_platform_device
into the common of_device so we don't really have to use an
of_platform_device for stuff that can be automatically probed.

I guess we'll see a patch from him soon.

> > +=A0=A0=A0=A0=A0for (child =3D NULL; (child =3D of_get_next_child(root,=
 child)); ) {
> > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0if (strcmp(child->type, "spider=
") =3D=3D 0) {
> > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0ret =3D=
 of_soc_device_create(child, NULL);
> > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0if (ret)
> > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=
=A0=A0=A0=A0=A0=A0goto out3;
> > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0}
> > +=A0=A0=A0=A0=A0}
>=20
> Why only spider?

Mistake on my side. That's what I have in my experimental device tree.
It should be more generic.

	Arnd <><

  reply	other threads:[~2006-10-04 16:37 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-03 22:56 [PATCH] Add of_platform_device_scan() Scott Wood
2006-10-03 23:18 ` Arnd Bergmann
2006-10-04 16:32   ` Scott Wood
2006-10-04 16:37     ` Arnd Bergmann [this message]
2006-10-04 19:33       ` Scott Wood

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=200610041837.47317.arnd@arndb.de \
    --to=arnd@arndb$(echo .)de \
    --cc=linuxppc-dev@ozlabs$(echo .)org \
    --cc=scottwood@freescale$(echo .)com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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