On Fri, Mar 07, 2025 at 02:10:28AM +0900, Vincent Mailhol wrote:
On 06/03/2025 at 23:34, Lucas De Marchi wrote:
On Thu, Mar 06, 2025 at 08:29:52PM +0900, Vincent Mailhol via B4 Relay
wrote:
(...)
it seems we now have 1 inconsistency that we comment why
GENMASK_U128() is not available in asm, but we don't comment why
GENMASK_INPUT_CHECK() is not available there. Maybe move this comment on
top of GENMASK_INPUT_CHECK().
I will restore the comment in v6 and put it next to the asm definition,
c.f. my reply to Andy.
Anyway,
Reviewed-by: Lucas De Marchi <lucas.demarchi@xxxxxxxxx>
Is this only valid for the first patch or for the full series? If this
is for the full series, would you mind replying to the cover letter with
your review tag?
only for this patch. I'm the author of some of the patches and also add
my s-o-b in others. I'm not sure what b4 is going to do with those - it
would be weird if it added a r-b on my own patch.
Now I added r-b in some and comments in another.
thanks
Lucas De Marchi
thanks for picking up this series.
You are welcome!
Lucas De Marchi
(...)
Yours sincerely,
Vincent Mailhol
