Re: Sashiko review feedback (was Re: [PATCH 2/3] soc: renesas: Add Renesas R-Car MFIS driver)

From: Wolfram Sang

Date: Tue Mar 31 2026 - 04:01:10 EST



> > What about Reported-by? The required closes by would then link to the
> > actual report:
> >
> > Reported-by: Sashiko:Gemini 3.1 Pro
> > Closes: https://sashiko.dev/#/patchset/20260319105947.6237-1-wsa%2Brenesas%40sang-engineering.com
> >
> > The drawback currently is that only the whole report for the patchset
> > can be linked. But probably Sashiko-reports could have some more HTML
> > tags to reference only the paragraph needed.
>
> And the Closes-tag can only be used when the issue is closed in full.
> You can still add a normal Link-tag.

We would need to update checkpatch for that. Doable. I prefer more HTML
tags in the Sashiko report, still, because it will be handy anyhow. The
reports can get quite large and it will make it easier to point people
to a specific item.

Attachment: signature.asc
Description: PGP signature