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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <3e3cf81452e33c9385900177d0630a11@codeaurora.org>
Date:   Fri, 26 Jun 2020 00:52:31 +0530
From:   Sai Prakash Ranjan <saiprakash.ranjan@...eaurora.org>
To:     Guenter Roeck <linux@...ck-us.net>
Cc:     Bjorn Andersson <bjorn.andersson@...aro.org>,
        Andy Gross <agross@...nel.org>,
        Douglas Anderson <dianders@...omium.org>,
        Stephen Boyd <swboyd@...omium.org>,
        Matthias Kaehlcke <mka@...omium.org>,
        linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
        linux-arm-msm@...r.kernel.org, linux-arm-msm-owner@...r.kernel.org,
        Rob Herring <robh+dt@...nel.org>, devicetree@...r.kernel.org,
        Guenter Roeck <groeck7@...il.com>
Subject: Re: [PATCHv3 0/2] Convert QCOM watchdog timer bindings to YAML

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@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 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@codeaurora.org/
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