From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: "Hawkins, Nick" <nick.hawkins@hpe.com>, Rob Herring <robh@kernel.org>
Cc: "Verdun, Jean-Marie" <verdun@hpe.com>,
"joel@jms.id.au" <joel@jms.id.au>,
"arnd@arndb.de" <arnd@arndb.de>,
"openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org>,
Wim Van Sebroeck <wim@linux-watchdog.org>,
Guenter Roeck <linux@roeck-us.net>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
"linux-watchdog@vger.kernel.org" <linux-watchdog@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v5 06/11] dt-bindings: watchdog: Add HPE GXP Watchdog timer binding
Date: Tue, 26 Apr 2022 15:34:39 +0200 [thread overview]
Message-ID: <b9d68100-c6f3-4f58-fe42-63f4cdf1dc50@linaro.org> (raw)
In-Reply-To: <PH0PR84MB171845F606BC40AD653BEC8888FB9@PH0PR84MB1718.NAMPRD84.PROD.OUTLOOK.COM>
On 26/04/2022 15:21, Hawkins, Nick wrote:
>> How is this h/w controlled? I'm guessing it's part of the timer? If so, you don't need this node. A single node can implement multiple functions.
>
> It is associated with the timer because of the shared register set. Based on feedback from Krzysztof I need to create a child node for gxp-timer. I therefore will remove this file and move gxp-wdt to the hpe,gxp-timer.yaml as a child node.
I have impression my feedback was about mapping entire address space,
not few registers of watchdog:
https://lore.kernel.org/all/c6309ed8-6e74-67d3-304a-f5399d16cc37@canonical.com/
However later during talks it turned out that the address space is
heavily shared.
Nick,
BTW, do you see my comments in the email I linked above? This v5 makes
the same mistake. We talked about this already, so please make note of
comments you receive and implement them.
Best regards,
Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: "Hawkins, Nick" <nick.hawkins@hpe.com>, Rob Herring <robh@kernel.org>
Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-watchdog@vger.kernel.org" <linux-watchdog@vger.kernel.org>,
"arnd@arndb.de" <arnd@arndb.de>,
"Verdun, Jean-Marie" <verdun@hpe.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"joel@jms.id.au" <joel@jms.id.au>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Wim Van Sebroeck <wim@linux-watchdog.org>,
"openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org>,
Guenter Roeck <linux@roeck-us.net>
Subject: Re: [PATCH v5 06/11] dt-bindings: watchdog: Add HPE GXP Watchdog timer binding
Date: Tue, 26 Apr 2022 15:34:39 +0200 [thread overview]
Message-ID: <b9d68100-c6f3-4f58-fe42-63f4cdf1dc50@linaro.org> (raw)
In-Reply-To: <PH0PR84MB171845F606BC40AD653BEC8888FB9@PH0PR84MB1718.NAMPRD84.PROD.OUTLOOK.COM>
On 26/04/2022 15:21, Hawkins, Nick wrote:
>> How is this h/w controlled? I'm guessing it's part of the timer? If so, you don't need this node. A single node can implement multiple functions.
>
> It is associated with the timer because of the shared register set. Based on feedback from Krzysztof I need to create a child node for gxp-timer. I therefore will remove this file and move gxp-wdt to the hpe,gxp-timer.yaml as a child node.
I have impression my feedback was about mapping entire address space,
not few registers of watchdog:
https://lore.kernel.org/all/c6309ed8-6e74-67d3-304a-f5399d16cc37@canonical.com/
However later during talks it turned out that the address space is
heavily shared.
Nick,
BTW, do you see my comments in the email I linked above? This v5 makes
the same mistake. We talked about this already, so please make note of
comments you receive and implement them.
Best regards,
Krzysztof
next prev parent reply other threads:[~2022-04-26 13:34 UTC|newest]
Thread overview: 77+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-21 19:21 [PATCH v5 01/11] aach: arm: mach-hpe: Introduce the HPE GXP architecture nick.hawkins
2022-04-21 19:21 ` nick.hawkins
2022-04-21 19:21 ` [PATCH v5 01/11] archh: " nick.hawkins
2022-04-21 19:21 ` nick.hawkins
2022-04-22 13:20 ` Arnd Bergmann
2022-04-22 13:20 ` Arnd Bergmann
2022-04-22 13:20 ` Arnd Bergmann
2022-04-21 19:21 ` [PATCH v5 02/11] arch: arm: configs: multi_v7_defconfig nick.hawkins
2022-04-21 19:21 ` nick.hawkins
2022-04-23 11:06 ` Krzysztof Kozlowski
2022-04-23 11:06 ` Krzysztof Kozlowski
2022-04-29 20:34 ` Hawkins, Nick
2022-04-29 20:34 ` Hawkins, Nick
2022-04-30 11:40 ` Krzysztof Kozlowski
2022-04-30 11:40 ` Krzysztof Kozlowski
2022-04-23 11:08 ` Krzysztof Kozlowski
2022-04-23 11:08 ` Krzysztof Kozlowski
2022-04-21 19:21 ` [PATCH v5 03/11] drivers: wdt: Introduce HPE GXP SoC Watchdog nick.hawkins
2022-04-21 19:21 ` [PATCH v5 04/11] clocksource/drivers: Add HPE GXP timer nick.hawkins
2022-04-22 13:16 ` Arnd Bergmann
2022-04-22 13:16 ` Arnd Bergmann
2022-04-25 20:38 ` Linus Walleij
2022-04-25 20:38 ` Linus Walleij
2022-04-25 21:05 ` Jonathan Neuschäfer
2022-04-26 6:00 ` Arnd Bergmann
2022-04-26 6:00 ` Arnd Bergmann
2022-04-26 21:38 ` Rob Herring
2022-04-26 21:38 ` Rob Herring
2022-04-26 21:55 ` Arnd Bergmann
2022-04-26 21:55 ` Arnd Bergmann
2022-04-26 22:04 ` Rob Herring
2022-04-26 22:04 ` Rob Herring
2022-04-26 22:26 ` Arnd Bergmann
2022-04-26 22:26 ` Arnd Bergmann
2022-04-22 14:56 ` Thomas Gleixner
2022-04-21 19:21 ` [PATCH v5 05/11] dt-bindings: timer: Add HPE GXP Timer Binding nick.hawkins
2022-04-23 10:50 ` Krzysztof Kozlowski
2022-04-21 19:21 ` [PATCH v5 06/11] dt-bindings: watchdog: Add HPE GXP Watchdog timer binding nick.hawkins
2022-04-23 10:52 ` Krzysztof Kozlowski
2022-04-25 22:04 ` Rob Herring
2022-04-25 22:04 ` Rob Herring
2022-04-26 13:21 ` Hawkins, Nick
2022-04-26 13:34 ` Krzysztof Kozlowski [this message]
2022-04-26 13:34 ` Krzysztof Kozlowski
2022-04-26 13:52 ` Hawkins, Nick
2022-04-26 15:44 ` Krzysztof Kozlowski
2022-04-26 15:44 ` Krzysztof Kozlowski
2022-04-21 19:21 ` [PATCH v5 07/11] dt-bindings: arm: Add HPE GXP Binding nick.hawkins
2022-04-23 10:58 ` Krzysztof Kozlowski
2022-04-21 19:21 ` [PATCH v5 08/11] dt-bindings: usb: generic-ehci: Add HPE GXP ehci binding nick.hawkins
2022-04-23 10:52 ` Krzysztof Kozlowski
2022-04-21 19:21 ` [PATCH v5 09/11] dt-bindings: usb: generic-ohci: Add HPE GXP ohci binding nick.hawkins
2022-04-23 10:53 ` Krzysztof Kozlowski
2022-04-21 19:21 ` [PATCH v5 10/11] arch: arm: boot: dts: Introduce HPE GXP Device tree nick.hawkins
2022-04-21 19:21 ` nick.hawkins
2022-04-21 19:21 ` nick.hawkins
2022-04-22 13:06 ` Arnd Bergmann
2022-04-22 13:06 ` Arnd Bergmann
2022-04-22 13:06 ` Arnd Bergmann
2022-04-23 11:01 ` Krzysztof Kozlowski
2022-04-23 11:01 ` Krzysztof Kozlowski
2022-04-26 7:22 ` Krzysztof Kozlowski
2022-04-26 7:22 ` Krzysztof Kozlowski
2022-04-26 7:22 ` Krzysztof Kozlowski
2022-04-21 19:21 ` [PATCH v5 11/11] maintainers: Introduce HPE GXP Architecture nick.hawkins
2022-04-23 11:04 ` [PATCH v5 01/11] aach: arm: mach-hpe: Introduce the HPE GXP architecture Krzysztof Kozlowski
2022-04-23 11:04 ` Krzysztof Kozlowski
2022-04-25 15:00 ` Hawkins, Nick
2022-04-25 15:00 ` Hawkins, Nick
2022-04-26 8:25 ` Paul Menzel
2022-04-26 8:25 ` Paul Menzel
2022-04-26 8:25 ` Paul Menzel
2022-04-26 17:28 ` Hawkins, Nick
2022-04-26 17:28 ` Hawkins, Nick
2022-04-26 17:50 ` Paul Menzel
2022-04-26 17:50 ` Paul Menzel
2022-04-26 17:50 ` Paul Menzel
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=b9d68100-c6f3-4f58-fe42-63f4cdf1dc50@linaro.org \
--to=krzysztof.kozlowski@linaro.org \
--cc=arnd@arndb.de \
--cc=devicetree@vger.kernel.org \
--cc=joel@jms.id.au \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-watchdog@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=nick.hawkins@hpe.com \
--cc=openbmc@lists.ozlabs.org \
--cc=robh@kernel.org \
--cc=verdun@hpe.com \
--cc=wim@linux-watchdog.org \
/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.