Re: [PATCH] mfd:Add SUNIX mfd & PCIe driver

[Lee Jones]

On Fri, 28 May 2021, Moriis Ku wrote:

> From: Morris Ku <saumah@xxxxxxxxx>
> 
> Add SUNIX mfd & PCIe driver
> 
> Cc: Jason Lee <jason_lee@xxxxxxxxx>
> Cc: Taian Chen <taian.chen@xxxxxxxxx>
> Signed-off-by: Morris Ku <saumah@xxxxxxxxx>
> ---
>  sdc_mfd.c | 513 ++++++++++++++++++++++++++++++++++++++++++++++++++++++

To be frank, this is a bit of a mess.

Let's take one step back for a moment.

Please tell me how this compares to any existing driver currently
present in drivers/mfd.  I'm 100% sure this H/W is not different to
anything we've seen before.  Thus, my advice is to base it off of a
piece of code that has been previously accepted and take it from
there.

>  sdc_mfd.h |  61 +++++++
>  sdc_pci.c |  92 ++++++++++
>  3 files changed, 666 insertions(+)
>  create mode 100644 sdc_mfd.c
>  create mode 100644 sdc_mfd.h
>  create mode 100644 sdc_pci.c


-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog