Re: [v5,0/4] watchdog: mt8192: add wdt support

On Thu, Oct 01, 2020 at 04:23:02PM +0200, Matthias Brugger wrote:

Hi Crystal,

It seems you forgot to send the email to one of the maintainers, Wim.
Please make sure you add all the maintainers from get_maintainers.pl when
you send a series.

Regards,
Matthias

On 29/09/2020 05:20, Crystal Guo wrote:

quoted

v5 changes:
fix typos on:
https://patchwork.kernel.org/patch/11697493/

v4 changes:
revise commit messages.

v3 changes:
https://patchwork.kernel.org/patch/11692731/
https://patchwork.kernel.org/patch/11692767/
https://patchwork.kernel.org/patch/11692729/
https://patchwork.kernel.org/patch/11692771/
https://patchwork.kernel.org/patch/11692733/

This is less than helpful. It doesn't tell me anything. It expects me to 
go back to the earlier versions, download them, and run a diff, to figure
out what changed. That means the patch or patch series ends at the bottom
of my pile of patches to review. Which, as it happens, is quite deep.

I will review this and similar patches without change log after (and only
after) I have reviewed all other patches in my queue.

Guenter

quoted

Crystal Guo (4):
   dt-binding: mediatek: watchdog: fix the description of compatible
   dt-binding: mediatek: mt8192: update mtk-wdt document
   dt-binding: mt8192: add toprgu reset-controller head file
   watchdog: mt8192: add wdt support

  .../devicetree/bindings/watchdog/mtk-wdt.txt       |  5 ++--
  drivers/watchdog/mtk_wdt.c                         |  6 +++++
  .../dt-bindings/reset-controller/mt8192-resets.h   | 30 ++++++++++++++++++++++
  3 files changed, 39 insertions(+), 2 deletions(-)
  create mode 100644 include/dt-bindings/reset-controller/mt8192-resets.h

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