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 X-Spam-Level: X-Spam-Status: No, score=-6.5 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AC170C433E0 for ; Tue, 9 Jun 2020 10:28:11 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 76CCF207ED for ; Tue, 9 Jun 2020 10:28:11 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="UY9uWp8t"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=microchip.com header.i=@microchip.com header.b="A8ScVzQU" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 76CCF207ED Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=microchip.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date: In-Reply-To:Subject:To:From:References:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=KuVVZoHnC50w0pDaQca8X4Jx48xNtBwfnHkoLOFAxr0=; b=UY9uWp8tctBR43+FBWd94tPuGO 8Z1Fi49s4i82k0BTcVvAiofqaR1dPrnClViG0dJxZAt6h+sVM6WvB6b0OkiEHJvNcVl1qxG29AlZ4 au57JGiEukRXAKX+70jtEMVGIX4iI5Nj9sbWMrKHgUxJkXL0YOH6/aml+ZS5R4sUVnR67htb9sQpj ngR/Ts9LF7QQTB/eAcXhaaJ+2sHaXXrOGLrnEVvbKDAUr62NGAi08vmUCeWTAGduUb5SYTq0IGG3m kpRugBA1WF0SdhWmv3zz5fLIF0+3JZ/peOwvhk/MXiuRXnCBpQEOp9ZqDh65u2wuskg4VCoFF+k7K YitZ4xhQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jibUB-0006Tk-3z; Tue, 09 Jun 2020 10:28:03 +0000 Received: from esa1.microchip.iphmx.com ([68.232.147.91]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jibU7-0006S7-0e for linux-arm-kernel@lists.infradead.org; Tue, 09 Jun 2020 10:28:00 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1591698478; x=1623234478; h=references:from:to:cc:subject:in-reply-to:date: message-id:mime-version; bh=PFE08cVYhQUpY5zaFOQ/GJEuFcqAMgONaAxufYheBDA=; b=A8ScVzQUe4BOmcYATj2CPsJlB8JEdtPkZ3olplQmciZ3ihfKZq3mcVK8 L0P/JngbK6biawggVTDwRjA1UOK5++VnuX7FPKDdDUiY/qIKpy0s+fVz+ tqR/XH5+1ASwOHSrIddLP7Z52ez0DbdsJ/lzSlViunTJdfmXd3VnWtym1 y5s2qpvlsKK2p2/duKsNQf5DEea29Yro3u4LkjA7RRDd+zYpZKoX0KgYo t1WX0H9C6B+Bvi/RLQqxBhJzZJydSHQiS9IoiTvDgpCxmdIdA7Emf+rWa YIxfyLGFUVEUN7meZm+goIiCUxQxLk3smRAzt/n36vc08arNhKuEQ2T8h Q==; IronPort-SDR: DvBSgCKDbJXDL0IpdE3lYQtMLoRVAuznXxSpGbvfLg8x6HIPEduN5SM1VV00Ua71tASeuVdMLA revxWCd3KL115Zfb0tHJwI1qVdIb5VAxn2zPdFvbOQfwyyItP+MJuGObwaiwXAZqA5xbO/YEMq kW20geL10MMNERGdm15KCxtrfFKMYYxba2pKsYohZSKIgfqRIIClQS3oHqHfExPwyt6tuTAqvr qO+a2MQpDv0dzAIfRNQpc5tUDVomRdNALzMDo0PYrwfsFFOIoCMQ/yUdGuSbtGBn/cUKA9nu1w 8SU= X-IronPort-AV: E=Sophos;i="5.73,491,1583218800"; d="scan'208";a="82797722" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa1.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 09 Jun 2020 03:27:48 -0700 Received: from chn-vm-ex01.mchp-main.com (10.10.85.143) by chn-vm-ex02.mchp-main.com (10.10.85.144) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1847.3; Tue, 9 Jun 2020 03:27:48 -0700 Received: from soft-dev15.microsemi.net.microchip.com (10.10.115.15) by chn-vm-ex01.mchp-main.com (10.10.85.143) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5 via Frontend Transport; Tue, 9 Jun 2020 03:27:45 -0700 References: <20200513140031.25633-1-lars.povlsen@microchip.com> <20200513140031.25633-5-lars.povlsen@microchip.com> <20200602194925.qbevttzz3ghvmd2d@mobilestation> From: Lars Povlsen To: Serge Semin Subject: Re: [PATCH 04/10] dt-bindings: spi: Add bindings for spi-dw-mchp In-Reply-To: <20200602194925.qbevttzz3ghvmd2d@mobilestation> Date: Tue, 9 Jun 2020 12:27:44 +0200 Message-ID: <87img0ilen.fsf@soft-dev15.microsemi.net> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200609_032759_065010_BE36493E X-CRM114-Status: GOOD ( 18.40 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , List-Id: Cc: devicetree@vger.kernel.org, Alexandre Belloni , Mark Brown , linux-kernel@vger.kernel.org, Serge Semin , linux-spi@vger.kernel.org, SoC Team , Rob Herring , linux-arm-kernel@lists.infradead.org, Microchip Linux Driver Support , Lars Povlsen Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Serge Semin writes: > On Wed, May 13, 2020 at 04:00:25PM +0200, Lars Povlsen wrote: >> This add DT bindings for the Microsemi/Microchip SPI controller used >> in various SoC's. It describes the "mscc,ocelot-spi" and >> "mscc,jaguar2-spi" bindings. > > As I see it, there is no need in this patch at all. Current DT binding file > describes the MSCC version of the DW APB SSI IP pretty well. You can add an > example to the DT schema though with "mscc,ocelot-spi" or "mscc,jaguar2-spi" > compatible string and additional registers range. > Fine by me. I just had the understanding that a YAML binding had to be given for a new driver. I will add the bindings to the existing YAML with proper guards. ---Lars > -Sergey > >> >> Reviewed-by: Alexandre Belloni >> Signed-off-by: Lars Povlsen >> --- >> .../bindings/spi/mscc,ocelot-spi.yaml | 60 +++++++++++++++++++ >> .../bindings/spi/snps,dw-apb-ssi.txt | 7 +-- >> MAINTAINERS | 1 + >> 3 files changed, 63 insertions(+), 5 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/spi/mscc,ocelot-spi.yaml >> >> diff --git a/Documentation/devicetree/bindings/spi/mscc,ocelot-spi.yaml b/Documentation/devicetree/bindings/spi/mscc,ocelot-spi.yaml >> new file mode 100644 >> index 0000000000000..a3ac0fa576553 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/spi/mscc,ocelot-spi.yaml >> @@ -0,0 +1,60 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: "http://devicetree.org/schemas/spi/mscc,ocelot-spi.yaml#" >> +$schema: "http://devicetree.org/meta-schemas/core.yaml#" >> + >> +title: Microsemi Vcore-III SPI Communication Controller >> + >> +maintainers: >> + - Alexandre Belloni >> + - Lars Povlsen >> + >> +allOf: >> + - $ref: "spi-controller.yaml#" >> + >> +description: | >> + The Microsemi Vcore-III SPI controller is a general purpose SPI >> + controller based upon the Designware SPI controller. It uses an 8 >> + byte rx/tx fifo. >> + >> +properties: >> + compatible: >> + enum: >> + - mscc,ocelot-spi >> + - mscc,jaguar2-spi >> + >> + interrupts: >> + maxItems: 1 >> + >> + reg: >> + minItems: 2 >> + items: >> + - description: Designware SPI registers >> + - description: CS override registers >> + >> + clocks: >> + maxItems: 1 >> + >> + reg-io-width: >> + description: | >> + The I/O register width (in bytes) implemented by this device. >> + items: >> + enum: [ 2, 4 ] >> + maxItems: 1 >> + >> +required: >> + - compatible >> + - reg >> + - clocks >> + >> +examples: >> + - | >> + spi0: spi@101000 { >> + compatible = "mscc,ocelot-spi"; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + reg = <0x101000 0x100>, <0x3c 0x18>; >> + interrupts = <9>; >> + clocks = <&ahb_clk>; >> + }; >> diff --git a/Documentation/devicetree/bindings/spi/snps,dw-apb-ssi.txt b/Documentation/devicetree/bindings/spi/snps,dw-apb-ssi.txt >> index 3ed08ee9feba4..5e1849be7bae5 100644 >> --- a/Documentation/devicetree/bindings/spi/snps,dw-apb-ssi.txt >> +++ b/Documentation/devicetree/bindings/spi/snps,dw-apb-ssi.txt >> @@ -1,10 +1,8 @@ >> Synopsys DesignWare AMBA 2.0 Synchronous Serial Interface. >> >> Required properties: >> -- compatible : "snps,dw-apb-ssi" or "mscc,-spi", where soc is "ocelot" or >> - "jaguar2", or "amazon,alpine-dw-apb-ssi" >> -- reg : The register base for the controller. For "mscc,-spi", a second >> - register set is required (named ICPU_CFG:SPI_MST) >> +- compatible : "snps,dw-apb-ssi" or "amazon,alpine-dw-apb-ssi" >> +- reg : The register base for the controller. >> - interrupts : One interrupt, used by the controller. >> - #address-cells : <1>, as required by generic SPI binding. >> - #size-cells : <0>, also as required by generic SPI binding. >> @@ -38,4 +36,3 @@ Example: >> cs-gpios = <&gpio0 13 0>, >> <&gpio0 14 0>; >> }; >> - >> diff --git a/MAINTAINERS b/MAINTAINERS >> index 1db598723a1d8..6472240b8391b 100644 >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -11231,6 +11231,7 @@ L: linux-mips@vger.kernel.org >> S: Supported >> F: Documentation/devicetree/bindings/mips/mscc.txt >> F: Documentation/devicetree/bindings/power/reset/ocelot-reset.txt >> +F: Documentation/devicetree/bindings/spi/mscc,ocelot-spi.yaml >> F: arch/mips/boot/dts/mscc/ >> F: arch/mips/configs/generic/board-ocelot.config >> F: arch/mips/generic/board-ocelot.c >> -- >> 2.26.2 >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Lars Povlsen, Microchip _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel