lkml.org 
[lkml]   [2017]   [Oct]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 03/14] soundwire: Add Master registration
On Thu, Oct 19, 2017 at 08:33:19AM +0530, Vinod Koul wrote:

> + /*
> + * SDW is an enumerable bus, but devices can be powered off. So,
> + * they won't be able to report as present.
> + *
> + * Create Slave devices based on Slaves described in
> + * the respective firmware (ACPI/DT)
> + */
> +
> + if (IS_ENABLED(CONFIG_ACPI) && bus->dev && ACPI_HANDLE(bus->dev))
> + ret = sdw_acpi_find_slaves(bus);
> + else if (IS_ENABLED(CONFIG_OF) && bus->dev && bus->dev->of_node)
> + ret = sdw_of_find_slaves(bus);
> + else
> + ret = -ENOTSUPP; /* No ACPI/DT so error out */

Devices *can* be powered off but is there any reason that a system
couldn't be designed with them always powered? Also given that we don't
actually have any DT support the stubs for it are at best misleading,
it's not going to be hard for someone to add them later.

> + mutex_lock(&bus->bus_lock);
> + if (!list_empty(&bus->slaves))
> + list_del(&slave->node);

Shouldn't that be a while? Or at least warn if there's anything extra
there. The code just looks very wrong as is.
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2017-10-22 17:49    [W:0.465 / U:0.328 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site