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 79C41C7EE45 for ; Thu, 8 Jun 2023 11:43:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235476AbjFHLnV (ORCPT ); Thu, 8 Jun 2023 07:43:21 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57642 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236372AbjFHLnQ (ORCPT ); Thu, 8 Jun 2023 07:43:16 -0400 Received: from mail.loongson.cn (mail.loongson.cn [114.242.206.163]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id B0FCB2712; Thu, 8 Jun 2023 04:42:53 -0700 (PDT) Received: from loongson.cn (unknown [10.20.42.35]) by gateway (Coremail) with SMTP id _____8BxL_C8voFkyIIAAA--.1963S3; Thu, 08 Jun 2023 19:42:52 +0800 (CST) Received: from [10.20.42.35] (unknown [10.20.42.35]) by localhost.localdomain (Coremail) with SMTP id AQAAf8DxluS5voFkFHQHAA--.23621S3; Thu, 08 Jun 2023 19:42:49 +0800 (CST) Subject: Re: [PATCH v12 1/2] spi: add loongson spi bindings To: Krzysztof Kozlowski , Mark Brown , Rob Herring , Krzysztof Kozlowski , linux-spi@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Jianmin Lv , wanghongliang@loongson.cn, Liu Peibao , loongson-kernel@lists.loongnix.cn, zhuyinbo@loongson.cn References: <20230608072819.25930-1-zhuyinbo@loongson.cn> <20230608072819.25930-2-zhuyinbo@loongson.cn> <6ebed84c-2b42-c981-7b3f-e71cc88e4c2c@linaro.org> <4bf747c4-b767-b20c-e00f-724b50f44edb@loongson.cn> <6bfc2a22-6901-0858-7b90-bc4c52c66810@linaro.org> <11ca2b90-544d-18c2-fb15-7909ca60507f@linaro.org> From: zhuyinbo Message-ID: Date: Thu, 8 Jun 2023 19:42:49 +0800 User-Agent: Mozilla/5.0 (X11; Linux loongarch64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 In-Reply-To: <11ca2b90-544d-18c2-fb15-7909ca60507f@linaro.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit X-CM-TRANSID: AQAAf8DxluS5voFkFHQHAA--.23621S3 X-CM-SenderInfo: 52kx5xhqerqz5rrqw2lrqou0/ X-Coremail-Antispam: 1Uk129KBjDUn29KB7ZKAUJUUUUU529EdanIXcx71UUUUU7KY7 ZEXasCq-sGcSsGvfJ3UbIjqfuFe4nvWSU5nxnvy29KBjDU0xBIdaVrnUUvcSsGvfC2Kfnx nUUI43ZEXa7xR_UUUUUUUUU== Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 在 2023/6/8 下午6:02, Krzysztof Kozlowski 写道: > On 08/06/2023 12:00, zhuyinbo wrote: >> >> >> 在 2023/6/8 下午4:53, Krzysztof Kozlowski 写道: >>> On 08/06/2023 10:39, zhuyinbo wrote: >>>>>> >>>>>> Signed-off-by: Yinbo Zhu >>>>>> Reviewed-by: Krzysztof Kozlowski >>>>>> --- >>>>>> .../bindings/spi/loongson,ls2k-spi.yaml | 41 +++++++++++++++++++ >>>>>> MAINTAINERS | 6 +++ >>>>>> 2 files changed, 47 insertions(+) >>>>>> create mode 100644 Documentation/devicetree/bindings/spi/loongson,ls2k-spi.yaml >>>>>> >>>>>> diff --git a/Documentation/devicetree/bindings/spi/loongson,ls2k-spi.yaml b/Documentation/devicetree/bindings/spi/loongson,ls2k-spi.yaml >>>>>> new file mode 100644 >>>>>> index 000000000000..423ee851edd5 >>>>>> --- /dev/null >>>>>> +++ b/Documentation/devicetree/bindings/spi/loongson,ls2k-spi.yaml >>>>> >>>>> Filename based on compatible. >>>> >>>> >>>> There will be more ls2k series SoC spi device in the future thus I still >>>> use "loongson,ls2k-spi.yaml" for cover it. >>> >>> Add them now. >> >> >> The 2k0500 doesn't support CCF and not use CCF to gain clock and We >> internally tend to prioritize supporting 2k1000. > > Don't you refer now to drivers? Because how hardware can not support > clocks if it has them? How CCF is anyhow related to hardware? The CCF (common clock framework) driver only affects the clock parameter pass method and isn't related to clock hardware. and if dts pass a "clock-frequency" that not need a clock driver but if dts pass a "clocks" that need a clock driver. Currently, only 2k1000 has implemented a clock driver. > >> >>> >>>> >>>>> >>>>>> @@ -0,0 +1,41 @@ >>>>>> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >>>>>> +%YAML 1.2 >>>>>> +--- >>>>>> +$id: http://devicetree.org/schemas/spi/loongson,ls2k-spi.yaml# >>>>>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>>>>> + >>>>>> +title: Loongson SPI controller >>>>>> + >>>>>> +maintainers: >>>>>> + - Yinbo Zhu >>>>>> + >>>>>> +allOf: >>>>>> + - $ref: /schemas/spi/spi-controller.yaml# >>>>>> + >>>>>> +properties: >>>>>> + compatible: >>>>>> + enum: >>>>>> + - loongson,ls2k1000-spi >>>>> >>>>> No compatibles for other devices? Didn't we have big discussion about this? >>>>> >>>>> https://elixir.bootlin.com/linux/v6.1-rc1/source/Documentation/devicetree/bindings/writing-bindings.rst#L42 >>>> >>>> >>>> There are other ls2k SPI devices compatible, such as, >>>> "loongson,ls2k0500-spi", "loongson,ls2k2000-spi" but currently I plan to >>>> add ls2k1000 spi device first, Other ls2k SoC spi device adaptation may >>>> require some additional work and I will add it later. >>> >>> Previously you claimed this serves entire family, so I don't understand >>> why you need to fix something. Why previously it was working for entire >>> family but now it does not? >> >> >> It can work was for ls2k1000 and ls2k0500 and it specifically refers to >> spi driver. but 2k0500 doesn't implementing a clock driver and doesn't > > We do not discuss here drivers, but bindings. Whatever your drivers are > not supporting, matters less. > >> use CCF to gain clock but can use "clock-frequency". Is it necessary to >> obtain a clock based on CCF? If it's necessary, then it seems that it >> can only added 2k1000 spi first. > > Not related to bindings... I may understand that what you said, and the dt-bindings only cover hardware and not involve the drivers. if so, I will add following: --- a/Documentation/devicetree/bindings/spi/loongson,ls2k-spi.yaml +++ b/Documentation/devicetree/bindings/spi/loongson,ls2k-spi.yaml @@ -16,6 +16,7 @@ properties: compatible: enum: - loongson,ls2k1000-spi + - loongson,ls2k0500-spi Thanks, Yinbo