* [PATCH 0/3] Use MFD for Dallas/Maxim DS1374 driver series
@ 2020-06-22 10:03 Johnson CH Chen (陳昭勳)
2020-06-22 11:25 ` Alexandre Belloni
0 siblings, 1 reply; 3+ messages in thread
From: Johnson CH Chen (陳昭勳) @ 2020-06-22 10:03 UTC (permalink / raw)
To: linux-kernel
Cc: linux-rtc, linux-watchdog, Alessandro Zummo, Alexandre Belloni,
Wim Van Sebroeck, linux, Lee Jones
Hello all,
This patch set uses MFD structure for DS1374 so that RTC and Watchdog
functions can be separately. Therefore, we can add more Watchdog
subfunctions here.
A DS1374 MFD core driver supports the I2C communication to RTC and
Watchdog devices.
1. Add DS1374 MFD core driver with I2C bus.
2. Let DS1374 RTC driver has RTC and Alarm functions only.
3. Add DS1374 Watchdog driver.
Thanks,
Johnson
Johnson Chen (3):
mfd: ds1374: Introduce Dallas/Maxim DS1374 MFD core driver
rtc: rtc-ds1374: Move out Watchdog function and I2C client
watchdog: ds1374_wdt: Introduce Dallas/Maxim DS1374 Watchdog driver
drivers/mfd/Kconfig | 11 +
drivers/mfd/Makefile | 2 +
drivers/mfd/ds1374.c | 101 ++++++++
drivers/rtc/Kconfig | 9 +-
drivers/rtc/rtc-ds1374.c | 458 ++++++----------------------------
drivers/watchdog/Kconfig | 11 +
drivers/watchdog/Makefile | 1 +
drivers/watchdog/ds1374_wdt.c | 330 ++++++++++++++++++++++++
8 files changed, 532 insertions(+), 391 deletions(-)
create mode 100644 drivers/mfd/ds1374.c
create mode 100644 drivers/watchdog/ds1374_wdt.c
--
2.20.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 0/3] Use MFD for Dallas/Maxim DS1374 driver series
2020-06-22 10:03 [PATCH 0/3] Use MFD for Dallas/Maxim DS1374 driver series Johnson CH Chen (陳昭勳)
@ 2020-06-22 11:25 ` Alexandre Belloni
2020-06-23 3:19 ` Johnson CH Chen (陳昭勳)
0 siblings, 1 reply; 3+ messages in thread
From: Alexandre Belloni @ 2020-06-22 11:25 UTC (permalink / raw)
To: Johnson CH Chen (陳昭勳)
Cc: linux-kernel, linux-rtc, linux-watchdog, Alessandro Zummo,
Wim Van Sebroeck, linux, Lee Jones
Hi,
On 22/06/2020 10:03:25+0000, Johnson CH Chen (陳昭勳) wrote:
> Hello all,
>
> This patch set uses MFD structure for DS1374 so that RTC and Watchdog
> functions can be separately. Therefore, we can add more Watchdog
> subfunctions here.
>
> A DS1374 MFD core driver supports the I2C communication to RTC and
> Watchdog devices.
>
> 1. Add DS1374 MFD core driver with I2C bus.
> 2. Let DS1374 RTC driver has RTC and Alarm functions only.
> 3. Add DS1374 Watchdog driver.
>
For reference, this was the last attempt:
https://lore.kernel.org/linux-rtc/20170718092245.tc5oosbbb6lzvqpy@dell/
The main issue I see with your series is that there is no way to select
which of the rtc or the watchdog driver has to be used as IIRC each
function is mutually exclusive. I think you should work on the DT
bindings, addressing the few remaining comments.
--
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [PATCH 0/3] Use MFD for Dallas/Maxim DS1374 driver series
2020-06-22 11:25 ` Alexandre Belloni
@ 2020-06-23 3:19 ` Johnson CH Chen (陳昭勳)
0 siblings, 0 replies; 3+ messages in thread
From: Johnson CH Chen (陳昭勳) @ 2020-06-23 3:19 UTC (permalink / raw)
To: Alexandre Belloni
Cc: linux-kernel, linux-rtc, linux-watchdog, Alessandro Zummo,
Wim Van Sebroeck, linux, Lee Jones
Hi,
>
> Hi,
>
> On 22/06/2020 10:03:25+0000, Johnson CH Chen (陳昭勳) wrote:
> > Hello all,
> >
> > This patch set uses MFD structure for DS1374 so that RTC and Watchdog
> > functions can be separately. Therefore, we can add more Watchdog
> > subfunctions here.
> >
> > A DS1374 MFD core driver supports the I2C communication to RTC and
> > Watchdog devices.
> >
> > 1. Add DS1374 MFD core driver with I2C bus.
> > 2. Let DS1374 RTC driver has RTC and Alarm functions only.
> > 3. Add DS1374 Watchdog driver.
> >
>
> For reference, this was the last attempt:
>
> https://lore.kernel.org/linux-rtc/20170718092245.tc5oosbbb6lzvqpy@dell/
>
> The main issue I see with your series is that there is no way to select which of
> the rtc or the watchdog driver has to be used as IIRC each function is mutually
> exclusive. I think you should work on the DT bindings, addressing the few
> remaining comments.
>
Thanks for your review and good suggestion!
Usage of rtc and watchdog should be defined in DT binding if we want to keep
them separate.
I'll work towards these later.
> --
> Alexandre Belloni, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
Best regards,
Johnson
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-06-23 3:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-22 10:03 [PATCH 0/3] Use MFD for Dallas/Maxim DS1374 driver series Johnson CH Chen (陳昭勳)
2020-06-22 11:25 ` Alexandre Belloni
2020-06-23 3:19 ` Johnson CH Chen (陳昭勳)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).