Re: [PATCH 08/16] wireless: Replace spin_is_locked() with lockdep

From: Kalle Valo
Date: Thu Oct 04 2018 - 06:05:18 EST


Lance Roy <ldr709@xxxxxxxxx> writes:

> On Wed, Oct 03, 2018 at 12:06:48PM +0300, Kalle Valo wrote:
>> Lance Roy <ldr709@xxxxxxxxx> writes:
>>
>> > lockdep_assert_held() is better suited to checking locking requirements,
>> > since it won't get confused when someone else holds the lock. This is
>> > also a step towards possibly removing spin_is_locked().
>> >
>> > Signed-off-by: Lance Roy <ldr709@xxxxxxxxx>
>> > Cc: Daniel Drake <dsd@xxxxxxxxxx>
>> > Cc: Ulrich Kunitz <kune@xxxxxxxxxxxxxx>
>> > Cc: Kalle Valo <kvalo@xxxxxxxxxxxxxx>
>> > Cc: "David S. Miller" <davem@xxxxxxxxxxxxx>
>> > Cc: <linux-wireless@xxxxxxxxxxxxxxx>
>> > Cc: <netdev@xxxxxxxxxxxxxxx>
>> > ---
>> > drivers/net/wireless/zydas/zd1211rw/zd_mac.c | 2 +-
>> > 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> Should I take this or is it going through some other tree?
>
> Sure you can take this if you like. If not, Paul McKenney will take it in his
> tree.

Ok, then I'll take this to wireless-drivers-next. And I'll change the
title prefix to "zd1211rw: ".

--
Kalle Valo