Messages in this thread | | | Date | Fri, 26 Jun 2020 00:52:31 +0530 | From | Sai Prakash Ranjan <> | 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
| |