From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christophe Ricard Subject: Re: [PATCH v2 05/12] tpm: Use read/write_bytes for drivers without more specialized methods Date: Thu, 14 Apr 2016 23:46:33 +0200 Message-ID: <57100FB9.5050908@gmail.com> References: <1460577351-24632-1-git-send-email-christophe-h.ricard@st.com> <1460577351-24632-6-git-send-email-christophe-h.ricard@st.com> <20160413204230.GB3836@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160413204230.GB3836-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: jean-luc.blanc-qxv4g6HH51o@public.gmane.org, ashley-fm2HMyfA2y6tG0bUXCXiUA@public.gmane.org, tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, christophe-h.ricard-qxv4g6HH51o@public.gmane.org, benoit.houyere-qxv4g6HH51o@public.gmane.org List-Id: tpmdd-devel@lists.sourceforge.net On 13/04/2016 22:42, Jason Gunthorpe wrote: > On Wed, Apr 13, 2016 at 09:55:44PM +0200, Christophe Ricard wrote: >> Some drivers might choose to implement only functions for transferring an >> arbitrary number of bytes, without special handling of word or dword >> transfers. > Hurm, better to provide common write/read16/32 helpers that a driver can > just dump into it's function pointers rather than using null, > simpler/faster. It is not really clear to me here. In short, Do you expect me to drop this patch and force drivers to implement their own write/read16/32 helpers so that it will avoid a null check ? > Jason ------------------------------------------------------------------------------ Find and fix application performance issues faster with Applications Manager Applications Manager provides deep performance insights into multiple tiers of your business applications. It resolves application problems quickly and reduces your MTTR. Get your free trial! https://ad.doubleclick.net/ddm/clk/302982198;130105516;z