Re: [PATCH 4/7] gpiolib: implement dev_gpiochip_{add,remove} calls

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: David Brownell
Date: Wednesday, October 22, 2008 - 2:04 pm

On Wednesday 22 October 2008, Anton Vorontsov wrote:

That's not a problem.  Why would you want to remove it?



Right.  Such board description data should be made available
early in boot.  As a rule:  before arch_initcall() finishes,
so that subsys_initcall() code can use the associated GPIOs.

(It's fairly well acknowledged that init dependency handling
has a lot of problems.  Until that's fixed ... for GPIOs, the
general advice is to make sure everything is available by
subsys_initcall time,  so the subsystems which rely on GPIOs
to initialize -- power switches, resets, etc -- can initialize.
That can require i2c adapter drivers to use subsys_initcall,
for example.)



See above.  If you're doing it right, there's no problem.
That is, scan the OF tables early.  Just like PNP tables
get scanned early, for example.

- Dave



--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 2/7] i2c: add info->archdata field, Anton Vorontsov, (Thu Oct 16, 10:12 am)
[PATCH 3/7] of: fill the archdata for I2C devices, Anton Vorontsov, (Thu Oct 16, 10:12 am)
[PATCH 6/7] gpio/pca953x: convert to dev_gpiochip_add and ..., Anton Vorontsov, (Thu Oct 16, 10:13 am)
Re: [PATCH 2/7] i2c: add info->archdata field, Jean Delvare, (Fri Oct 17, 2:21 am)
Re: [PATCH 2/7] i2c: add info->archdata field, Benjamin Herrenschmidt, (Tue Oct 21, 5:27 pm)
Re: [PATCH 4/7] gpiolib: implement dev_gpiochip_{add,remov ..., Benjamin Herrenschmidt, (Tue Oct 21, 5:29 pm)
Re: [PATCH 4/7] gpiolib: implement dev_gpiochip_{add,remov ..., Benjamin Herrenschmidt, (Tue Oct 21, 7:27 pm)
Re: [PATCH 4/7] gpiolib: implement dev_gpiochip_{add,remov ..., Benjamin Herrenschmidt, (Tue Oct 21, 7:28 pm)
Re: [PATCH 3/7] of: fill the archdata for I2C devices, Grant Likely, (Tue Oct 21, 9:14 pm)
Re: [PATCH 2/7] i2c: add info->archdata field, Jean Delvare, (Tue Oct 21, 11:50 pm)
Re: [PATCH 2/7] i2c: add info->archdata field, Benjamin Herrenschmidt, (Wed Oct 22, 12:37 am)
Re: [PATCH 2/7] i2c: add info->archdata field, Anton Vorontsov, (Wed Oct 22, 3:08 am)
Re: [PATCH 2/7] i2c: add info->archdata field, Jean Delvare, (Wed Oct 22, 4:07 am)
Re: [PATCH 2/7] i2c: add info->archdata field, Anton Vorontsov, (Wed Oct 22, 5:50 am)
Re: [PATCH 4/7] gpiolib: implement dev_gpiochip_{add,remov ..., David Brownell, (Wed Oct 22, 2:04 pm)
Re: [PATCH 4/7] gpiolib: implement dev_gpiochip_{add,remov ..., Benjamin Herrenschmidt, (Wed Oct 22, 9:45 pm)
[PATCH 0/6 RFC] OF-glue devices for I2C/SPI (was: Re: [P ..., Anton Vorontsov, (Tue Oct 28, 10:45 am)
[PATCH 1/6] of/base: Add new helper of_should_create_pdev(), Anton Vorontsov, (Tue Oct 28, 10:46 am)
[PATCH 2/6] of/of_i2c: implement of_{,un}register_i2c_device, Anton Vorontsov, (Tue Oct 28, 10:46 am)
[PATCH 6/6] gpio: OpenFirmware bindings for the pca953x, Anton Vorontsov, (Tue Oct 28, 10:46 am)