Re: [PATCH v6 2/2] staging: ion: create one device entry per heap

From: Laura Abbott <labbott@redhat.com>
To: Mark Brown <broonie@kernel.org>
Cc: Benjamin Gaignard <benjamin.gaignard@linaro.org>,
	sumit.semwal@linaro.org, gregkh@linuxfoundation.org,
	arve@android.com, riandrews@android.com,
	dan.carpenter@oracle.com, devel@driverdev.osuosl.org,
	linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
	linux-api@vger.kernel.org
Subject: Re: [PATCH v6 2/2] staging: ion: create one device entry per heap
Date: Tue, 31 Oct 2017 12:45:25 -0700	[thread overview]
Message-ID: <75c8c6ac-2f56-f477-8fcd-794504499111@redhat.com> (raw)
In-Reply-To: <20171031191153.avolosp6n5gb3xao@sirena.co.uk>

On 10/31/2017 12:11 PM, Mark Brown wrote:
> On Tue, Oct 31, 2017 at 12:03:35PM -0700, Laura Abbott wrote:
> 
>> I'm not a fan of the platform bus but I have mixed feelings about
>> creating a dedicated bus type. I guess if we really need a bus
>> type we can do it later?
> 
> There was a discussion a while ago in the context of I2C/SPI MFDs
> which concluded that if you need a bus and it's going to be effectively
> noop then you should just use the platform bus as anything else will
> consist almost entirely of cut'n'paste from the platform bus with some
> light sed usage and code duplication is bad.  It's not super lovely as
> it's not actually a memory mapped device but it's the best idea we've
> got.
> 

Thanks for the pointer.