lkml.org 
[lkml]   [2017]   [Nov]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [v11,1/4] drivers: jtag: Add JTAG core driver
On Tue, Nov 14, 2017 at 11:19:16AM +0000, Russell King - ARM Linux wrote:
> On Tue, Nov 14, 2017 at 12:10:46PM +0100, gregkh@linuxfoundation.org wrote:
> > On Tue, Nov 14, 2017 at 10:34:49AM +0000, Oleksandr Shamray wrote:
> > > Greg, what you can suggest about it. May be better to add again single-open()-per-device lock with right locking way like:
> > >
> > > >if (mutex_lock_interruptible(&jtag->open_lock)) {
> >
> > You would stall an open? Why not just return saying you can't do that?
>
> Greg, I think you need to look at the code again.

Heh, sorry, I probably got it wrong. I'll be glad to review it in a
real patch when needed, but not in the middle of the merge window :)

thanks,

greg k-h

\
 
 \ /
  Last update: 2017-11-14 12:32    [W:0.045 / U:2.876 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site