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 0F81DCCA47B for ; Mon, 11 Jul 2022 11:26:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231288AbiGKL0y (ORCPT ); Mon, 11 Jul 2022 07:26:54 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39880 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231273AbiGKL0L (ORCPT ); Mon, 11 Jul 2022 07:26:11 -0400 Received: from mail-lf1-x12d.google.com (mail-lf1-x12d.google.com [IPv6:2a00:1450:4864:20::12d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2D0B92198 for ; Mon, 11 Jul 2022 04:02:35 -0700 (PDT) Received: by mail-lf1-x12d.google.com with SMTP id d12so8034948lfq.12 for ; Mon, 11 Jul 2022 04:02:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=MeyCZN/nvsiRNNCAAR15uWNXyKxL+fyysaeRMmXtoJA=; b=D4OVU2SdZDl1a2Jmvol1Su2acwSKKhbqYZKZn6/Qc8eT2iND/KO9t9DapzIOJtxYXe Me3ljHAhTzWcyA62SHeecPkZ9BDGif3nIQh/4L6wxXSsOfYRKcaerfZkctH+lPcPcaXR 9nwslGSNuJR55nrpcs0QsF89vgcW8SmISLwf7h2vLLq4lkAVjwTUIVYO0/uBy/LsND6c fUnyCOa9fgewXhbnW8EdRaMnWofxJvAWbnpmaDz4VV5KTLCEbsVuj6MgdDHsj0ja4Cm4 7iMTtMjKPL9Xg15Ww0kzLHvpVQi9YC6P0O0itH4F7zJK4x/PaxvCYROPFLopO6UyHl1u Htcw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=MeyCZN/nvsiRNNCAAR15uWNXyKxL+fyysaeRMmXtoJA=; b=7vzWIVe+f+qKRJzP0LEkgSr8FiMjxbeDtoO5BjA1KlX/5QLx31Dg5UeJyyePiH980g q6UsD9Jl2fjLcYNf83/o7QPkRZrA7ildumc8krksjY6A6BVmqx/e0q40gi7SwweCdiko MwncksUyw3D6D3YiwwIROxvCRWMXM1/uXq5gZXcjUV965TX835pDRRdTAjKsAlk5em6b g9KnlIjj4MoywM93THdgX79Ad2ZfSJUVWZmmdMT/enRlQhgaMk4jZtlUnTYE82TZNWtk O3LR5boIWzenbXA6w5eyYyCb5vW3CRmy6EgdMtlbDN3fZW8uz2G1rP5Jic8dadJ73aT7 d4Kw== X-Gm-Message-State: AJIora+Habb6qwk7bfnwJkLjyew8w5vs+R5MnaFt1OOUMP2v3BmZWodB xP+OgZipjKG/irJHco+OmnogQQ== X-Google-Smtp-Source: AGRyM1vhnNj6aFdiTr/noru4YDiDuSvAcJcPUSJPfEwIMQ9AAASbaicPsvFT+S0lPieRGdaNAEiPLw== X-Received: by 2002:a05:6512:3b20:b0:47f:8ca1:bd7 with SMTP id f32-20020a0565123b2000b0047f8ca10bd7mr10686215lfv.105.1657537353492; Mon, 11 Jul 2022 04:02:33 -0700 (PDT) Received: from [10.0.0.8] (fwa5cab-55.bb.online.no. [88.92.171.55]) by smtp.gmail.com with ESMTPSA id u10-20020ac258ca000000b00478a311d399sm1489609lfo.0.2022.07.11.04.02.31 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 11 Jul 2022 04:02:32 -0700 (PDT) Message-ID: Date: Mon, 11 Jul 2022 13:02:30 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH] dt-bindings: mmc: Add compatible for MediaTek MT8188 Content-Language: en-US To: Johnson Wang , ulf.hansson@linaro.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org Cc: linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, Project_Global_Chrome_Upstream_Group@mediatek.com References: <20220707054710.1396-1-johnson.wang@mediatek.com> <41e17a56-5cb0-8e90-c7ae-a7a56de986d0@linaro.org> <3216ec92952c7d722feaeb76986e0a6a54340646.camel@mediatek.com> From: Krzysztof Kozlowski In-Reply-To: <3216ec92952c7d722feaeb76986e0a6a54340646.camel@mediatek.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 07/07/2022 09:30, Johnson Wang wrote: > On Thu, 2022-07-07 at 08:46 +0200, Krzysztof Kozlowski wrote: >> On 07/07/2022 07:47, Johnson Wang wrote: >>> This commit adds dt-binding documentation of mmc for MediaTek >>> MT8188 SoC >>> platform. >>> >>> Signed-off-by: Johnson Wang >>> --- >>> Documentation/devicetree/bindings/mmc/mtk-sd.yaml | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/mmc/mtk-sd.yaml >>> b/Documentation/devicetree/bindings/mmc/mtk-sd.yaml >>> index 2a2e9fa8c188..3fbf33ad4f7c 100644 >>> --- a/Documentation/devicetree/bindings/mmc/mtk-sd.yaml >>> +++ b/Documentation/devicetree/bindings/mmc/mtk-sd.yaml >>> @@ -32,6 +32,9 @@ properties: >>> - items: >>> - const: mediatek,mt8186-mmc >>> - const: mediatek,mt8183-mmc >>> + - items: >>> + - const: mediatek,mt8188-mmc >> >> You duplicate quite a lot. Use enum. >> >> Best regards, >> Krzysztof > > Hi Krzysztof, > > Thanks for you suggestion. > > I will use 'enum' to place this compatible strings. > > Just like this: > - items: > - enum: > - mediatek,mt8186-mmc > - mediatek,mt8188-mmc > - mediatek,mt8192-mmc > - mediatek,mt8195-mmc > - const: mediatek,mt8183-mmc > > Moreover, it seems that missing an "items:" between oneOf and enum in > the compatible property. > Is my understanding wrong? I did not propose to remove items, so where are they missing? Not sure what you are asking about... Best regards, Krzysztof 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id ED593C433EF for ; Mon, 11 Jul 2022 11:03:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=y3phx6PInloGodQPXp3C0XN4JwYONbcwn/1zdB9oxa4=; b=mXLbZzZ+yOWOwI vCR6Y627luTU6UbN3FYT8+O5gXN/1fIR00gNn6u1OEJByHs4JCnRcnX3Ktk9bd6XGL/ZKOEoGUUEX 87RvJYoxO+uwiEXbDDR2s/9Dk0/NzJjbcfVMLiE6MAPlklZezea6Uz7YSgVYNrK2PwNKfCHq8Kcb1 Z1xKIq0TqRADCG7EL4HZVvPCdsQlpZxouCyGj41ue+Lys8retqz8PrVYPhDHLR4TG5qXs0eL5zf3n rCGruXT4DRJqQRrnoybyCFbQHjtPrztIn8uHhl4IrKKSzA9RdNB/XEwSyFhnRqP+el/y/2nSHjZE5 RpwaH/wzmH6SO7pap6XQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oArBX-000uWp-3y; Mon, 11 Jul 2022 11:02:39 +0000 Received: from mail-lf1-x12c.google.com ([2a00:1450:4864:20::12c]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oArBT-000uVO-6E for linux-arm-kernel@lists.infradead.org; Mon, 11 Jul 2022 11:02:36 +0000 Received: by mail-lf1-x12c.google.com with SMTP id t1so4517547lft.8 for ; Mon, 11 Jul 2022 04:02:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=MeyCZN/nvsiRNNCAAR15uWNXyKxL+fyysaeRMmXtoJA=; b=D4OVU2SdZDl1a2Jmvol1Su2acwSKKhbqYZKZn6/Qc8eT2iND/KO9t9DapzIOJtxYXe Me3ljHAhTzWcyA62SHeecPkZ9BDGif3nIQh/4L6wxXSsOfYRKcaerfZkctH+lPcPcaXR 9nwslGSNuJR55nrpcs0QsF89vgcW8SmISLwf7h2vLLq4lkAVjwTUIVYO0/uBy/LsND6c fUnyCOa9fgewXhbnW8EdRaMnWofxJvAWbnpmaDz4VV5KTLCEbsVuj6MgdDHsj0ja4Cm4 7iMTtMjKPL9Xg15Ww0kzLHvpVQi9YC6P0O0itH4F7zJK4x/PaxvCYROPFLopO6UyHl1u Htcw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=MeyCZN/nvsiRNNCAAR15uWNXyKxL+fyysaeRMmXtoJA=; b=Ic9AwJpeO/UfXl6x0kqNbMrZzoNv/Zo1aXf/nKMwTvEs2xjGc/w1BM1wEBIPYjQklS o7ANGYu8uxPnjeIpGTsHs71+liuNXeRyHrFYFePdOn3zo1g/UWoMiHNjk1SxQhi8q77e FPqclx62Ya9kHgyY6yRmUIbsL0t0bXfQriMjjkl3zyG2k7PM3lxc9VkjiWL21uYrD5DA JlZoiLqOPtPqYqvUa1yMU78Qveg+kkmcP+yA3gQ3zmfq8v6YKPlfo9AWt1dMNvFONyDp jLzSy5EJmQzcUitj0Pzro32JAoRsnSoEphmpK1cieYKKKiub5FnC+93nfWt9muUVjF7G lZHw== X-Gm-Message-State: AJIora9KbW4WzEE6Adpk8yAiMJb23eVMmI/F9aZZ+wdNUEjLQdKgifvL VmeWx1wUTnsCudAN/I61/Wpijg== X-Google-Smtp-Source: AGRyM1vhnNj6aFdiTr/noru4YDiDuSvAcJcPUSJPfEwIMQ9AAASbaicPsvFT+S0lPieRGdaNAEiPLw== X-Received: by 2002:a05:6512:3b20:b0:47f:8ca1:bd7 with SMTP id f32-20020a0565123b2000b0047f8ca10bd7mr10686215lfv.105.1657537353492; Mon, 11 Jul 2022 04:02:33 -0700 (PDT) Received: from [10.0.0.8] (fwa5cab-55.bb.online.no. [88.92.171.55]) by smtp.gmail.com with ESMTPSA id u10-20020ac258ca000000b00478a311d399sm1489609lfo.0.2022.07.11.04.02.31 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 11 Jul 2022 04:02:32 -0700 (PDT) Message-ID: Date: Mon, 11 Jul 2022 13:02:30 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH] dt-bindings: mmc: Add compatible for MediaTek MT8188 Content-Language: en-US To: Johnson Wang , ulf.hansson@linaro.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org Cc: linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, Project_Global_Chrome_Upstream_Group@mediatek.com References: <20220707054710.1396-1-johnson.wang@mediatek.com> <41e17a56-5cb0-8e90-c7ae-a7a56de986d0@linaro.org> <3216ec92952c7d722feaeb76986e0a6a54340646.camel@mediatek.com> From: Krzysztof Kozlowski In-Reply-To: <3216ec92952c7d722feaeb76986e0a6a54340646.camel@mediatek.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220711_040235_242417_AA7FDFF7 X-CRM114-Status: GOOD ( 17.75 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 07/07/2022 09:30, Johnson Wang wrote: > On Thu, 2022-07-07 at 08:46 +0200, Krzysztof Kozlowski wrote: >> On 07/07/2022 07:47, Johnson Wang wrote: >>> This commit adds dt-binding documentation of mmc for MediaTek >>> MT8188 SoC >>> platform. >>> >>> Signed-off-by: Johnson Wang >>> --- >>> Documentation/devicetree/bindings/mmc/mtk-sd.yaml | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/mmc/mtk-sd.yaml >>> b/Documentation/devicetree/bindings/mmc/mtk-sd.yaml >>> index 2a2e9fa8c188..3fbf33ad4f7c 100644 >>> --- a/Documentation/devicetree/bindings/mmc/mtk-sd.yaml >>> +++ b/Documentation/devicetree/bindings/mmc/mtk-sd.yaml >>> @@ -32,6 +32,9 @@ properties: >>> - items: >>> - const: mediatek,mt8186-mmc >>> - const: mediatek,mt8183-mmc >>> + - items: >>> + - const: mediatek,mt8188-mmc >> >> You duplicate quite a lot. Use enum. >> >> Best regards, >> Krzysztof > > Hi Krzysztof, > > Thanks for you suggestion. > > I will use 'enum' to place this compatible strings. > > Just like this: > - items: > - enum: > - mediatek,mt8186-mmc > - mediatek,mt8188-mmc > - mediatek,mt8192-mmc > - mediatek,mt8195-mmc > - const: mediatek,mt8183-mmc > > Moreover, it seems that missing an "items:" between oneOf and enum in > the compatible property. > Is my understanding wrong? I did not propose to remove items, so where are they missing? Not sure what you are asking about... Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel