[lkml]   [2008]   [Jun]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH, RFC] Earlier I2C initialization
    Jean Delvare wrote:
    > On Wed, 11 Jun 2008 13:27:09 -0700, David Brownell wrote:
    >> On Wednesday 11 June 2008, Jean Delvare wrote:
    >>> On Thu, 12 Jun 2008 08:13:09 +1200, Ryan Mallon wrote:
    >>>> As David suggested, if i2c is needed early
    >>>> in enough cases, why not just move it early in the link order? My patch
    >>>> was just an alternative approach which mimics the current behaviour, but
    >>>> makes it possible to get any i2c driver early. Why not just mark all of
    >>>> the drivers/busses that get used on embedded devices as subsys_initcall,
    >>>> just in case somebody needs them early?
    >>> Because this is an abuse of subsys_initcall? I guess that was
    >>> acceptable when only a couple drivers were doing that, but making it
    >>> official sounds bad.
    >> How would it be an abuse? On those systems, I2C is a "system bus"
    >> and needs to be initialized early for the same reasons PCI gets set
    >> up very early on PC hardware.
    > But the pci subsystem doesn't make use of subsys_initcall(). Instead,
    > it is simply placed early in the link order.
    > That being said, I'm not sure if the comparison with the PCI subsystem
    > holds... I am under the impression that PCI bus handling doesn't
    > require dedicated drivers? At least I can't see any under drivers/pci.
    >> There's no rule saying that subsystem initialization may not include
    >> the essential drivers -- in this case, i2c_adapter drivers. PCI hubs
    >> and bridges are certainly initialized very early, before module_init
    >> code runs...
    > Care to point me to actual code to backup this "certainly"?

    ryan@okiwi:pci$ grep initcall * -R

    At a quick glance (I don't know much about the PCI subsystem) at least
    the bus and class are registered early on.

    >> And in fact it seems a bit odd to think that initializing any bus
    >> subsystem shouldn't be allowed to include its bus adapters. It's
    >> not as if the subsystem has completed initializiation until those
    >> adapters are usable!!
    > I think it makes a lot of sense to initialize the core of a subsystem
    > early, so that all devices and drivers can be registered. This doesn't
    > imply registering the hardware bus drivers too, even though in some
    > cases it is also needed. I doubt that whoever designed subsys_initcall
    > meant it to be used for all bus drivers, otherwise he/she would have
    > named it, say, busdrv_initcall.

    At least in the case of i2c on many embedded devices we want the bus
    driver early, but many other i2c bus drivers don't need this. There
    seem to be two main options:

    1) Use subsys_initcall on the ones that may need to be early (current
    2) Split the bus drivers into two directories, say drivers/i2c-early and
    drivers/i2c, and put the former earlier in the link order.

    Maybe a decent compromise approach is to move the drivers which are used
    on embedded systems to a new directory such as drivers/i2c/embedded or
    drivers/i2c/soc and then only use subsys_initcall on bus drivers in that

    If all of the i2c drivers then go in the right place, ie drivers/gpio
    instead of drivers/i2c/chips, then only the bus drivers, not the device
    drivers, should ever need to use subsys_initcall right?

    > But don't get me wrong: if subsys_initcall is the way to go, that's
    > alright with me, that's way less work than having to move drivers to
    > different directories and fixing the link order.

    I don't think multiple directories for i2c at the drivers/ level is the
    right way to go, that just going to get confusing. I think moving the
    i2c bus drivers which may be needed early to a new sub directory under
    drivers/i2c and then tidying up the Kconfig to categorise the bus
    drivers as embedded (soc), pc, external, etc.


    Bluewater Systems Ltd - ARM Technology Solution Centre

    Ryan Mallon Unit 5, Amuri Park
    Phone: +64 3 3779127 404 Barbadoes St
    Fax: +64 3 3779135 PO Box 13 889
    Email: Christchurch, 8013
    Web: New Zealand
    Freecall Australia 1800 148 751 USA 1800 261 2934

     \ /
      Last update: 2008-06-11 23:27    [W:0.048 / U:3.480 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site