From: Alexandre Belloni <alexandre.belloni@bootlin.com>
To: Paul Fertser <fercerpav@gmail.com>
Cc: Ivan Mikhaylov <i.mikhaylov@yadro.com>,
Alessandro Zummo <a.zummo@towertech.it>,
openbmc@lists.ozlabs.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/2] rtc: pch-rtc: add Intel Series PCH built-in read-only RTC
Date: Sun, 15 Aug 2021 01:22:15 +0200 [thread overview]
Message-ID: <YRhQJ4kdyu1Xs1Rb@piout.net> (raw)
In-Reply-To: <20210814224215.GX15173@home.paul.comp>
On 15/08/2021 01:42:15+0300, Paul Fertser wrote:
> On Tue, Aug 10, 2021 at 06:44:34PM +0300, Ivan Mikhaylov wrote:
> > Add RTC driver with dt binding tree document. Also this driver adds one sysfs
> > attribute for host power control which I think is odd for RTC driver.
> > Need I cut it off and use I2C_SLAVE_FORCE? I2C_SLAVE_FORCE is not good
> > way too from my point of view. Is there any better approach?
>
> Reading the C620 datasheet I see this interface also allows other
> commands (wake up, watchdog feeding, reboot etc.) and reading statuses
> (e.g Intruder Detect, POWER_OK_BAD).
>
> I think if there's any plan to use anything other but RTC via this
> interface then the driver should be registered as an MFD.
>
This is not the current thinking, if everything is integrated, then
there is no issue registering a watchdog from the RTC driver. I'll let
you check with Lee...
However, I'm not sure what is the correct interface for poweroff/reboot
control.
--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
next prev parent reply other threads:[~2021-08-14 23:22 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-10 15:44 [PATCH 0/2] rtc: pch-rtc: add Intel Series PCH built-in read-only RTC Ivan Mikhaylov
2021-08-10 15:44 ` Ivan Mikhaylov
2021-08-10 15:44 ` [PATCH 1/2] rtc: pch-rtc: add RTC driver for Intel Series PCH Ivan Mikhaylov
2021-08-10 15:44 ` Ivan Mikhaylov
2021-08-14 22:52 ` Paul Fertser
2021-08-20 12:34 ` Ivan Mikhaylov
2021-08-20 12:34 ` Ivan Mikhaylov
2021-09-25 22:24 ` Alexandre Belloni
2021-09-25 22:24 ` Alexandre Belloni
2021-08-10 15:44 ` [PATCH 2/2] dt-bindings: rtc: provide RTC PCH device tree binding doc Ivan Mikhaylov
2021-08-10 15:44 ` Ivan Mikhaylov
2021-08-14 22:42 ` [PATCH 0/2] rtc: pch-rtc: add Intel Series PCH built-in read-only RTC Paul Fertser
2021-08-14 23:22 ` Alexandre Belloni [this message]
2021-08-17 18:04 ` Milton Miller II
2021-08-17 18:04 ` Milton Miller II
2021-08-17 20:05 ` Alexandre Belloni
2021-08-17 20:05 ` Alexandre Belloni
2021-08-30 11:56 ` Ivan Mikhaylov
2021-08-30 11:56 ` Ivan Mikhaylov
2021-09-14 23:52 ` Ivan Mikhaylov
2021-09-14 23:52 ` Ivan Mikhaylov
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=YRhQJ4kdyu1Xs1Rb@piout.net \
--to=alexandre.belloni@bootlin.com \
--cc=a.zummo@towertech.it \
--cc=fercerpav@gmail.com \
--cc=i.mikhaylov@yadro.com \
--cc=linux-kernel@vger.kernel.org \
--cc=openbmc@lists.ozlabs.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.