From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DE988C04FDE for ; Tue, 13 Dec 2022 09:31:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234567AbiLMJbq (ORCPT ); Tue, 13 Dec 2022 04:31:46 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58836 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234276AbiLMJbm (ORCPT ); Tue, 13 Dec 2022 04:31:42 -0500 Received: from mail-lf1-x132.google.com (mail-lf1-x132.google.com [IPv6:2a00:1450:4864:20::132]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F1DF8CF3 for ; Tue, 13 Dec 2022 01:31:40 -0800 (PST) Received: by mail-lf1-x132.google.com with SMTP id q6so3918335lfm.10 for ; Tue, 13 Dec 2022 01:31:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=eBNGkpSXOiqxD0QFw5sTJNcEwlVctRLV2sWFD35T7yM=; b=wgXxB7i9Myiz3fetBQR2uDgOzp8pixRPZY7JajZqv2ZHuw8Kvfs/kjuy2Bc75ziv6G g52YZfw4cCzgQkGCl1TJ3RNbIZsp14K9L/rv1Ztk70RnbeuzXbwESSNQ5pcRJrFPOLOa riEgRB/yPlM8Kc+5K9HnrO4AQdafEnkm4a2OChbhDyBnB7dENko89aPBrZ/LkDQex0YR ZPmip2txg/tQ1mJvow0qD6k8xHx79Ny0guy661fLdbcKB2umPv+JlY7AF1zX+tcDfJnx ij6jK3IinSGQpapR7FSZ7dSPJX5ynlX1dma59irSYrYmqTrhLRKUHfdsRlJwQqw2rNRY 8uWg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=eBNGkpSXOiqxD0QFw5sTJNcEwlVctRLV2sWFD35T7yM=; b=YX5y1L0u6ncGBv+eOzer/G8wVEiagly7xUamYv1PqsK1bTGfT/0Yp+cj3DY7n0L4dx spETp29oj1qizlamrxKl6yAtLiQHzbaeF/geLMn7ENCDdPcS1jhtwiFmvGP0FwJwmObl euz13JkN9ZXKXGTBJBpj39yeaCZJhcoCXdqV/GGItj5lOcFkfmloxKMij/5mmdB4v8jb TR6pXYTnilFn1DdiGVI2yK/+g7jAthC2S/a1B3tT89a2uU4uaAkSODdIRZWk9tWUrI7+ nEX+evEcBuayBTqB+RvZ5cn2tBeuEb+Czoe6JWialmKFVPVF7I4XjiA3OefKJOraLk4k 3QPw== X-Gm-Message-State: ANoB5pnBFxR7V62yamYg0zXt/Zv2rkPrSTH79W6Y/MpI6cTytn1kwQGP e5RfQPFegOsxu493x5hWuBIPDT94EOp47sgg X-Google-Smtp-Source: AA0mqf5FI/j9Le7pQBHJzExIQOzU6406ebgIjketJYTAuv2HASPjuYxFAMBnopyXLe0ptTyK/+cLNQ== X-Received: by 2002:a05:6512:3092:b0:4a4:68b7:dedd with SMTP id z18-20020a056512309200b004a468b7deddmr7261189lfd.57.1670923899326; Tue, 13 Dec 2022 01:31:39 -0800 (PST) Received: from [192.168.1.101] (abxh44.neoplus.adsl.tpnet.pl. [83.9.1.44]) by smtp.gmail.com with ESMTPSA id f22-20020a19dc56000000b004b4ea0f4e7fsm285654lfj.299.2022.12.13.01.31.38 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 13 Dec 2022 01:31:38 -0800 (PST) Message-ID: <48bda658-2312-809c-6dac-227b15f3ec40@linaro.org> Date: Tue, 13 Dec 2022 10:31:37 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.5.1 Subject: Re: [PATCH 07/12] ARM: dts: qcom: reverse compatibles to match bindings To: Krzysztof Kozlowski , Andy Gross , Bjorn Andersson , Rob Herring , Krzysztof Kozlowski Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20221212163532.142533-1-krzysztof.kozlowski@linaro.org> <20221212163532.142533-7-krzysztof.kozlowski@linaro.org> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <20221212163532.142533-7-krzysztof.kozlowski@linaro.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12.12.2022 17:35, Krzysztof Kozlowski wrote: > The most specific compatible should be upfront. > > Signed-off-by: Krzysztof Kozlowski > --- Reviewed-by: Konrad Dybcio Konrad > arch/arm/boot/dts/qcom-apq8064.dtsi | 4 ++-- > arch/arm/boot/dts/qcom-ipq4019.dtsi | 2 +- > arch/arm/boot/dts/qcom-ipq8064.dtsi | 4 ++-- > arch/arm/boot/dts/qcom-msm8960.dtsi | 4 ++-- > 4 files changed, 7 insertions(+), 7 deletions(-) > > diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi > index 0da9623ea084..79575d6b3a81 100644 > --- a/arch/arm/boot/dts/qcom-apq8064.dtsi > +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi > @@ -375,8 +375,8 @@ intc: interrupt-controller@2000000 { > }; > > timer@200a000 { > - compatible = "qcom,kpss-timer", > - "qcom,kpss-wdt-apq8064", "qcom,msm-timer"; > + compatible = "qcom,kpss-wdt-apq8064", "qcom,kpss-timer", > + "qcom,msm-timer"; > interrupts = <1 1 0x301>, > <1 2 0x301>, > <1 3 0x301>; > diff --git a/arch/arm/boot/dts/qcom-ipq4019.dtsi b/arch/arm/boot/dts/qcom-ipq4019.dtsi > index acb08dcf9442..a73c3a17b6a4 100644 > --- a/arch/arm/boot/dts/qcom-ipq4019.dtsi > +++ b/arch/arm/boot/dts/qcom-ipq4019.dtsi > @@ -400,7 +400,7 @@ blsp1_uart2: serial@78b0000 { > }; > > watchdog: watchdog@b017000 { > - compatible = "qcom,kpss-wdt", "qcom,kpss-wdt-ipq4019"; > + compatible = "qcom,kpss-wdt-ipq4019", "qcom,kpss-wdt"; > reg = <0xb017000 0x40>; > clocks = <&sleep_clk>; > timeout-sec = <10>; > diff --git a/arch/arm/boot/dts/qcom-ipq8064.dtsi b/arch/arm/boot/dts/qcom-ipq8064.dtsi > index 7e784b0995da..9daafe9de02a 100644 > --- a/arch/arm/boot/dts/qcom-ipq8064.dtsi > +++ b/arch/arm/boot/dts/qcom-ipq8064.dtsi > @@ -549,8 +549,8 @@ intc: interrupt-controller@2000000 { > }; > > timer@200a000 { > - compatible = "qcom,kpss-timer", > - "qcom,kpss-wdt-ipq8064", "qcom,msm-timer"; > + compatible = "qcom,kpss-wdt-ipq8064", "qcom,kpss-timer", > + "qcom,msm-timer"; > interrupts = IRQ_TYPE_EDGE_RISING)>, > diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi b/arch/arm/boot/dts/qcom-msm8960.dtsi > index 7debf9db7cb1..63c3c40fe9a2 100644 > --- a/arch/arm/boot/dts/qcom-msm8960.dtsi > +++ b/arch/arm/boot/dts/qcom-msm8960.dtsi > @@ -103,8 +103,8 @@ intc: interrupt-controller@2000000 { > }; > > timer@200a000 { > - compatible = "qcom,kpss-timer", > - "qcom,kpss-wdt-msm8960", "qcom,msm-timer"; > + compatible = "qcom,kpss-wdt-msm8960", "qcom,kpss-timer", > + "qcom,msm-timer"; > interrupts = , > , > ;