2021-06-22 15:21:18

by Christine Zhu

[permalink] [raw]
Subject: [v2,0/3] watchdog: mt8195: add wdt support

Supports MT8195 watchdog device.
Supports MT8195 watchdog reset-controller feature.

Change since v1:
-Remove the unneeded tag in [v1,1/3] [v1,2/3] [v1,3/3]
-Add of_device_id of MT8195 in [v1,3/3]
-use more proper prefixes, such as "dt-bindings: mediatek: mt8195:"
-provide more information in the cover letter

christine.zhu (3):
dt-binding: mediatek: mt8195: update mtk-wdt document
dt-binding: reset: mt8195: add toprgu reset-controller head file
watchdog: mediatek: mt8195: add wdt support

.../devicetree/bindings/watchdog/mtk-wdt.txt | 2 +-
drivers/watchdog/mtk_wdt.c | 6 +++++
.../dt-bindings/reset-controller/mt8195-resets.h | 29 ++++++++++++++++++++++
3 files changed, 36 insertions(+), 1 deletion(-)
create mode 100644 include/dt-bindings/reset-controller/mt8195-resets.h



2021-06-23 03:18:35

by Tzung-Bi Shih

[permalink] [raw]
Subject: Re: [v2,0/3] watchdog: mt8195: add wdt support

On Tue, Jun 22, 2021 at 11:18 PM Christine Zhu
<[email protected]> wrote:
> christine.zhu (3):
> dt-binding: mediatek: mt8195: update mtk-wdt document
> dt-binding: reset: mt8195: add toprgu reset-controller head file
See [1] as a reference, "dt-bindings" is preferred.

> watchdog: mediatek: mt8195: add wdt support
If the commit messages in the 3 patches are sentences, they should end
with a period (i.e. ".").

[1]: https://www.kernel.org/doc/html/latest/devicetree/bindings/submitting-patches.html