Linux-RTC Archive on lore.kernel.org
 help / color / Atom feed
* [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, back to index

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 (陳昭勳)

Linux-RTC Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-rtc/0 linux-rtc/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-rtc linux-rtc/ https://lore.kernel.org/linux-rtc \
		linux-rtc@vger.kernel.org
	public-inbox-index linux-rtc

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-rtc


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git