Re: [PATCH 6/7 v2] HID: picoLCD: drop version check during probe

From: Bruno PrÃmont
Date: Mon Sep 17 2012 - 14:21:58 EST


Jiri,

You seem not to have applied this one yet (it doesn't show up in picolcd
branch).

Could you please apply it?
(or did I miss locking changes making it obsolete?)

Thanks,
Bruno

On Sun, 19 August 2012 Bruno PrÃmont <bonbons@xxxxxxxxxxxxxxxxx> wrote:
> Commit 4ea5454203d991ec85264f64f89ca8855fce69b0
> [HID: Fix race condition between driver core and ll-driver] introduced
> new locking around probe/remove functions that prevents any report/reply
> from hardware to reach driver until it returned from probe.
>
> As such, the ask-reply way to checking picoLCD firmware version during
> probe is bound to timeout and let probe fail.
>
> Drop the check to let driver successfully probe again (until locking issues
> are resolved allowing to reinstate the check).
>
> Signed-off-by: Bruno PrÃmont <bonbons@xxxxxxxxxxxxxxxxx>
> ---
>
> Changes since v1:
> - drop version check during probe instead of commenting it out.
>
>
> drivers/hid/hid-picolcd_core.c | 18 ------------------
> 1 files changed, 0 insertions(+), 18 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/