Received-SPF: pass (google.com: domain of linux-kernel+bounces-101184-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <f515f9f4-b87c-465b-83c0-f4b7b5c47840@quicinc.com>
Date: Wed, 13 Mar 2024 15:55:38 +0800
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v3 1/2] dt-bindings: pinctrl: qcom: update compatible name
 for match with driver
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        <andersson@kernel.org>, <konrad.dybcio@linaro.org>,
        <linus.walleij@linaro.org>, <robh@kernel.org>,
        <krzysztof.kozlowski+dt@linaro.org>, <conor+dt@kernel.org>,
        <dmitry.baryshkov@linaro.org>
CC: <linux-arm-msm@vger.kernel.org>, <linux-gpio@vger.kernel.org>,
        <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <kernel@quicinc.com>
References: <20240312025807.26075-1-quic_tengfan@quicinc.com>
 <20240312025807.26075-2-quic_tengfan@quicinc.com>
 <0d768f17-22d9-448e-9253-8498b61bf71e@linaro.org>
 <31b02b76-88ff-42d7-a665-18d2661e028c@quicinc.com>
 <6a3b5c9d-6375-457f-83c9-269746c1612a@linaro.org>
 <ef237b3c-8613-4cd8-9391-e4a08d50cc6c@quicinc.com>
 <60a0e51f-dc0e-4bbf-8127-f987ac2aae71@linaro.org>
From: Tengfei Fan <quic_tengfan@quicinc.com>
In-Reply-To: <60a0e51f-dc0e-4bbf-8127-f987ac2aae71@linaro.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit


On 3/13/2024 3:23 PM, Krzysztof Kozlowski wrote:
> On 13/03/2024 02:30, Tengfei Fan wrote:
>>
>>
>> On 3/12/2024 6:55 PM, Krzysztof Kozlowski wrote:
>>> On 12/03/2024 08:47, Tengfei Fan wrote:
>>>>
>>>>
>>>> On 3/12/2024 3:41 PM, Krzysztof Kozlowski wrote:
>>>>> On 12/03/2024 03:58, Tengfei Fan wrote:
>>>>>> Use compatible name "qcom,sm4450-tlmm" instead of "qcom,sm4450-pinctrl"
>>>>>> to match the compatible name in sm4450 pinctrl driver.
>>>>>>
>>>>>> Fixes: 7bf8b78f86db ("dt-bindings: pinctrl: qcom: Add SM4450 pinctrl")
>>>>>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>>>>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>>>>> Signed-off-by: Tengfei Fan <quic_tengfan@quicinc.com>
>>>>>> ---
>>>>>>     Documentation/devicetree/bindings/pinctrl/qcom,sm4450-tlmm.yaml | 2 +-
>>>>>>     1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> Wasn't this applied?
>>>>
>>>> My test code base on tag: next-20240308, this patch is still not applied.
>>>>
>>>> In fact, the following dt binding check warning only can be got before
>>>> this patch is applied.
>>>>
>>>
>>> Please read all emails in the previous thread. You ignored two emails in
>>> the past and apparently one more recent.
>>
>> I don't know if you mean I ignored the email which related with "Patch
>> applied" tag from Linus Walleij. If so, the following is the reasion why
>> I still include this patch:
> 
> Yep, that's the one. Please do not send patches which were already
> applied. It causes unnecessary effort on reviewer and maintainer side.
> 
>>
>> I synced the latest upstream code on 03/12/2024, the latest tag is
>> next-20240308, this tag still doesn't include this patch[PATCH v3 1/2].
> 
> Happens, considering Linus applied it after 8th of March, I think.
> 
>>
>> Dt binding check still get warning if I only send [PATCH v3 2/2] patch
>> to upstream base on next-20240308. so I include this patch[PATCH v3 1/2]
> 
> If you send patch 1+2, dt_binding_check will have exactly the same
> result. I don't know about what sort of dt binding check you talk, but
> for all cases: you changed nothing by sending these two patches in that
> regard. Only noise on the lists.

The dt binding check failed which Rob Herring remind me in previous 
patch series as the following:

Documentation/devicetree/bindings/pinctrl/qcom,sm4450-tlmm.example.dtb:
/example-0/pinctrl@f100000: failed to match any schema with
compatible: ['qcom,sm4450-tlmm']

This failed is introduced by 
https://lore.kernel.org/linux-arm-msm/20231206020840.33228-2-quic_tengfan@quicinc.com/. 
Something got broken aroud -m flags for dtschema, so indeed no reports 
this unmatched compatibles warning when this patch was revriwed. We also 
have some discusstion in patch email.

The patch[PATCH v3 1/2] is made for fix this previous patch dt binding 
check failed. So dt binding check failed will disappear after this 
patch[PATCH v3 1/2] is applied.

> 
>> in patch series even if this patch have "Patch applied" tag.
>>
>> Looking forward to getting your advice if submitting patch series this
>> way is problematic.
> 
> Do not send patches which are known to be applied.

Yes, I will be careful not to resend the patch which have already been 
applied in the future work.

Do you think it is necessary to send another version patch series for 
remove this applied patch[PATCH v3 1/2] from patch series?

> 
> Best regards,
> Krzysztof
> 

-- 
Thx and BRs,
Tengfei Fan