RE: [PATCH v2 2/2] misc: Add iop driver for Sunplus SP7021

[Tony Huang 黃懷厚]

Dear Greg KH, Arnd:

> -----Original Message-----
> From: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>
> Sent: Friday, December 3, 2021 6:39 PM
> To: Tony Huang <tonyhuang.sunplus@xxxxxxxxx>
> Cc: derek.kiernan@xxxxxxxxxx; dragan.cvetic@xxxxxxxxxx; arnd@xxxxxxxx;
> linux-kernel@xxxxxxxxxxxxxxx; robh+dt@xxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx;
> Wells Lu 呂芳騰 <wells.lu@xxxxxxxxxxx>; Tony Huang 黃懷厚
> <tony.huang@xxxxxxxxxxx>
> Subject: Re: [PATCH v2 2/2] misc: Add iop driver for Sunplus SP7021
> 
> On Fri, Dec 03, 2021 at 11:48:45AM +0800, Tony Huang wrote:
> > IOP (IO Processor) embedded inside SP7021 which is used as Processor
> > for I/O control, RTC wake-up and cooperation with CPU & PMC in power
> > management purpose.
> > The IOP core is DQ8051, so also named IOP8051, it supports dedicated
> > JTAG debug pins which share with SP7021.
> > In standby mode operation, the power spec reach 400uA.
> >
> > Signed-off-by: Tony Huang <tonyhuang.sunplus@xxxxxxxxx>
> > ---
> > Changes in v2:
> >  - Addressed comments from Arnd Bergmann.
> >  - Addressed comments from Greg KH.
> >  - Addressed comments from kernel test robot.
> >
> >  MAINTAINERS                    |   1 +
> >  drivers/misc/Kconfig           |   1 +
> >  drivers/misc/Makefile          |   1 +
> >  drivers/misc/iop/Kconfig       |  13 ++
> >  drivers/misc/iop/Makefile      |   6 +
> 
> Why do you need a subdirectory for a single .c file?
> 

1. Currently my bin file is placed in the root file system. I need to wait for the kernel booting to succeed before loading the bin code.
2. In addition, I need the kernel booting process to be able to mount the iop module and load bin file.I need to put bin file in /iop.
Do you have a good way to load bin code during kernel booting?

Thanks