Re: [PATCH] slimbus: Linux driver framework for SLIMbus.

From: Mark Brown
Date: Mon Jun 04 2012 - 06:27:47 EST


On Mon, Jun 04, 2012 at 03:21:06AM -0700, Sagar Dharia wrote:

> > b) Similarly to (a) the driver may be probed before the device has
> > been given a logical address (LA). This makes sense in the case of
> > driver that turns on the device (say via gpio) once the bus has
> > booted. However, the driver then needs to sit and poll
> > slim_get_logical_addr() until the logical address.

> This is not the case anymore.
> While taking care of the comments for RFC, I have introduced a completion
> that will be signalled when LA is given to the device. The driver can wait
> on that completion (wait_enum) instead of polling.

This would mean that the thread the probe is happening in will be
blocked until the LA is assigned. That sounds like it might cause
problems, either slowing things down or worst case causing a deadlock.

Attachment: signature.asc
Description: Digital signature