Re: [PATCHv3 0/2] Convert QCOM watchdog timer bindings to YAML

From: Guenter Roeck <linux@roeck-us.net>
To: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>,
	Andy Gross <agross@kernel.org>,
	Douglas Anderson <dianders@chromium.org>,
	Stephen Boyd <swboyd@chromium.org>,
	Matthias Kaehlcke <mka@chromium.org>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	linux-arm-msm-owner@vger.kernel.org,
	Rob Herring <robh+dt@kernel.org>,
	devicetree@vger.kernel.org
Subject: Re: [PATCHv3 0/2] Convert QCOM watchdog timer bindings to YAML
Date: Thu, 25 Jun 2020 09:00:42 -0700	[thread overview]
Message-ID: <20200625160042.GC149301@roeck-us.net> (raw)
In-Reply-To: <ce4c2b44cb15af12b04c09f1786a6c1a@codeaurora.org>

On Mon, Jun 22, 2020 at 11:50:52AM +0530, Sai Prakash Ranjan wrote:
> On 2020-06-21 13:03, Bjorn Andersson wrote:
> > On Tue 16 Jun 23:56 PDT 2020, Sai Prakash Ranjan wrote:
> > 
> > > Hi Bjorn,
> > > 
> > 
> > Hi Sai,
> > 
> > > On 2020-02-12 03:54, Sai Prakash Ranjan wrote:
> > > > This series converts QCOM watchdog timer bindings to YAML. Also
> > > > it adds the missing SoC-specific compatible for QCS404, SC7180,
> > > > SDM845 and SM8150 SoCs.
> > > >
> > > > v1:
> > > > https://lore.kernel.org/lkml/cover.1576211720.git.saiprakash.ranjan@codeaurora.org/
> > > > v2:
> > > > https://lore.kernel.org/lkml/cover.1580570160.git.saiprakash.ranjan@codeaurora.org/
> > > >
> > > > Changes since v2:
> > > >  * Add missing compatibles to enum.
> > > >
> > > > Changes since v1:
> > > >  As per Rob's suggestion:
> > > >   * Replaced oneOf+const with enum.
> > > >   * Removed timeout-sec and included watchdog.yaml.
> > > >   * Removed repeated use of const:qcom,kpss-wdt and made use of enum.
> > > >
> > > > Sai Prakash Ranjan (2):
> > > >   dt-bindings: watchdog: Convert QCOM watchdog timer bindings to YAML
> > > >   dt-bindings: watchdog: Add compatible for QCS404, SC7180, SDM845,
> > > >     SM8150
> > > >
> > > >  .../devicetree/bindings/watchdog/qcom-wdt.txt | 28 -----------
> > > >  .../bindings/watchdog/qcom-wdt.yaml           | 48 +++++++++++++++++++
> > > >  2 files changed, 48 insertions(+), 28 deletions(-)
> > > >  delete mode 100644
> > > > Documentation/devicetree/bindings/watchdog/qcom-wdt.txt
> > > >  create mode 100644
> > > > Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
> > > 
> > > 
> > > Gentle ping!
> > > 
> > 
> > This should better go through the watchdog tree, so I believe Guenter
> > would be the one to pick this up.
> > 
> 
> Ah right, then a gentle ping for Guenter.
> 
I don't think the watchdog mailing list has been copied on this series,
meaning I don't have a copy that I could apply if I wanted to. I also see
no evidence for a Reviewed-by: tag from Rob or any other DT maintainer.

Guenter