On 11/29/2012 01:59 AM, Alexander Sverdlin wrote:
> From: Alexander Sverdlin <[email protected]>
>
> of/spi: Honour "status=disabled" property of device
>
> Currently of_register_spi_devices() function registers all SPI devices,
> independetly from their status property in device tree. According to
> "ePAPR 1.1" spec, device should only be registered if there is no
> "status" property, or it has "ok" (or "okay") value (see
> of_device_is_available()). In case of "platform devices",
> of_platform_device_create_pdata() checks for "status" and ensures
> that disabled devices are not pupulated. But such check for SPI buses
> was missing until now. Fix it.
>
> Signed-off-by: Alexander Sverdlin <[email protected]>
> ---
This is under spi, but I've applied with the rest of the series (mdio
and i2c). It all goes to Grant anyway.
BTW, there's a problem that the subject line is duplicated in your
patches. I've fixed that up.
Rob
> --- linux.orig/drivers/spi/spi.c
> +++ linux/drivers/spi/spi.c
> @@ -819,7 +819,7 @@ static void of_register_spi_devices(stru
> if (!master->dev.of_node)
> return;
>
> - for_each_child_of_node(master->dev.of_node, nc) {
> + for_each_available_child_of_node(master->dev.of_node, nc) {
> /* Alloc an spi_device */
> spi = spi_alloc_device(master);
> if (!spi) {
> _______________________________________________
> devicetree-discuss mailing list
> [email protected]
> https://lists.ozlabs.org/listinfo/devicetree-discuss
>
------------------------------------------------------------------------------
Keep yourself connected to Go Parallel:
VERIFY Test and improve your parallel project with help from experts
and peers. http://goparallel.sourceforge.net
_______________________________________________
spi-devel-general mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/spi-devel-general