All of lore.kernel.org
 help / color / mirror / Atom feed
From: Boris Lysov <arzamas-16@mail.ee>
To: Mark Rutland <mark.rutland@arm.com>
Cc: Guenter Roeck <linux@roeck-us.net>,
	linux-kernel@vger.kernel.org, linux-watchdog@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: Missing ARM TWD Watchdog driver?
Date: Tue, 18 Jan 2022 04:25:12 +0300	[thread overview]
Message-ID: <20220118042512.360ee83a@pc> (raw)
In-Reply-To: <20220117170225.GB94025@C02TD0UTHF1T.local>

Hi Mark and Guenter,

On Mon, 17 Jan 2022 17:02:25 +0000,
Mark Rutland <mark.rutland@arm.com> wrote:

> On Mon, Jan 17, 2022 at 08:34:57AM -0800, Guenter Roeck wrote:
> > On 1/17/22 8:01 AM, Boris Lysov wrote:
> > > Hello everyone,
> > > 
> > > I stumbled upon the ARM TWD Watchdog driver which seems to be missing (or
> > > I'm just blind).
> > > 
> > > As per commit a33f5c380c4bd3fa5278d690421b72052456d9fe ("Merge tag
> > > 'xfs-5.17-merge-3' of git://git.kernel.org/pub/scm/fs/xfs/xfs-linux"),
> > > multiple device trees declare usage of arm-twd:
> > > 
> > > (...)
> > > 
> > > and it is documented in
> > > Documentation/devicetree/bindings/watchdog/arm,twd-wdt.yaml
> > > 
> > > However I could not find the driver itself. I tried running
> > > case-insensitive grep and ripgrep to no avail. Does this driver actually
> > > exist? Is it gone?
> > > 
> > 
> > I may be missing it, but I do not see any evidence that it ever existed.
> 
> Likewise, AFAICT we added the binding but never the driver.
> 
> Looking at the git history, that schema was introduced in commit:
> 
>   50e02e9a030a9ae3 ("dt-bindings: timer: arm,twd: Convert to json-schema")
> 
> ... and replaced Documentation/devicetree/bindings/timer/arm,twd.txt
> 
> Per:
> 
>   $ git log --follow Documentation/devicetree/bindings/timer/arm,twd.txt
> 
> That was introduced alongside a driver in commit:
> 
>   d8e0364364d333fe ("ARM: smp_twd: add device tree support")
> 
> ... which only introduced a driver for the timer, and not the watchdog,
> as the commit message notes.
> 
> Thanks.
> Mark.

Now I understand there's no ARM TWD Watchdog driver.
TYVM for your replies!

WARNING: multiple messages have this Message-ID (diff)
From: Boris Lysov <arzamas-16@mail.ee>
To: Mark Rutland <mark.rutland@arm.com>
Cc: Guenter Roeck <linux@roeck-us.net>,
	linux-kernel@vger.kernel.org, linux-watchdog@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: Missing ARM TWD Watchdog driver?
Date: Tue, 18 Jan 2022 04:25:12 +0300	[thread overview]
Message-ID: <20220118042512.360ee83a@pc> (raw)
In-Reply-To: <20220117170225.GB94025@C02TD0UTHF1T.local>

Hi Mark and Guenter,

On Mon, 17 Jan 2022 17:02:25 +0000,
Mark Rutland <mark.rutland@arm.com> wrote:

> On Mon, Jan 17, 2022 at 08:34:57AM -0800, Guenter Roeck wrote:
> > On 1/17/22 8:01 AM, Boris Lysov wrote:
> > > Hello everyone,
> > > 
> > > I stumbled upon the ARM TWD Watchdog driver which seems to be missing (or
> > > I'm just blind).
> > > 
> > > As per commit a33f5c380c4bd3fa5278d690421b72052456d9fe ("Merge tag
> > > 'xfs-5.17-merge-3' of git://git.kernel.org/pub/scm/fs/xfs/xfs-linux"),
> > > multiple device trees declare usage of arm-twd:
> > > 
> > > (...)
> > > 
> > > and it is documented in
> > > Documentation/devicetree/bindings/watchdog/arm,twd-wdt.yaml
> > > 
> > > However I could not find the driver itself. I tried running
> > > case-insensitive grep and ripgrep to no avail. Does this driver actually
> > > exist? Is it gone?
> > > 
> > 
> > I may be missing it, but I do not see any evidence that it ever existed.
> 
> Likewise, AFAICT we added the binding but never the driver.
> 
> Looking at the git history, that schema was introduced in commit:
> 
>   50e02e9a030a9ae3 ("dt-bindings: timer: arm,twd: Convert to json-schema")
> 
> ... and replaced Documentation/devicetree/bindings/timer/arm,twd.txt
> 
> Per:
> 
>   $ git log --follow Documentation/devicetree/bindings/timer/arm,twd.txt
> 
> That was introduced alongside a driver in commit:
> 
>   d8e0364364d333fe ("ARM: smp_twd: add device tree support")
> 
> ... which only introduced a driver for the timer, and not the watchdog,
> as the commit message notes.
> 
> Thanks.
> Mark.

Now I understand there's no ARM TWD Watchdog driver.
TYVM for your replies!

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2022-01-18  1:25 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-17 16:01 Missing ARM TWD Watchdog driver? Boris Lysov
2022-01-17 16:01 ` Boris Lysov
2022-01-17 16:34 ` Guenter Roeck
2022-01-17 16:34   ` Guenter Roeck
2022-01-17 17:02   ` Mark Rutland
2022-01-17 17:02     ` Mark Rutland
2022-01-18  1:25     ` Boris Lysov [this message]
2022-01-18  1:25       ` Boris Lysov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220118042512.360ee83a@pc \
    --to=arzamas-16@mail.ee \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-watchdog@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=mark.rutland@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.