From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeremy Kerr Subject: Re: [PATCH 05/16] drivers/fsi: Add fake master driver Date: Thu, 8 Dec 2016 10:27:46 +1100 Message-ID: References: <1481069677-53660-1-git-send-email-christopher.lee.bostic@gmail.com> <1481069677-53660-6-git-send-email-christopher.lee.bostic@gmail.com> <20161207120905.GD7054@leverpostej> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20161207120905.GD7054@leverpostej> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Mark Rutland , Chris Bostic Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, geert+renesas@glider.be, andrew@aj.id.au, gregkh@linuxfoundation.org, mturquette@baylibre.com, linux@armlinux.org.uk, Chris Bostic , sre@kernel.org, robh+dt@kernel.org, benh@kernel.crashing.org, joel@jms.id.au, alistair@popple.id.au, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org Hi Mark & Chris, > On Tue, Dec 06, 2016 at 06:14:26PM -0600, Chris Bostic wrote: >> From: Jeremy Kerr >> >> For debugging, add a fake master driver, that only supports reads, >> returning a fixed set of data. > >> +config FSI_MASTER_FAKE >> + tristate "Fake FSI master" >> + depends on FSI >> + ---help--- >> + This option enables a fake FSI master driver for debugging. >> +endif > >> +static const struct of_device_id fsi_master_fake_match[] = { >> + { .compatible = "ibm,fsi-master-fake" }, >> + { }, >> +}; > > NAK. > > DT should be treated as an ABI, and should describe the HW explicitly. > This makes no sense. This is also missing a binding document. > > Have your module take a module parameter allowing you to bind it to > arbitrary devices, or do something like what PCI does where you can > bind/unbind arbitrary drivers to devices using sysfs. This driver is purely for testing the FSI engine scan code; we could probably just drop this patch since I suspect that it's no longer useful (now that we have an actual master driver). If we do want to keep it though, I'd say we remove the device tree dependency; all this is doing at the moment is triggering the ->probe, and there are better ways to do that. Cheers, Jeremy