On 15/11/2022 12:31, Alex Elder wrote: > Add a new enumerated value to those defined for the qcom,gsi-loader > property. If the qcom,gsi-loader is "skip", the GSI firmware will > already be loaded, so neither the AP nor modem is required to load > GSI firmware. > > Signed-off-by: Alex Elder <elder@xxxxxxxxxx> > --- This is a friendly reminder during the review process. It looks like you received a tag and forgot to add it. If you do not know the process, here is a short explanation: Please add Acked-by/Reviewed-by/Tested-by tags when posting new versions. However, there's no need to repost patches *only* to add the tags. The upstream maintainer will do that for acks received on the version they apply. https://elixir.bootlin.com/linux/v5.17/source/Documentation/process/submitting-patches.rst#L540 If a tag was not added on purpose, please state why and what changed. Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> Best regards, Krzysztof