linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: "Jon Smirl" <jonsmirl@gmail.com>
To: "Timur Tabi" <timur@freescale.com>
Cc: linuxppc-dev@ozlabs.org, alsa-devel@alsa-project.org,
	liam.girdwood@wolfsonmicro.com
Subject: Re: [alsa-devel] [PATCH v2 1/3] ALSA SoC: Add OpenFirmware helper for matching bus and codec drivers
Date: Mon, 14 Jul 2008 19:45:46 -0400	[thread overview]
Message-ID: <9e4733910807141645l404881b0m1d17d88c26792c78@mail.gmail.com> (raw)
In-Reply-To: <487B9D98.8010008@freescale.com>

On 7/14/08, Timur Tabi <timur@freescale.com> wrote:
> Mark Brown wrote:
>
>  > Desktop Management Interface, a standard BIOS interface for getting
>  > system data on x86 class hardware.  Of particular interest here is the
>  > fact that it contains various ID strings for things like motherboard and
>  > chassis - on Linux drivers can be automatically loaded based on these
>  > strings.  See drivers/misc/thinkpad_acpi.c for an example of a driver
>  > that does this.
>
>
> The only problem with this is that the OF probing code in the kernel binds
>  drivers to device tree nodes.  So when a driver claims a node, no other driver
>  will be probed with it.
>
>  So we can't have generic nodes that classify the motherboard and just let
>  everyone get probed on it.

Allowing multiple binds at the root causes the problem of something
like compatible="lite5200b,mpc5200-simple". Both platforms would bind
and that's not what you want.

I'm not a fan of each platform creating a platform device in
arch/powerpc/platforms/*. That implies that each platform would cause
another source file to be added each containing pretty much identical
code. It also makes the mpc5200-simple platform pointless. Of course
this scheme works and I'm doing it right now, but it's clearly not an
optimal solution.

Another scheme would be to add kernel code to always create virtual OF
devices like "lite5200b-fabric" that are derived off from the machine
name that achieved a bind.

A third scheme would be to convert the powerpc machine drivers
themselves to device drivers and move them out of
arch/powerpc/platforms/* into drivers/whatever. Then add the ASoC
fabric code to them. I don't know if you can load device drivers early
enough to load a powerpc machine driver from initrd.

A fourth scheme is to do it at compile time. But that means no
universal firmware images that support multiple hardware revisions. We
have this one today too.

-- 
Jon Smirl
jonsmirl@gmail.com

  parent reply	other threads:[~2008-07-14 23:45 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-12  8:39 [PATCH v2 1/3] ALSA SoC: Add OpenFirmware helper for matching bus and codec drivers Grant Likely
2008-07-12  8:39 ` [PATCH v2 2/3] ALSA SoC: Add mpc5200-psc I2S driver Grant Likely
2008-07-14 12:10   ` [alsa-devel] " Mark Brown
2008-07-12  8:39 ` [PATCH v2 3/3] ALSA SoC: Add Texas Instruments TLV320AIC26 codec driver Grant Likely
2008-07-12 18:10   ` [alsa-devel] " Mark Brown
2008-07-12 18:14     ` Grant Likely
2008-07-14 11:45   ` Mark Brown
2008-07-18  6:29     ` Grant Likely
2008-07-18 10:39       ` Mark Brown
2008-07-15  7:57   ` dinesh
2008-07-15 10:33     ` Mark Brown
2008-07-15 12:38       ` dinesh
2008-07-15 12:46         ` Mark Brown
2008-07-16  9:05   ` WRITING AN SOC DRIVER WITHOUT DMA dinesh
2008-07-16 10:07     ` [alsa-devel] " Nobin Mathew
2008-07-16 10:13       ` dinesh
2008-07-17  6:03         ` dinesh
2008-07-17 10:56           ` Mark Brown
2008-07-17 11:26             ` Nobin Mathew
2008-07-17 12:05               ` Jon Smirl
2008-07-17 16:02   ` [PATCH v2 3/3] ALSA SoC: Add Texas Instruments TLV320AIC26 codec driver Timur Tabi
2008-07-14 13:49 ` [alsa-devel] [PATCH v2 1/3] ALSA SoC: Add OpenFirmware helper for matching bus and codec drivers Mark Brown
2008-07-14 14:13   ` Jon Smirl
2008-07-14 15:05     ` Mark Brown
2008-07-14 16:14       ` Timur Tabi
2008-07-14 16:27         ` Grant Likely
2008-07-14 16:53         ` Mark Brown
2008-07-14 17:21           ` Grant Likely
2008-07-14 18:36             ` Mark Brown
2008-07-14 18:40               ` Timur Tabi
2008-07-14 18:49                 ` Mark Brown
2008-07-14 18:53                   ` Timur Tabi
2008-07-14 22:28                 ` Grant Likely
2008-07-14 23:45                 ` Jon Smirl [this message]
2008-07-15 10:13                   ` Mark Brown
2008-07-15 13:08                     ` Jon Smirl
2008-07-15 14:04                       ` Mark Brown
2008-07-14 15:51     ` Timur Tabi
2008-07-14 17:06   ` Grant Likely
2008-07-14 17:16     ` Mark Brown
2008-07-14 17:22       ` Grant Likely
2008-07-18  7:17       ` Grant Likely
2008-07-18 10:00         ` Mark Brown
2008-07-18 14:59         ` Timur Tabi
2008-07-14 14:16 ` Anton Vorontsov
2008-07-14 17:11   ` Grant Likely

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=9e4733910807141645l404881b0m1d17d88c26792c78@mail.gmail.com \
    --to=jonsmirl@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=liam.girdwood@wolfsonmicro.com \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=timur@freescale.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 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).