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

[Sai Prakash Ranjan]

On 2020-06-25 21:30, Guenter Roeck wrote:
> 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@xxxxxxxxxxxxxx/
> > > > > v2:
> > > > > https://lore.kernel.org/lkml/cover.1580570160.git.saiprakash.ranjan@xxxxxxxxxxxxxx/
> > > > >
> > > > > 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 kept you in CC for all the revisions but missed adding watchdog list.
Will resend with the appropriate lists added.

> I also see
> no evidence for a Reviewed-by: tag from Rob or any other DT maintainer.

Not sure why you think I would add reviewed-by tag from Rob without him 
giving one.
But here's the evidence - 
https://lore.kernel.org/lkml/cover.1580570160.git.saiprakash.ranjan@xxxxxxxxxxxxxx/
This link is also given v3 cover letter.

Thanks,
Sai

--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a 
member
of Code Aurora Forum, hosted by The Linux Foundation