From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ye0-x232.google.com (mail-ye0-x232.google.com [IPv6:2607:f8b0:4002:c04::232]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority" (not verified)) by ozlabs.org (Postfix) with ESMTPS id C1A312C00BD for ; Fri, 20 Sep 2013 22:18:41 +1000 (EST) Received: by mail-ye0-f178.google.com with SMTP id m5so77228yen.23 for ; Fri, 20 Sep 2013 05:18:38 -0700 (PDT) Sender: Tejun Heo Date: Fri, 20 Sep 2013 07:18:34 -0500 From: Tejun Heo To: Russell King Subject: Re: [PATCH 39/51] DMA-API: others: use dma_set_coherent_mask() Message-ID: <20130920121834.GB7630@mtj.dyndns.org> References: <20130919212235.GD12758@n2100.arm.linux.org.uk> <20130920121652.GA7630@mtj.dyndns.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20130920121652.GA7630@mtj.dyndns.org> Cc: alsa-devel@alsa-project.org, linux-doc@vger.kernel.org, David Airlie , linux-mmc@vger.kernel.org, linux-fbdev@vger.kernel.org, linux-nvme@lists.infradead.org, linux-ide@vger.kernel.org, devel@driverdev.osuosl.org, linux-samsung-soc@vger.kernel.org, Joonyoung Shim , linux-scsi@vger.kernel.org, e1000-devel@lists.sourceforge.net, Seung-Woo Kim , b43-dev@lists.infradead.org, linux-media@vger.kernel.org, devicetree@vger.kernel.org, Inki Dae , Kukjin Kim , dri-devel@lists.freedesktop.org, linux-tegra@vger.kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Solarflare linux maintainers , netdev@vger.kernel.org, linux-usb@vger.kernel.org, linux-wireless@vger.kernel.org, Kyungmin Park , linux-crypto@vger.kernel.org, uclinux-dist-devel@blackfin.uclinux.org, linuxppc-dev@lists.ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, Sep 20, 2013 at 07:16:52AM -0500, Tejun Heo wrote: > On Fri, Sep 20, 2013 at 12:11:38AM +0100, Russell King wrote: > > The correct way for a driver to specify the coherent DMA mask is > > not to directly access the field in the struct device, but to use > > dma_set_coherent_mask(). Only arch and bus code should access this > > member directly. > > > > Convert all direct write accesses to using the correct API. > > > > Signed-off-by: Russell King > > Acked-by: Tejun Heo > > The patch is pretty widely spread. I don't mind how it gets routed > but what's the plan? Hmm... maybe hte pata_ixp4xx_cf.c part should be moved to the one which updates pata_octeon_cf.c? Thanks. -- tejun