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 C58B4C7EE25 for ; Mon, 12 Jun 2023 11:39:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232409AbjFLLjP (ORCPT ); Mon, 12 Jun 2023 07:39:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52666 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231178AbjFLLiQ (ORCPT ); Mon, 12 Jun 2023 07:38:16 -0400 Received: from mail.loongson.cn (mail.loongson.cn [114.242.206.163]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id AB3F07EEE; Mon, 12 Jun 2023 04:29:35 -0700 (PDT) Received: from loongson.cn (unknown [10.20.42.35]) by gateway (Coremail) with SMTP id _____8AxGuqdAYdkiLkDAA--.8129S3; Mon, 12 Jun 2023 19:29:33 +0800 (CST) Received: from [10.20.42.35] (unknown [10.20.42.35]) by localhost.localdomain (Coremail) with SMTP id AQAAf8BxC8qbAYdkquoVAA--.54094S3; Mon, 12 Jun 2023 19:29:32 +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> <9c94397d-1e31-02fa-bdbe-af888c72eac4@loongson.cn> <657f8d19-de83-8be6-4a9d-5f13b1df7383@linaro.org> <84ccf4cc-072d-adbf-0361-95ceae13f333@linaro.org> <5d060cac-ff28-60e9-98a8-f2bd4d378455@loongson.cn> <4e30870d-86e2-8536-8e0d-aab4ce5027d2@linaro.org> <0c532e09-4821-5e07-92e6-7bc3cd79869e@loongson.cn> From: zhuyinbo Message-ID: <9fec9cfa-0686-91d8-cba4-91ea67243b47@loongson.cn> Date: Mon, 12 Jun 2023 19:29:31 +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: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit X-CM-TRANSID: AQAAf8BxC8qbAYdkquoVAA--.54094S3 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/12 下午4:16, Krzysztof Kozlowski 写道: >>>>>>>> 在 2023/6/8 下午7:45, Krzysztof Kozlowski 写道: >>>>>>>>> On 08/06/2023 13:42, zhuyinbo wrote: >>>>>>>>>> --- 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 >>>>>>>>> >>>>>>>>> Aren't they compatible? >>>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> Are you saying that the spi driver is compatible with 2k0500 ? >>>>>>> >>>>>>> Didn't you say this through 11 previous revisions? >>>>>> >>>>>> >>>>>> Yes, did I understand your meaning incorrectly ? >>>>> >>>>> If they are compatible, then they are not part of one enum. They could >>>>> not be as this would easily fail in testing of your DTS. >>>>> >>>> >>>> >>>> The "loongson,ls2k0500-spi" wasn't a compatible in previous version and >>>> I will add "loongson,ls2k0500-spi" as a compatible in spi driver and >>>> added it as a part of the one enum in dt-binding. >>> >>> No, because you claimed - if I understood correctly - that they are >>> compatible. Don't add fake entries to the driver. >>> >> >> >> I'm a bit confused, and I just need to add 'loongson,ls2k0500-spi' as >> one enum in dt-bindings, but driver don't add this entry ? > > Compatibility is expressed with a list: > https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#compatible > so it cannot be just one enum, but "items". There are hundreds of > examples including example-schema. Is it a description like the following? properties: compatible: - enum: - - loongson,ls2k1000-spi + oneOf: + - enum: + - loongson,ls2k1000-spi + - items: + - enum: + - loongson,ls2k1000-spi + - const: loongson,ls2k1000-spi + - items: + - enum: + - loongson,ls2k0500-spi + - const: loongson,ls2k1000-spi reg: Thanks, Yinbo