Re: [PATCH v4 0/3] Add watchdog for Mstar SoCs

From: Guenter Roeck <linux@roeck-us.net>
To: Romain Perier <romain.perier@gmail.com>
Cc: Wim Van Sebroeck <wim@linux-watchdog.org>,
	Rob Herring <robh+dt@kernel.org>, Daniel Palmer <daniel@0x0f.com>,
	Mohammed Billoo <mohammed.billoo@gmail.com>,
	linux-watchdog@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 0/3] Add watchdog for Mstar SoCs
Date: Sat, 12 Jun 2021 05:59:15 -0700	[thread overview]
Message-ID: <20210612125915.GA320112@roeck-us.net> (raw)
In-Reply-To: <20210611200801.52139-1-romain.perier@gmail.com>

On Fri, Jun 11, 2021 at 10:07:58PM +0200, Romain Perier wrote:
> This patches series adds a new driver for the watchdog found in the Mstar
> MSC313e SoCs and newer. It adds a basic watchdog driver, the
> corresponding devicetree bindings and its documentation.
> 
> This work has been co-developed with Daniel Palmer.
> 
> Changes since v3:
> - Rebased onto v5.13-rc5
> - Added Reviewed-by tags

This works better. I applied patches 1 & 2 to my watchdog-next branch for
hand-off to Wim. Note that the arm dts patch needs to be pushed through the
arm tree.

Thanks,
Guenter

> 
> Changes since v2:
> - Renamed the dt-bindings documentation to mstar,msc313e-wdt.yaml
> - Fixed the subject of the first commit
> - Dropped unused dt labels
> 
> Changes since v1:
> - Dropped "depends OF" from Kconfig (not required)
> - Dropped unneeded include files
> - Re-ordered include files
> - Compute timeout_max dynamically from the probe function to avoid
>   an overflow in _start().
> 
> Daniel Palmer (1):
>   watchdog: Add Mstar MSC313e WDT driver
> 
> Romain Perier (2):
>   dt-bindings: watchdog: Add Mstar MSC313e WDT devicetree bindings
>     documentation
>   ARM: dts: mstar: Add watchdog device_node definition
> 
>  .../bindings/watchdog/mstar,msc313e-wdt.yaml  |  40 +++++
>  MAINTAINERS                                   |   1 +
>  arch/arm/boot/dts/mstar-v7.dtsi               |  14 ++
>  drivers/watchdog/Kconfig                      |  12 ++
>  drivers/watchdog/Makefile                     |   1 +
>  drivers/watchdog/msc313e_wdt.c                | 166 ++++++++++++++++++
>  6 files changed, 234 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/watchdog/mstar,msc313e-wdt.yaml
>  create mode 100644 drivers/watchdog/msc313e_wdt.c
> 
> -- 
> 2.30.2
> 