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 2BE05C433F5 for ; Thu, 28 Apr 2022 17:27:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345839AbiD1Ra2 (ORCPT ); Thu, 28 Apr 2022 13:30:28 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35052 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1350428AbiD1Ra2 (ORCPT ); Thu, 28 Apr 2022 13:30:28 -0400 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 699A43B000; Thu, 28 Apr 2022 10:27:12 -0700 (PDT) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 16D841474; Thu, 28 Apr 2022 10:27:12 -0700 (PDT) Received: from donnerap.cambridge.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id C58143F774; Thu, 28 Apr 2022 10:27:10 -0700 (PDT) Date: Thu, 28 Apr 2022 18:27:08 +0100 From: Andre Przywara To: Krzysztof Kozlowski Cc: Rob Herring , Krzysztof Kozlowski , Liviu Dudau , Robin Murphy , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Dmitry Torokhov , linux-input@vger.kernel.org Subject: Re: [PATCH 05/11] dt-bindings: serio: add Arm PL050 DT schema Message-ID: <20220428182708.0c1065f8@donnerap.cambridge.arm.com> In-Reply-To: <55574426-df20-cdd7-c97d-092726981a42@linaro.org> References: <20220427112528.4097815-1-andre.przywara@arm.com> <20220427112528.4097815-6-andre.przywara@arm.com> <55574426-df20-cdd7-c97d-092726981a42@linaro.org> Organization: ARM X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.32; aarch64-unknown-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Thu, 28 Apr 2022 09:07:53 +0200 Krzysztof Kozlowski wrote: Hi Krzysztof, many thanks for having a look and your comments (here and on the other patches). I will fix them and send a new version ASAP. One thing below: > On 27/04/2022 13:25, Andre Przywara wrote: > > The Arm PL050 "Keyboard/Mouse Interface" is an Arm system IP providing a > > PS/2 compatible serial interface. > > > > Add a simple DT schema binding, based on the TRM[1], the existing DTs and > > the Linux driver. > > > > [1] https://developer.arm.com/documentation/ddi0143/latest > > > > Signed-off-by: Andre Przywara > > --- > > .../devicetree/bindings/serio/amba-pl050.yaml | 67 +++++++++++++++++++ > > 1 file changed, 67 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/serio/amba-pl050.yaml > > > > diff --git a/Documentation/devicetree/bindings/serio/amba-pl050.yaml b/Documentation/devicetree/bindings/serio/amba-pl050.yaml > > new file mode 100644 > > index 0000000000000..9732a84550098 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/serio/amba-pl050.yaml > > Filename: vendor,device > > > @@ -0,0 +1,67 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/serio/amba-pl050.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Arm Ltd. PrimeCell PL050 PS/2 Keyboard/Mouse Interface > > + > > +maintainers: > > + - Andre Przywara > > + > > +description: > > + The Arm PrimeCell PS2 Keyboard/Mouse Interface (KMI) is an AMBA compliant > > + peripheral that can be used to implement a keyboard or mouse interface that > > + is IBM PS2 or AT compatible. > > + > > +# We need a select here so we don't match all nodes with 'arm,primecell' > > +select: > > + properties: > > + compatible: > > + contains: > > + const: arm,pl050 > > + required: > > + - compatible > > + > > +properties: > > + compatible: > > + items: > > + - const: arm,pl050 > > + - const: arm,primecell > > + > > + reg: > > + maxItems: 1 > > + > > + interrupts: > > + maxItems: 1 > > + > > + clocks: > > + items: > > + - description: KMI reference clock, used to generate the bus timing > > + - description: APB register access clock > > + > > + clock-names: > > + items: > > + - const: KMIREFCLK > > lowercase letters only I am afraid this ship has sailed: the various DTs in the kernel tree use it in that way, and the Linux driver insists on that spelling. So by changing this we would break both the existing DT's compliance and also existing Linux kernels. So is lowercase something that is mandated by DT schema, or can we just make an exception here? Cheers, Andre > > + - const: apb_pclk > > + > > +required: > > + - compatible > > + - reg > > + - interrupts > > + - clocks > > + - clock-names > > + > > +additionalProperties: false > > + > > +examples: > > + - | > > + kmi@70000 { > > Generic node names, so "serio". > > > + compatible = "arm,pl050", "arm,primecell"; > > + reg = <0x070000 0x1000>; > > + interrupts = <8>; > > + clocks = <&mb_clk24mhz>, <&soc_smc50mhz>; > > + clock-names = "KMIREFCLK", "apb_pclk"; > > + }; > > + > > +... > > > 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 3C3B4C433FE for ; Thu, 28 Apr 2022 17:28:25 +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:MIME-Version:References:In-Reply-To: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=S7dCdZvhIvQM0vUBU6fs3ekyScn657yVJxCGUJ9KWaw=; b=QzGhZEP1u17aUe DNbtHP34vAoxcGLMHJe433IEVJwlAZaK/P90llgTdElJDq9Q0R69WCHsG2YA5zflYLX4PgqWw0utg ndTbVMSdqMgi9VffZR/tMp6AE2yG2Vz8FJz80Sh9PT9sy6l0wfW8cUwtmmhi1W245LpUfh8UutF/O iakz71N4L8a6h0dpBI4xqadARSRkDo7Ig/vAvPQEYkUIOrgL60jQy07RN/Jjsu1t0ObcB2LLHhQPO w86U3DeNXsmfN23dM7j6ghFWhpL57Z64eY5h23Ygo2Udv7KVhn6mz4OlvhFt63TIl2ToXgBlLiV2t MIeONnI+ld/EM2+qUvaA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nk7vE-00824R-Cz; Thu, 28 Apr 2022 17:27:20 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nk7vA-00823B-15 for linux-arm-kernel@lists.infradead.org; Thu, 28 Apr 2022 17:27:18 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 16D841474; Thu, 28 Apr 2022 10:27:12 -0700 (PDT) Received: from donnerap.cambridge.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id C58143F774; Thu, 28 Apr 2022 10:27:10 -0700 (PDT) Date: Thu, 28 Apr 2022 18:27:08 +0100 From: Andre Przywara To: Krzysztof Kozlowski Cc: Rob Herring , Krzysztof Kozlowski , Liviu Dudau , Robin Murphy , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Dmitry Torokhov , linux-input@vger.kernel.org Subject: Re: [PATCH 05/11] dt-bindings: serio: add Arm PL050 DT schema Message-ID: <20220428182708.0c1065f8@donnerap.cambridge.arm.com> In-Reply-To: <55574426-df20-cdd7-c97d-092726981a42@linaro.org> References: <20220427112528.4097815-1-andre.przywara@arm.com> <20220427112528.4097815-6-andre.przywara@arm.com> <55574426-df20-cdd7-c97d-092726981a42@linaro.org> Organization: ARM X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.32; aarch64-unknown-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220428_102716_188349_A39C6E11 X-CRM114-Status: GOOD ( 25.12 ) 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 Thu, 28 Apr 2022 09:07:53 +0200 Krzysztof Kozlowski wrote: Hi Krzysztof, many thanks for having a look and your comments (here and on the other patches). I will fix them and send a new version ASAP. One thing below: > On 27/04/2022 13:25, Andre Przywara wrote: > > The Arm PL050 "Keyboard/Mouse Interface" is an Arm system IP providing a > > PS/2 compatible serial interface. > > > > Add a simple DT schema binding, based on the TRM[1], the existing DTs and > > the Linux driver. > > > > [1] https://developer.arm.com/documentation/ddi0143/latest > > > > Signed-off-by: Andre Przywara > > --- > > .../devicetree/bindings/serio/amba-pl050.yaml | 67 +++++++++++++++++++ > > 1 file changed, 67 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/serio/amba-pl050.yaml > > > > diff --git a/Documentation/devicetree/bindings/serio/amba-pl050.yaml b/Documentation/devicetree/bindings/serio/amba-pl050.yaml > > new file mode 100644 > > index 0000000000000..9732a84550098 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/serio/amba-pl050.yaml > > Filename: vendor,device > > > @@ -0,0 +1,67 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/serio/amba-pl050.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Arm Ltd. PrimeCell PL050 PS/2 Keyboard/Mouse Interface > > + > > +maintainers: > > + - Andre Przywara > > + > > +description: > > + The Arm PrimeCell PS2 Keyboard/Mouse Interface (KMI) is an AMBA compliant > > + peripheral that can be used to implement a keyboard or mouse interface that > > + is IBM PS2 or AT compatible. > > + > > +# We need a select here so we don't match all nodes with 'arm,primecell' > > +select: > > + properties: > > + compatible: > > + contains: > > + const: arm,pl050 > > + required: > > + - compatible > > + > > +properties: > > + compatible: > > + items: > > + - const: arm,pl050 > > + - const: arm,primecell > > + > > + reg: > > + maxItems: 1 > > + > > + interrupts: > > + maxItems: 1 > > + > > + clocks: > > + items: > > + - description: KMI reference clock, used to generate the bus timing > > + - description: APB register access clock > > + > > + clock-names: > > + items: > > + - const: KMIREFCLK > > lowercase letters only I am afraid this ship has sailed: the various DTs in the kernel tree use it in that way, and the Linux driver insists on that spelling. So by changing this we would break both the existing DT's compliance and also existing Linux kernels. So is lowercase something that is mandated by DT schema, or can we just make an exception here? Cheers, Andre > > + - const: apb_pclk > > + > > +required: > > + - compatible > > + - reg > > + - interrupts > > + - clocks > > + - clock-names > > + > > +additionalProperties: false > > + > > +examples: > > + - | > > + kmi@70000 { > > Generic node names, so "serio". > > > + compatible = "arm,pl050", "arm,primecell"; > > + reg = <0x070000 0x1000>; > > + interrupts = <8>; > > + clocks = <&mb_clk24mhz>, <&soc_smc50mhz>; > > + clock-names = "KMIREFCLK", "apb_pclk"; > > + }; > > + > > +... > > > Best regards, > Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel