Hello Azhar, On 12/20/2017 08:15 PM, Shaikh, Azhar wrote: > > >> -----Original Message----- >> From: Jason Gunthorpe [mailto:[email protected]] >> Sent: Wednesday, December 20, 2017 10:55 AM >> To: Javier Martinez Canillas <[email protected]> >> Cc: [email protected]; James Ettle <[email protected]>; Hans de >> Goede <[email protected]>; Shaikh, Azhar <[email protected]>; >> Arnd Bergmann <[email protected]>; Jarkko Sakkinen >> <[email protected]>; Peter Huewe <[email protected]>; >> Greg Kroah-Hartman <[email protected]>; linux- >> [email protected] >> Subject: Re: [PATCH 1/4] tpm: fix access attempt to an already unmapped I/O >> memory region >> >> On Wed, Dec 20, 2017 at 07:21:31PM +0100, Javier Martinez Canillas wrote: >> >>>> The below draft fixes everything except #1. That needs a more >>>> thoughtful idea.. >>>> >>> >>> I'll just drop this patch from the series and you can fix all the >>> issues in the error / driver removal paths. It's not a dependency >>> anyways, I included it just because noticed the issue while reading the >>> code. >> >> Azhar, this means it becomes your problem :) >> > > Sure, I will fix this. > Javier you can drop this patch. >
On a second though, we should just merge $SUBJECT since is an obvious fix for one of the issues. Then you could fix the remaining bugs in error and driver removal paths. >> Since the patches Jarkko has queued from you introduce oops's you need to >> fix them.. >> >> Jason > > Regards, > Azhar Shaikh > Best regards, -- Javier Martinez Canillas Software Engineer - Desktop Hardware Enablement Red Hat
