Please don't add a Reviewed-by unless explicitly provided -

Acknowledged.

I'm not sure what rules apply for Yocto in general, but I suspect they are similar to
the kernel's [1]. In this case I certainly haven't reviewed the patch as
a whole, since I am familiar neither with the subtleties of Yocto files
nor with those of LLVM configuration :) Same applies to patch 8.

Your feedback has changed the shape of these patches and was a valid contribution,

but I shall remove your email in V3 :)

Small note in passing, there should be a space between ":" and the name
in tags.

Aye.

Pawel

On Thu, 12 Jan 2023 at 13:37, Kevin Brodsky <kevin.brodsky@arm.com> wrote:

On 09/01/2023 17:30, Pawel Zalewski wrote:
> |Reviewed-by:Kevin Brodsky <kevin.brodsky@arm.com>|

Please don't add a Reviewed-by unless explicitly provided - I'm not sure
what rules apply for Yocto in general, but I suspect they are similar to
the kernel's [1]. In this case I certainly haven't reviewed the patch as
a whole, since I am familiar neither with the subtleties of Yocto files
nor with those of LLVM configuration :) Same applies to patch 8.

Small note in passing, there should be a space between ":" and the name
in tags.

Kevin

[1]
https://docs.kernel.org/process/submitting-patches.html#using-reported-by-tested-by-reviewed-by-suggested-by-and-fixes