From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from newverein.lst.de (verein.lst.de [213.95.11.211]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3xf42Z6r3BzDrVs for ; Sat, 26 Aug 2017 00:55:00 +1000 (AEST) Date: Fri, 25 Aug 2017 16:54:57 +0200 From: Christoph Hellwig To: Rob Herring Cc: Robin Murphy , Frank Rowand , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , Christoph Hellwig , Marek Szyprowski , Stefan Wahren , Andreas =?iso-8859-1?Q?F=E4rber?= , Hans Verkuil , Johan Hovold , linuxppc-dev Subject: Re: [PATCH 2/2] of: Restrict DMA configuration Message-ID: <20170825145457.GA17842@lst.de> References: <0819179085df6c41c70e83a2c5c138b95c0386b3.1502468875.git.robin.murphy@arm.com> <82633e62b64e28dc18bc466319065b92faf2414f.1502468875.git.robin.murphy@arm.com> <6819c6ae-e898-3293-d8b0-7a80108d7628@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, Aug 15, 2017 at 09:19:50AM -0500, Rob Herring wrote: > > For the immediate issue at hand, I guess the alternative plan of attack > > would be to stick a flag in struct bus_type for the bus drivers > > themselves to opt into generic DMA configuration. That at least keeps > > everything within the kernel (and come to think of it probably works > > neatly for modular bus types as well). > > I'm fine with the change as is, it's really just the commit text I'm > commenting on. Robin, can you resend this with an updated commit text?