From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751533AbeEDJC0 (ORCPT ); Fri, 4 May 2018 05:02:26 -0400 Received: from mail-wr0-f177.google.com ([209.85.128.177]:36415 "EHLO mail-wr0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751287AbeEDJCU (ORCPT ); Fri, 4 May 2018 05:02:20 -0400 X-Google-Smtp-Source: AB8JxZo9w6hNLbN4fm6440edGeZdIWJ9uB1Cmzpw9HwSHqu06AfajkHMZ2NA15UWMK+5+u//0dI5PQ== Date: Fri, 4 May 2018 10:02:16 +0100 From: Lee Jones To: Andy Shevchenko Cc: John Garry , xuwei5@huawei.com, mika.westerberg@linux.intel.com, rjw@rjwysocki.net, linux-kernel@vger.kernel.org, arnd@arndb.de, graeme.gregory@linaro.org, helgaas@kernel.org, z.liuxinliang@hisilicon.com, linuxarm@huawei.com Subject: Re: [PATCH 1/2] HISI LPC: Reference static MFD cells for ACPI support Message-ID: <20180504090216.GC3928@dell> References: <1525360119-102166-1-git-send-email-john.garry@huawei.com> <1525360119-102166-2-git-send-email-john.garry@huawei.com> <1525366486.21176.653.camel@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1525366486.21176.653.camel@linux.intel.com> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 03 May 2018, Andy Shevchenko wrote: > On Thu, 2018-05-03 at 23:08 +0800, John Garry wrote: > > Currently for ACPI support the driver models the host as > > an MFD. For a device connected to the LPC bus, we dynamically > > create an MFD cell for that device, configuring the cell > > name and ACPI match parameters manually. This makes supporting > > named devices and also special setup handling for certain devices > > awkward, as we would need to introduce some special ACPI device > > handling according to device HID. > > > > To avoid this, create reference static MFD cells for known > > child devices, so when adding an MFD cell we can fix the cell > > platform data as required. For this, a setup callback function > > is added. > > > > For now, only the IPMI cell is added. > > > +static const struct mfd_cell *hisi_lpc_acpi_mfd_get_cell(const char > > *hid) > > +{ > > + const struct hisi_lpc_acpi_mfd_cell *cell = > > hisi_lpc_acpi_mfd_cells; > > + > > + for (; cell && cell->mfd_cell.name; cell++) { > > + const struct mfd_cell *mfd_cell = &cell->mfd_cell; > > + const struct mfd_cell_acpi_match *acpi_match; > > + > > + acpi_match = mfd_cell->acpi_match; > > + if (!strcmp(acpi_match->pnpid, hid)) > > + return mfd_cell; > > + } > > + > > + return NULL; > > +} > > I'm not sure I understand why MFD core can't do it (as seen in lines > drivers/mfd/core.c:105 and below). You shouldn't be using the MFD API outside of MFD anyway. Either it is an MFD driver, or it isn't. If it is, please move it. If it's not, please don't use the API. My current suspicion is that the driver needs splitting and only part of it ends up in MFD. -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog