Re: [PATCH v3 0/4] support watchdog with longer timeout period

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, May 03, 2020 at 04:13:54PM +0000, Michael Shych wrote:
> 
> 
> > -----Original Message-----
> > From: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
> > Sent: Sunday, May 3, 2020 7:11 PM
> > To: Michael Shych <michaelsh@xxxxxxxxxxxx>
> > Cc: Guenter Roeck <linux@xxxxxxxxxxxx>; Wim Van Sebroeck <wim@linux-
> > watchdog.org>; Andy Shevchenko <andy@xxxxxxxxxxxxx>; Darren Hart
> > <dvhart@xxxxxxxxxxxxx>; linux-watchdog@xxxxxxxxxxxxxxx; Platform Driver
> > <platform-driver-x86@xxxxxxxxxxxxxxx>; Vadim Pasternak
> > <vadimp@xxxxxxxxxxxx>
> > Subject: Re: [PATCH v3 0/4] support watchdog with longer timeout period
> > 
> > On Sun, May 3, 2020 at 8:34 AM <michaelsh@xxxxxxxxxxxx> wrote:
> > >
> > > From: Michael Shych <michaelsh@xxxxxxxxxxxx>
> > >
> > > This patchset adds support of extended new watchdog type 3 of Mellanox
> > > Ethernet and Infiniband switch systems.
> > > This type of watchdog can have a timeout period longer than 255 or 32 sec.
> > > as it was before.
> > >
> > 
> > If you ignore my tags, why are you bothering about changes at all?
> 
> Sorry that I missed it.
> Should I resend this patchset?
> If yes, as still v3 or new one v4?
> 
If you resend as v3 but make changes, how do you expect the maintainers
to know which v3 is the correct one ?

Guenter

> > 
> > > Michael Shych (4):
> > >   platform_data/mlxreg: support new watchdog type with longer timeout
> > >     period
> > >   platform/x86: mlx-platform: support new watchdog type with longer
> > >     timeout
> > >   watchdog: mlx-wdt: support new watchdog type with longer timeout
> > >     period
> > >   docs: watchdog: mlx-wdt: Add description of new watchdog type 3
> > >
> > >  Documentation/watchdog/mlx-wdt.rst   |  10 ++++
> > >  drivers/platform/x86/mlx-platform.c  | 106
> > ++++++++++++++++++++++++++++++
> > >  drivers/watchdog/mlx_wdt.c           |  73 ++++++++++++++++++++----
> > >  include/linux/platform_data/mlxreg.h |   5 +-
> > >  4 files changed, 182 insertions(+), 12 deletions(-)
> > >
> > > --
> > > 2.11.0
> > >
> > 
> > 
> > --
> > With Best Regards,
> > Andy Shevchenko



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux