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
