Re: [PATCH v1 1/2] dt-bindings: arm: qcom: Add Maili SoC compatible
Krzysztof Kozlowski <[email protected]>
| Newsgroups | org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
On 19/08/2026 09:02, Yijie Yang wrote: > > > On 8/19/2026 2:35 PM, Krzysztof Kozlowski wrote: >> On 19/08/2026 08:28, Yijie Yang wrote: >>> >>> >>> On 8/19/2026 2:09 PM, Krzysztof Kozlowski wrote: >>>> On 19/08/2026 07:42, Prasad Kumpatla wrote: >>>>> From: Yijie Yang <[email protected]> >>>>> >>>>> Add the qcom,maili SoC compatible string to the QCOM SoC bindings. >>>>> >>>>> Signed-off-by: Yijie Yang <[email protected]> >>>>> --- >>>>> Documentation/devicetree/bindings/arm/qcom.yaml | 3 +++ >>>>> 1 file changed, 3 insertions(+) >>>>> >>>>> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml >>>>> index ff8a35df7862..00d9a493fe76 100644 >>>>> --- a/Documentation/devicetree/bindings/arm/qcom.yaml >>>>> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml >>>>> @@ -175,6 +175,9 @@ properties: >>>>> - qcom,kaanapali-qrd >>>>> - const: qcom,kaanapali >>>>> >>>>> + - items: >>>>> + - const: qcom,maili >>>> >>>> Heh? This cannot go alone. >>> >>> Keep only the SoC compatible string and exclude the board for now, as >>> the latter is not currently used. >>> https://lore.kernel.org/all/20260622-brainy-rapid-tody-dd7c3a@quoll/ >> >> No, read my message there - this should be rejected! I even wrote: >> >> "But if ever this commit is used in argument "oh, compatible without >> user was accepted here by Krzysztof, so I can do the same", I will >> simply NAK such future contribution without reading the rest." >> >> So NAK because you did not even bother to read what I already stated! > > In patch 2/2, pd-mapper serves as the user. Is there any other reason to > reject this binding? I already wrote - this compatible cannot go alone. It makes no sense alone. Otherwise explain me what are you expressing here? Because commit msg did not help me to understand it - it just repeats the diff, which is pointless. So why are you doing this? Best regards, Krzysztof