Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <c89f9330-715f-4830-49fb-64c0b533ed39@linaro.org>
Date:   Sun, 27 Nov 2022 20:07:07 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.5.0
Subject: Re: [PATCH] arm64: dts: amlogic: align LED node names with dtschema
Content-Language: en-US
To:     Neil Armstrong <neil.armstrong@linaro.org>
Cc:     Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Kevin Hilman <khilman@baylibre.com>,
        Jerome Brunet <jbrunet@baylibre.com>,
        Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org
References: <20221125144141.477253-1-krzysztof.kozlowski@linaro.org>
 <3719828c-0ac5-34c9-a04b-251cbeb5f6ef@linaro.org>
 <a135f93a-235a-27b8-f649-69d62f6ebd30@linaro.org>
 <54f500cb-d7d0-8af1-eb58-99fbe71791a5@linaro.org>
 <CAB6niQ4808JEgdQEvRvaYqq8YmFCGo_YLnxXo+rjpSgxQxu+7g@mail.gmail.com>
 <615eb86b-a00a-56bc-ae1a-9f0e4e86f77c@linaro.org>
 <a3b7e1f0-16cb-4284-c740-5b5874612ea4@linaro.org>
From:   Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
In-Reply-To: <a3b7e1f0-16cb-4284-c740-5b5874612ea4@linaro.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 27/11/2022 14:33, Neil Armstrong wrote:
> 
> Le 27/11/2022 à 13:25, Krzysztof Kozlowski a écrit :
>> On 26/11/2022 17:03, Neil Armstrong wrote:
>>>>>> Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
>>>>> Aren't you maintainer of the platform? This means I should take care of
>>>>> this patch?
>>>> Ah, this will be picked up by Kevin or Jerome? So this tag means only
>>>> Reviewer's statement of oversight, right?
>>> Of course I'll pick it.
>>>
>> So that's confusing to see Review which does not mean review, but
>> maintainer pick...
> 
> Reviewed-by means reviewed, there's no confusion here, the significance
> is clearly documented on https://docs.kernel.org/process/submitting-patches.html
> 
>>
>> Reviewed-by means, I should take your tag and add to v2 or the
>> maintainer will add it when picking up.
> 
> Reviewed-by means reviewed, fully reviewed patches are ready to
> be picked unless a change is asked by a reviewer.
> 
> Decision to pick or not a patch does not depend on any tags, but
> is the sole decision of the maintainer adding it's Signed-off-by.

Then why giving review tag and not notifying about the pick, while the
pick actually happens? This is confusing. When the maintainer, which is
supposed to pick patch, gives reviewed it means always the same as
acked, so the patch is expected to go other way. Not through that
maintainer's tree.

If you just apply it, say you apply it.

Best regards,
Krzysztof