From: "Tony Huang 黃懷厚" <tony.huang@sunplus.com>
To: Arnd Bergmann <arnd@arndb.de>, Tony Huang <tonyhuang.sunplus@gmail.com>
Cc: "Rob Herring" <robh+dt@kernel.org>,
DTML <devicetree@vger.kernel.org>,
"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
"Derek Kiernan" <derek.kiernan@xilinx.com>,
"Dragan Cvetic" <dragan.cvetic@xilinx.com>,
gregkh <gregkh@linuxfoundation.org>,
"Wells Lu 呂芳騰" <wells.lu@sunplus.com>
Subject: RE: [PATCH v9 2/2] misc: Add iop driver for Sunplus SP7021
Date: Wed, 2 Mar 2022 02:48:14 +0000 [thread overview]
Message-ID: <5c7e4c014858485a8b0eb120f85e98de@sphcmbx02.sunplus.com.tw> (raw)
In-Reply-To: <CAK8P3a1FZQ2LQco8D263+YXjcSQvzU6RHAL8SKcKy8hiLnHY8g@mail.gmail.com>
Dear Arnd, Gregkh:
> -----Original Message-----
> From: Arnd Bergmann <arnd@arndb.de>
> Sent: Monday, February 21, 2022 3:20 PM
> To: Tony Huang <tonyhuang.sunplus@gmail.com>
> Cc: Rob Herring <robh+dt@kernel.org>; DTML <devicetree@vger.kernel.org>;
> Linux Kernel Mailing List <linux-kernel@vger.kernel.org>; Derek Kiernan
> <derek.kiernan@xilinx.com>; Dragan Cvetic <dragan.cvetic@xilinx.com>; Arnd
> Bergmann <arnd@arndb.de>; gregkh <gregkh@linuxfoundation.org>; Wells Lu
> 呂芳騰 <wells.lu@sunplus.com>; Tony Huang 黃懷厚
> <tony.huang@sunplus.com>
> Subject: Re: [PATCH v9 2/2] misc: Add iop driver for Sunplus SP7021
>
> On Mon, Feb 21, 2022 at 3:14 AM Tony Huang
> <tonyhuang.sunplus@gmail.com> wrote:
> >
> > IOP(8051) embedded inside SP7021 which is used as Processor for I/O
> > control, monitor RTC interrupt 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@gmail.com>
> > ---
> > Changes in v9:
> > - Remove custom attributes sysfs.
>
> You are now back to a driver that does nothing at all, please make it use at
> least one kernel subsystem that it can hook up to before you send it again.
>
monitor RTC interrupt:
SP7021 system has its own RTC device driver (rtc-sunplus.c).
The user can set the RTC wake-up time through rtc-sunplus.c.
The IOP code does not need to increase the RTC subsystem function, set the RTC register.
When the linux kernel system. Only the 8051 CPU has power and can monitor the RTC wake-up interrupt.
> Also, when listing the functionality above, describe which subsystems you want
> to use for those in the long run, this would make it clearer to see which
> direction you want to take this driver when you add back the features.
>
> Arnd
prev parent reply other threads:[~2022-03-02 2:48 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-21 2:14 [PATCH v9 0/2] " Tony Huang
2022-02-21 2:14 ` [PATCH v9 1/2] dt-binding: misc: Add iop yaml file " Tony Huang
2022-02-21 2:14 ` [PATCH v9 2/2] misc: Add iop driver " Tony Huang
2022-02-21 7:20 ` Arnd Bergmann
2022-03-02 2:48 ` Tony Huang 黃懷厚 [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5c7e4c014858485a8b0eb120f85e98de@sphcmbx02.sunplus.com.tw \
--to=tony.huang@sunplus.com \
--cc=arnd@arndb.de \
--cc=derek.kiernan@xilinx.com \
--cc=devicetree@vger.kernel.org \
--cc=dragan.cvetic@xilinx.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=tonyhuang.sunplus@gmail.com \
--cc=wells.lu@sunplus.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®