All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: jassi brar <jassisinghbrar@gmail.com>
Cc: alsa-devel@alsa-project.org, Jassi Brar <jassi.brar@samsung.com>,
	ben-linux@fluff.org, Liam Girdwood <lrg@slimlogic.co.uk>
Subject: Re: Reforming S3C I2S towards supporting I2Sv4
Date: Wed, 10 Mar 2010 10:18:17 +0000	[thread overview]
Message-ID: <20100310101817.GC24422@rakim.wolfsonmicro.main> (raw)
In-Reply-To: <1b68c6791003100122u16294a69y541f28b6cabc69a2@mail.gmail.com>

On Wed, Mar 10, 2010 at 06:22:29PM +0900, jassi brar wrote:
> On Wed, Mar 10, 2010 at 6:04 PM, Liam Girdwood <lrg@slimlogic.co.uk> wrote:

> > I don't think your reason of "co-ordination between ASoC and S3C ARCH
> > trees" can justify breaking kernel policy.

> I believe the policy is not against moving definitions of bits that are
> tightly linked to the device controller esp when we seeing more and more
> SoCs coming with standard of-the-shelf third party controllers.

Plus controllers that may be used in multiple subsystems (like the
generic serial ports a lot of SoCs have).  Some architectures also like
to keep headers for the IPs with the arch either due to preference or
due to device differences which can be hidden from the driver with arch
level compile stuff.

I personally don't mind either way for things like the I2S controller
which don't have any obvious application outside ASoC so whatever you
guys and Ben want works for me.

> Then, I really want S3C ASoC modifications to not wait on S3C
> ARCH patches appearing upstream first.

If the headers stay in arch/arm we can still deal with that by either
agreeing to merge them via ASoC (as has been done in the past) or by
creating a branch which is merged into both ASoC and Samsung trees.  It
is obviously less effort to avoid this, but it's not insurmountable.

  reply	other threads:[~2010-03-10 10:18 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-10  7:48 Reforming S3C I2S towards supporting I2Sv4 Jassi Brar
2010-03-10  8:24 ` Liam Girdwood
2010-03-10  8:31   ` jassi brar
2010-03-10  9:04     ` Liam Girdwood
2010-03-10  9:22       ` jassi brar
2010-03-10 10:18         ` Mark Brown [this message]
2010-03-10  8:35 ` jassi brar
2010-03-10  8:51   ` Liam Girdwood
2010-03-10  9:02     ` jassi brar
2010-03-10 10:20   ` Mark Brown
2010-03-10 14:23 ` Mark Brown
2010-04-06  0:33   ` jassi brar
2010-04-27  2:10     ` jassi brar
2010-04-27  2:58   ` Ben Dooks
2010-04-27  4:25     ` jassi brar
2010-05-02  4:38       ` Kyungmin Park
2010-05-02  8:28         ` jassi brar

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=20100310101817.GC24422@rakim.wolfsonmicro.main \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=ben-linux@fluff.org \
    --cc=jassi.brar@samsung.com \
    --cc=jassisinghbrar@gmail.com \
    --cc=lrg@slimlogic.co.uk \
    /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.