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

[karthikeyan]

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