Re: [PATCHv3] i2c: cadence: Fix the driver in interrupt flurry case

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Feb 20, 2019 at 08:18:17AM +0100, Michal Simek wrote:
> On 20. 02. 19 8:13, shubhrajyoti.datta@xxxxxxxxx wrote:
> > From: Shubhrajyoti Datta <shubhrajyoti.datta@xxxxxxxxxx>
> > 
> > In case there are a lot of interrupts then the hold bit is not
> > released. Protect the code by making it atomic by disabling interrupts.
> > 
> > Fixes: df8eb5691c48 ("i2c: Add driver for Cadence I2C controller"
> 
> Nit. Missing ).

For, me this is not a nit in this case, so I want this fixed. As I said
before, your patches often feel rushed, and I don't want to be offloaded
with fixing all these nits all the time. So, please spend the few
seconds double checking everything about your patch.

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux