[RFC PATCH 01/21] mfd: uniphier: add UniPhier MFD driver
Lee Jones
lee.jones at linaro.org
Wed May 11 00:38:55 PDT 2016
On Wed, 11 May 2016, Masahiro Yamada wrote:
> Hi Lee,
>
> 2016-05-10 20:26 GMT+09:00 Lee Jones <lee.jones at linaro.org>:
> >> +};
> >> +MODULE_DEVICE_TABLE(of, uniphier_mfd_match);
> >
> > NACK. Please do not mix MFD and DT registration.
>
> OK, thanks for review.
>
>
> My basic thought was:
>
> - Add an MFD node to my DTS only once.
> If I need to expand it in the future, I will just
> need to modify drivers/mtd/uniphier-mfd.c.
> It is just kernel-internal modification, so it does not
> bother DT binding at all.
>
> - Simplify the node topology of DTS.
Just describe the hardware in DT. If you do so accurately, there
should be no requirement to change it (in theory).
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
More information about the linux-arm-kernel
mailing list