Re: [PATCH 2/7] mfd: MT6397: Add regmap for MT8135 and MT6397 SoC

From: Lee Jones
Date: Wed Nov 19 2014 - 12:05:13 EST


On Tue, 18 Nov 2014, Sascha Hauer wrote:

> On Tue, Nov 18, 2014 at 11:46:45AM +0000, Lee Jones wrote:
> > On Mon, 17 Nov 2014, Flora Fu wrote:
> >
> > > Add PMIC wrapper of MT8135 to access MFD MT6397.
> > > This is regmap of MT6397 MFD.
> > >
> > > Signed-off-by: Flora Fu <flora.fu@xxxxxxxxxxxx>
> > > ---
> > > drivers/mfd/Kconfig | 8 +
> > > drivers/mfd/Makefile | 1 +
> > > drivers/mfd/mt8135-pmic-wrap.c | 847 +++++++++++++++++++++++++++++++++++++++++
> >
> > All of the PMIC functionality needs removing from MFD and placed
> > somewhere else. I suggest either drivers/power or drivers/regulator.
>
> This is no PMIC functionality. The MT8135 has a unit which is is used to
> access the PMIC (which is not only a PMIC, but also Touchscreen
> interface and other stuff). This unit is called pmic-wrapper in the
> docs. See the introductory mail for a nice picture.

I saw the picture, it's very nice. Whatever this is, it's not an
MFD. It's a device which is located on an MFD. There is far too much
functional (the operative word here) code contained in this patch.

I'm not sure where this device should live, but I'm keen for MFD not
to become a dumping ground for ill-fitting devices. Happy for you to
create resources and register it from here, but the bulk for the
functional driver needs to live somewhere else.

--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/