[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <12b3dce8-92c4-4084-9cc2-4e0d6432c6f3@linaro.org>
Date: Fri, 8 Dec 2023 21:28:52 +0100
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Kamal Dasu <kamal.dasu@...adcom.com>, ulf.hansson@...aro.org,
linux-kernel@...r.kernel.org, alcooperx@...il.com,
linux-arm-kernel@...ts.infradead.org, adrian.hunter@...el.com,
linux-mmc@...r.kernel.org, robh+dt@...nel.org,
krzysztof.kozlowski+dt@...aro.org, conor+dt@...nel.org,
devicetree@...r.kernel.org
Cc: f.fainelli@...il.com, bcm-kernel-feedback-list@...adcom.com,
Kamal Dasu <kdasu@...adcom.com>
Subject: Re: [PATCH 1/2] dt-bindings: mmc: brcm,sdhci-brcmstb: Add support for
74165b0
On 08/12/2023 21:21, Kamal Dasu wrote:
> From: Kamal Dasu <kdasu@...adcom.com>
>
> With newer sdio controller core used for 74165b0 we need to update
> the compatibility with "brcm,bcm74165b0-sdhci".
>
> Signed-off-by: Kamal Dasu <kdasu@...adcom.com>
> ---
> .../devicetree/bindings/mmc/brcm,sdhci-brcmstb.yaml | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/mmc/brcm,sdhci-brcmstb.yaml b/Documentation/devicetree/bindings/mmc/brcm,sdhci-brcmstb.yaml
> index c028039bc477..cec9ff063794 100644
> --- a/Documentation/devicetree/bindings/mmc/brcm,sdhci-brcmstb.yaml
> +++ b/Documentation/devicetree/bindings/mmc/brcm,sdhci-brcmstb.yaml
> @@ -13,6 +13,11 @@ maintainers:
> properties:
> compatible:
> oneOf:
> + - items:
> + - enum:
> + - brcm,bcm74165b0-sdhci
> + - const: brcm,bcm74165-sdhci
What is exactly the difference between bcm74165b0 and bcm74165? Your
driver does not use bcm74165, so I wonder what its purpose is.
Few days ago, for different patchset, I was asking "why", because the
motivation was not clear from the code. Here you said "we need to", but
I would argue: no you do not need to add bcm74165 if it means nothing,
thus this commit msg has similar problem. Does not answer why it is done
like this.
Best regards,
Krzysztof
Powered by blists - more mailing lists