public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Uli Luckas <u.luckas@road.de>
To: Ryan Mallon <ryan@bluewatersys.com>
Cc: Jean Delvare <khali@linux-fr.org>,
	Russell King - ARM Linux <linux@arm.linux.org.uk>,
	i2c@lm-sensors.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH, RFC] Earlier I2C initialization
Date: Tue, 10 Jun 2008 11:46:11 +0200	[thread overview]
Message-ID: <200806101146.11915.u.luckas@road.de> (raw)
In-Reply-To: <484EF343.9090102@bluewatersys.com>

OT: David Brownell, I get bounces from your mail server refering me to a site 
that is down:
http://worldnet.att.net/general-info/bls_info/block_inquiry.html

On Tuesday, 10. June 2008, Ryan Mallon wrote:
> Jean Delvare wrote:
> > On Tue, 10 Jun 2008 09:27:34 +1200, Ryan Mallon wrote:
> >> David Brownell wrote:
> >>> On Monday 09 June 2008, Ryan Mallon wrote:
> >>>> We have made i2c the first driver subsystem to come up in our 2.6.20
> >>>> kernel since we use i2c io expanders for power domain control. All we
> >>>> did was change drivers/Makefile so that obj-$(CONFIG_I2C) += i2c/ is
> >>>> at the very top of the file. We didn't have any problems with doing
> >>>> this. YMMV of course.
> >
> > Why don't you simply initialize the drivers in question with
> > subsys_initcall()? That's what i2c-pnx, i2c-omap, i2c-davinci and
> > tps65010 are doing at the moment.
>
> How does this work for embedded devices where the same architecture is
> used in many different configurations? For example, we have a PXA270
> setup where we need i2c early, but many other PXA setups do not, so
> making i2c-pxa subsys_initcall to support a single board is maybe the
> wrong way to go?
>
Why would an architecture not want i2c to be available early? The only reason 
would be dependencies of the bus driver. These are constant over all pxa 
platforms though. So most likely the platforms that don't _need_ early i2c 
just don't care.

-- 

------- ROAD ...the handyPC Company - - -  ) ) )

Uli Luckas
Software Development

ROAD GmbH
Bennigsenstr. 14 | 12159 Berlin | Germany
fon: +49 (30) 230069 - 64 | fax: +49 (30) 230069 - 69
url: www.road.de

Amtsgericht Charlottenburg: HRB 96688 B
Managing directors: Hans-Peter Constien, Hubertus von Streit

  reply	other threads:[~2008-06-10  9:46 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200806091541.43899.u.luckas@road.de>
     [not found] ` <20080609135739.GE30971@flint.arm.linux.org.uk>
     [not found]   ` <484D947D.1090900@bluewatersys.com>
     [not found]     ` <200806091359.12791.david-b@pacbell.net>
2008-06-09 21:27       ` [PATCH, RFC] Earlier I2C initialization Ryan Mallon
2008-06-10  6:57         ` Jean Delvare
2008-06-10 20:55           ` David Brownell
2008-06-11  8:11             ` Jean Delvare
2008-06-11  9:00               ` Russell King - ARM Linux
2008-06-11  9:14                 ` Jean Delvare
2008-06-11 18:31               ` David Brownell
2008-06-12 18:44                 ` Jean Delvare
2008-06-12 19:57                   ` David Brownell
2008-06-24 17:06                     ` Jean Delvare
2008-06-11 20:23               ` Ryan Mallon
2008-06-11 12:05                 ` Jean Delvare
2008-06-10 21:33           ` Ryan Mallon
2008-06-10  9:46             ` Uli Luckas [this message]
2008-06-11  3:12           ` Ryan Mallon
2008-06-11  7:40             ` Jean Delvare
2008-06-11 20:13               ` Ryan Mallon
2008-06-11 12:18                 ` Jean Delvare
2008-06-11 20:27                   ` David Brownell
2008-06-11 20:54                     ` Jean Delvare
2008-06-11 21:24                       ` Ryan Mallon
2008-06-24 16:39                         ` Jean Delvare
2008-06-26 21:12                           ` Ryan Mallon
2008-06-27 10:41                             ` Jean Delvare
2008-06-29 20:34                               ` Ryan Mallon
2008-06-11 21:31                       ` Maciej W. Rozycki
2008-06-12 20:21                         ` David Brownell

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=200806101146.11915.u.luckas@road.de \
    --to=u.luckas@road.de \
    --cc=i2c@lm-sensors.org \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=ryan@bluewatersys.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