From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jarkko Sakkinen Subject: Re: [PATCH v3 06/11] tpm: Split out the devm stuff from tpmm_chip_alloc Date: Tue, 23 Feb 2016 13:31:21 +0200 Message-ID: <20160223113108.GA5014@intel.com> References: <1455885728-10315-1-git-send-email-stefanb@linux.vnet.ibm.com> <1455885728-10315-7-git-send-email-stefanb@linux.vnet.ibm.com> <20160222211414.GC3310@intel.com> <20160222221328.GA27228@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: <20160222221328.GA27228-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: tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: tpmdd-devel@lists.sourceforge.net On Mon, Feb 22, 2016 at 03:13:28PM -0700, Jason Gunthorpe wrote: > On Mon, Feb 22, 2016 at 11:14:14PM +0200, Jarkko Sakkinen wrote: > > On Fri, Feb 19, 2016 at 07:42:03AM -0500, Stefan Berger wrote: > > > From: Jason Gunthorpe > > > > > > tpm_chip_alloc becomes a typical subsystem allocate call. > > > > Maybe a more verbose commit message? > > What more do you want to say? I would just add that: "This is needed for virtual devices because they are not associated to any parent." Of course when you read the patch set in reverse order it is clear but this one sentence would make it easier to read it when you iterate it from beginning :) /Jarkko > > > spin_lock(&driver_lock); > > > chip->dev_num = find_first_zero_bit(dev_mask, TPM_NUM_DEVICES); > > > + if (chip->dev_num < TPM_NUM_DEVICES) > > > + set_bit(chip->dev_num, dev_mask); > > > > I guess this change should a separate patch. > > Hurm, Stefan? This was not in the patch I sent you? This hunk looks > incomplete to me, and replaced by the IDR stuff anyhow. Drop it? > > You need to put your Signed-off-by on patches you make notable changes > to, this one would qualify > > Jason ------------------------------------------------------------------------------ Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140