Re: [PATCH] pinctrl: mediatek: mt8192: Fix compile warnings

[Linus Walleij <linus.walleij@xxxxxxxxxx>]

On Mon, Jun 27, 2022 at 4:45 PM Nícolas F. R. A. Prado
<nfraprado@xxxxxxxxxxxxx> wrote:
> On Mon, Jun 27, 2022 at 01:57:43AM +0200, Linus Walleij wrote:
> > After applying patches I get these warnings:
> >
> >   drivers/pinctrl/mediatek/pinctrl-mt8192.c:1302:56:
> >   warning: "/*" within comment [-Wcomment]
> >   drivers/pinctrl/mediatek/pinctrl-mt8192.c:1362:56:
> >   warning: "/*" within comment [-Wcomment]
> >
> > Something is wrong with the missing end-slashes. Add them.
> >
> > Cc: Guodong Liu <guodong.liu@xxxxxxxxxxxx>
> > Cc: Nícolas F. R. A. Prado <nfraprado@xxxxxxxxxxxxx>
> > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
>
> Hi,
>
> yes, this issue was noticed during review [1], and fixed on the following
> version [2], but since the commit before the correction was applied, this fix
> LGTM.
>
> Reviewed-by: Nícolas F. R. A. Prado <nfraprado@xxxxxxxxxxxxx>

Thanks, yeah the real problem is me lagging behind in reviewing
and applying patches and making mistakes like not seeing there
are newer patches in the front of the review queue...

Yours,
Linus Walleij