Re: [PATCH 0/1] Add RV1126 compatible watchdog string

From: karthikeyan
Date: Thu Oct 10 2024 - 11:35:52 EST




On 10/10/24 19:44, Guenter Roeck wrote:
On 10/9/24 23:14, Karthikeyan Krishnasamy wrote:
This patch is introduces a watchdog compatible string for rockchip's
RV1126. I have already send this patch[1] in the series[2] but somehow
missed watchdog maintainers and list. So resending this patch alone
again from the series[2].

Signed-off-by: Karthikeyan Krishnasamy <karthikeyan@xxxxxxxxxxx>
---

- Link to patch: [1]. https://lore.kernel.org/all/20240912142451.2952633-2-karthikeyan@xxxxxxxxxxx
- Link to series: [2]. https://lore.kernel.org/all/20240912142451.2952633-1-karthikeyan@xxxxxxxxxxx

Karthikeyan Krishnasamy (1):
   dt-bindings: watchdog: rockchip: Add rockchip,rv1126-wdt string

  Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml | 1 +
  1 file changed, 1 insertion(+)


I am curious: More and more people send introduction patches for individual
patches. I don't see any value in it, the intro patch is not available
in patchwork, and thus the context is missing when trying to review the
patch from there. This makes reviews much more difficult if one doesn't
reply directly to the patch. It doesn't make any sense to me, yet people
do it more and more.

Where is it suggested to send introduction patches for single-patch series,
and what is the rationale ?

Thanks,
Guenter


Hi,

I had missed watchdog maintainers in previous patch series, and received a tag for this patch from the one of the maintainer. So i thought, mention this make sense and added this info in introduction patch. That is the reason. Looks like i made a mistake.

Best Regards,
Karthikeyan