Re: [PATCH 0/2] rtc: pcf2127: only use watchdog when explicitly available

[Guenter Roeck]

On Mon, Oct 26, 2020 at 09:48:11PM +0100, Alexandre Belloni wrote:
> On 26/10/2020 07:23:26+0000, Qiang Zhao wrote:
> > Any update for this patchset?
> > 
> 
> The whole point would be to get the DT and the watchdog maintainers
> agree on the property name. Once done, the driver implementation is
> trivial and will get applied.
> 

DT maintainers make that decision.

Guenter

> > Best Regards
> > Qiang Zhao
> > 
> > > -----Original Message-----
> > > From: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>
> > > Sent: 2020年9月24日 18:53
> > > To: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxx>; Qiang Zhao
> > > <qiang.zhao@xxxxxxx>; Bruno Thomsen <bruno.thomsen@xxxxxxxxx>
> > > Cc: linux-rtc@xxxxxxxxxxxxxxx; a.zummo@xxxxxxxxxxxx;
> > > linux-watchdog@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx;
> > > linux-kernel@xxxxxxxxxxxxxxx; robh+dt@xxxxxxxxxx; kernel@xxxxxxxxxxxxxx;
> > > Wim Van Sebroeck <wim@xxxxxxxxxxxxxxxxxx>; Guenter Roeck
> > > <linux@xxxxxxxxxxxx>
> > > Subject: [PATCH 0/2] rtc: pcf2127: only use watchdog when explicitly available
> > > 
> > > Hello,
> > > 
> > > now that there are two people stumbling over the pcf2127 driver providing a
> > > non-functional watchdog device, here comes an RFC patch to address this.
> > > 
> > > Note this is only compile tested and dt-documentation is still missing.
> > > Still send this series because the cleanup is nice independent of this discussion
> > > and to have something to argue about.
> > > 
> > > Does someone can offer a better name than "has-watchdog", is there a scheme
> > > that could be used already that I'm not aware of?
> > > 
> > > Best regards
> > > Uwe
> > > 
> > > Uwe Kleine-König (2):
> > >   rtc: pcf2127: move watchdog initialisation to a separate function
> > >   [RFC] rtc: pcf2127: only use watchdog when explicitly available
> > > 
> > >  drivers/rtc/rtc-pcf2127.c | 57 ++++++++++++++++++++++-----------------
> > >  1 file changed, 32 insertions(+), 25 deletions(-)
> > > 
> > > --
> > > 2.28.0
> > 
> 
> -- 
> Alexandre Belloni, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com