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-te...