From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jarkko Sakkinen Subject: Re: [PATCH 1/3] tpm_crb: map locality registers Date: Tue, 11 Oct 2016 22:48:59 +0300 Message-ID: <20161011194859.GA11477@intel.com> References: <1476177787-15003-1-git-send-email-jarkko.sakkinen@linux.intel.com> <1476177787-15003-3-git-send-email-jarkko.sakkinen@linux.intel.com> <20161011170143.GC6881@obsidianresearch.com> <20161011182100.GC6900@intel.com> <20161011182922.GB20253@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20161011182922.GB20253-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: tpmdd-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org To: Jason Gunthorpe Cc: g@intel.com, "moderated list:TPM DEVICE DRIVER" , open list List-Id: tpmdd-devel@lists.sourceforge.net On Tue, Oct 11, 2016 at 12:29:22PM -0600, Jason Gunthorpe wrote: > On Tue, Oct 11, 2016 at 09:21:00PM +0300, Jarkko Sakkinen wrote: > > On Tue, Oct 11, 2016 at 11:01:43AM -0600, Jason Gunthorpe wrote: > > > On Tue, Oct 11, 2016 at 12:23:04PM +0300, Jarkko Sakkinen wrote: > > > > In order to provide access to locality registers, this commits adds > > > > mapping of the head of the CRB registers, which are located right > > > > before the control area. > > > > > > I think you should squash this into the prior patch, no sense in > > > changing all these lines twice > > > > > > But looks better to me. > > > > So... do you suggest to squash with request/reliquish locality stuff? > > Just checking because this is 1/3 :) > > Hm, maybe my mailer went wrong, but I saw the original patch right > before this one.. Never mind I did format-patch -o to the same folder where I had the first series so I posted the RFC patches by accident. Sorry. Should always do --dry-run. > Jason /Jarkko ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot