[{"id":3188481,"web_url":"http://patchwork.ozlabs.org/comment/3188481/","msgid":"","list_archive_url":null,"date":"2023-09-27T07:59:05","subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":703,"url":"http://patchwork.ozlabs.org/api/people/703/","name":"Geert Uytterhoeven","email":"geert@linux-m68k.org"},"content":"Hi Fabrizio,\n\nOn Tue, Sep 26, 2023 at 11:08 PM Fabrizio Castro\n wrote:\n> The CSI IP found inside the Renesas RZ/V2M SoC can also work\n> in SPI slave mode.\n> When working in slave mode, the IP can make use of the SS\n> (Slave Select) pin, with \"low\" as default active level.\n> The active level of SS can be changed to \"high\" upon configuration.\n> This patch adds two new properties, one to make use of the\n> SS pin when in slave mode, and one to make the SS pin active high.\n>\n> Signed-off-by: Fabrizio Castro \n\nThanks for your patch!\n\n> --- a/Documentation/devicetree/bindings/spi/renesas,rzv2m-csi.yaml\n> +++ b/Documentation/devicetree/bindings/spi/renesas,rzv2m-csi.yaml\n> @@ -39,6 +39,17 @@ properties:\n> power-domains:\n> maxItems: 1\n>\n> + renesas,csi-ss:\n> + type: boolean\n> + description:\n> + Use CSI Slave Selection (SS) pin to enable transmission and reception when\n> + in slave mode.\n\nCan't this be done in a more generic way? I had expected that the\nexisting SPI_NO_CS flag can be set using a property in the \"slave\" subnode,\nbut apparently there is no \"spi-no-cs\" property defined yet.\n\n> +\n> + renesas,csi-ss-high:\n> + type: boolean\n> + description:\n> + The SS pin is active high (by default the SS pin is active low).\n\nCan't you use the \"spi-cs-high\" property in the \"slave\" subnode instead?\n\n> +\n> required:\n> - compatible\n> - reg\n\nGr{oetje,eeting}s,\n\n Geert","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":"legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45d1:ec00::1;\n helo=ny.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3678-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","Received":["from ny.mirrors.kernel.org (ny.mirrors.kernel.org\n [IPv6:2604:1380:45d1:ec00::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwTXq047xz1yp8\n\tfor ;\n Wed, 27 Sep 2023 17:59:38 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby ny.mirrors.kernel.org (Postfix) with ESMTP id E79851C20B9C\n\tfor ; Wed, 27 Sep 2023 07:59:36 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id EDCFE1D68A;\n\tWed, 27 Sep 2023 07:59:24 +0000 (UTC)","from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net\n [23.128.96.19])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 084E31D546\n\tfor ; Wed, 27 Sep 2023 07:59:22 +0000 (UTC)","from mail-yw1-f172.google.com (mail-yw1-f172.google.com\n [209.85.128.172])\n\tby lindbergh.monkeyblade.net (Postfix) with ESMTPS id EEF7A13A;\n\tWed, 27 Sep 2023 00:59:20 -0700 (PDT)","by mail-yw1-f172.google.com with SMTP id\n 00721157ae682-59f6763767dso85555607b3.2;\n Wed, 27 Sep 2023 00:59:20 -0700 (PDT)","from mail-yw1-f172.google.com (mail-yw1-f172.google.com.\n [209.85.128.172])\n by smtp.gmail.com with ESMTPSA id\n fu6-20020a05690c368600b0059590440338sm2510336ywb.43.2023.09.27.00.59.18\n (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128);\n Wed, 27 Sep 2023 00:59:18 -0700 (PDT)","by mail-yw1-f172.google.com with SMTP id\n 00721157ae682-59bf1dde73fso135389437b3.3;\n Wed, 27 Sep 2023 00:59:18 -0700 (PDT)"],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20230601; t=1695801560; x=1696406360;\n h=content-transfer-encoding:cc:to:subject:message-id:date:from\n :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc\n :subject:date:message-id:reply-to;\n bh=rMT1j26t8dOkRFRh4332nTiRpDFbKB81QWUMtGHO2gA=;\n b=kqUp42mMNpNq8bM9HthNz5TP9S551B3NIbNkJvCo4G1pKXVBldPyjZQlOx6cAcNDw0\n dOAqaM7pbKREnq5Sjt+77m2ad2sdStiWtWHzg3s+xf86vvtDoz1V4FKBWHtwX9GBQK9Z\n w8Ctra3+qjHUTQIHe1xRmOfqYwJt1MvLSKsL8zPJfNtWyiqAg75WrWkitkqT/cEhybiu\n 7y96XQTIpoFBkSeHFfIpxKfhiLcZMG7BP2NxsAsurYceT5wdiNLgkOjXpszcEry1fhBI\n i3JNpYcDRzJQmZAzw7VxxUR6SBpZogZcu1RZCUDp/UD/EpiHdxcrSehpP+gBZaYULYI3\n B+iQ==","X-Gm-Message-State":"AOJu0Yzid9Ayiwo3idVJD8ovp1qeePQ+UCbQGefZGtsqApEGwsrcGrXH\n\tOJd3wc/IyR2HITSk0OArEgyZk2RaMGyI3Q==","X-Google-Smtp-Source":"\n AGHT+IF0eGSNtWdD+DoeXCyKYUNMFhixXcqM/++g+vkb5sF4JOI5Towv7P7wAzB4A+9BwpohjVMHCA==","X-Received":["by 2002:a05:690c:2a81:b0:5a1:fb1d:740a with SMTP id\n ek1-20020a05690c2a8100b005a1fb1d740amr804324ywb.51.1695801559785;\n Wed, 27 Sep 2023 00:59:19 -0700 (PDT)","by 2002:a0d:d784:0:b0:595:80be:fc6b with SMTP id\n z126-20020a0dd784000000b0059580befc6bmr1630689ywd.18.1695801558289; Wed, 27\n Sep 2023 00:59:18 -0700 (PDT)"],"Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>","In-Reply-To":"<20230926210818.197356-2-fabrizio.castro.jz@renesas.com>","From":"Geert Uytterhoeven ","Date":"Wed, 27 Sep 2023 09:59:05 +0200","X-Gmail-Original-Message-ID":"\n ","Message-ID":"\n ","Subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","To":"Fabrizio Castro ","Cc":"Mark Brown , Rob Herring ,\n\tKrzysztof Kozlowski ,\n Conor Dooley ,\n\tMagnus Damm , linux-spi@vger.kernel.org,\n\tdevicetree@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-renesas-soc@vger.kernel.org,\n\tChris Paterson ,\n Biju Das ,\n\tLad Prabhakar ","Content-Type":"text/plain; charset=\"UTF-8\"","Content-Transfer-Encoding":"quoted-printable","X-Spam-Status":"No, score=-1.4 required=5.0 tests=BAYES_00,\n\tFREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,\n\tRCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_PASS\n\tautolearn=no autolearn_force=no version=3.4.6","X-Spam-Checker-Version":"SpamAssassin 3.4.6 (2021-04-09) on\n\tlindbergh.monkeyblade.net"}},{"id":3188539,"web_url":"http://patchwork.ozlabs.org/comment/3188539/","msgid":"","list_archive_url":null,"date":"2023-09-27T08:56:42","subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":24878,"url":"http://patchwork.ozlabs.org/api/people/24878/","name":"Mark Brown","email":"broonie@kernel.org"},"content":"On Tue, Sep 26, 2023 at 10:08:17PM +0100, Fabrizio Castro wrote:\n> The CSI IP found inside the Renesas RZ/V2M SoC can also work\n> in SPI slave mode.\n> When working in slave mode, the IP can make use of the SS\n> (Slave Select) pin, with \"low\" as default active level.\n> The active level of SS can be changed to \"high\" upon configuration.\n> This patch adds two new properties, one to make use of the\n> SS pin when in slave mode, and one to make the SS pin active high.\n\nPlease avoid the use of outdated terminology like this, prefer \"device\nmode\" or similar.\n\n> + renesas,csi-ss:\n> + type: boolean\n> + description:\n> + Use CSI Slave Selection (SS) pin to enable transmission and reception when\n> + in slave mode.\n\nWhen would this ever not be true when in device mode?","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=qe6KTU7S;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3713-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)"],"Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org\n [IPv6:2604:1380:45e3:2400::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwVpn5vZ3z1yp8\n\tfor ;\n Wed, 27 Sep 2023 18:56:49 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id 4246328392C\n\tfor ; Wed, 27 Sep 2023 08:56:48 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 680971D6BF;\n\tWed, 27 Sep 2023 08:56:46 +0000 (UTC)","from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org\n [10.30.226.201])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 3F46213ADF\n\tfor ; Wed, 27 Sep 2023 08:56:45 +0000 (UTC)","by smtp.kernel.org (Postfix) with ESMTPSA id 52132C433C7;\n\tWed, 27 Sep 2023 08:56:45 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org;\n\ts=k20201202; t=1695805005;\n\tbh=Cao/bocpgqpFeJIKvW5opyJ4/rHkM9Gq7550IJgQQDg=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=qe6KTU7Sn8Hs0MapzvaEx7O/dKZhmloYikn5MDNsYhg/AbGi8CTbhVwZZZ/yV63TW\n\t NrJlCYAghT2nraItdlDQmhX/ERO/CfbOZE/aV1/fLY6QSvjprJ/7u9gKHHYRXAgliH\n\t inT6TzQzp1Gn1lFvUWMhqgM5FDzs3tsSOmLzXMHH0EsztxOsW36EGbeajGwKKU6vXQ\n\t rEP/i7/ACILKb0kmxJoWRyQt85wfUgXgaoT9OebJqq/h+sCXUzqyRNKbHB6HQ5/34i\n\t T6b9CEhgpMOZUlNJMC9+zyKjsTej6d0XbEcoS36VI12Lbk+H/wKPCK37tDXdzU5AEt\n\t hkDIHF7mctJ0Q==","Date":"Wed, 27 Sep 2023 10:56:42 +0200","From":"Mark Brown ","To":"Fabrizio Castro ","Cc":"Rob Herring ,\n\tKrzysztof Kozlowski ,\n\tConor Dooley ,\n\tGeert Uytterhoeven ,\n\tMagnus Damm , linux-spi@vger.kernel.org,\n\tdevicetree@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-renesas-soc@vger.kernel.org,\n\tChris Paterson ,\n\tBiju Das ,\n\tLad Prabhakar ","Subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Message-ID":"","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>","Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","Content-Type":"multipart/signed; micalg=pgp-sha512;\n\tprotocol=\"application/pgp-signature\"; boundary=\"NweErwwnK6vf4dVT\"","Content-Disposition":"inline","In-Reply-To":"<20230926210818.197356-2-fabrizio.castro.jz@renesas.com>","X-Cookie":"Save energy: Drive a smaller shell."}},{"id":3188549,"web_url":"http://patchwork.ozlabs.org/comment/3188549/","msgid":"","list_archive_url":null,"date":"2023-09-27T09:00:12","subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":24878,"url":"http://patchwork.ozlabs.org/api/people/24878/","name":"Mark Brown","email":"broonie@kernel.org"},"content":"On Wed, Sep 27, 2023 at 09:59:05AM +0200, Geert Uytterhoeven wrote:\n> On Tue, Sep 26, 2023 at 11:08 PM Fabrizio Castro\n\n> > + type: boolean\n> > + description:\n> > + Use CSI Slave Selection (SS) pin to enable transmission and reception when\n> > + in slave mode.\n\n> Can't this be done in a more generic way? I had expected that the\n> existing SPI_NO_CS flag can be set using a property in the \"slave\" subnode,\n> but apparently there is no \"spi-no-cs\" property defined yet.\n\nThe description is clearly saying there is a chip select, _NO_CS seems\nentirely inappropriate. It's not specified in the device tree because\nwhen there's no chip select for a device it's a fundamental property of\nhow the device is controlled and we don't need any information beyond\nthe compatible.","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=fwEW/w7n;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3719-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)"],"Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org\n [IPv6:2604:1380:45e3:2400::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwVtq2JTVz1yp0\n\tfor ;\n Wed, 27 Sep 2023 19:00:19 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id 115D5282017\n\tfor ; Wed, 27 Sep 2023 09:00:18 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 542AC1F5EB;\n\tWed, 27 Sep 2023 09:00:16 +0000 (UTC)","from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org\n [10.30.226.201])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 2ED081F189\n\tfor ; Wed, 27 Sep 2023 09:00:15 +0000 (UTC)","by smtp.kernel.org (Postfix) with ESMTPSA id 2CA96C433C7;\n\tWed, 27 Sep 2023 09:00:14 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org;\n\ts=k20201202; t=1695805215;\n\tbh=3hwoD84olVZvjou5/kpVm2ZQJLfeUVEZBHvliNaakz8=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=fwEW/w7n8LRFF9M0zplL9D8uw/SWwiTSVd0hIYDS9p/5TOw47gAFnttbn9UIvIDWm\n\t rOZ+yF2krn173lbJYz3yZ91794OuDlI752kcTAZ19MQ83Ps+RaFYW9Do6NDMKeWrO+\n\t JT1UHT1FnzBnD5aE/Rw5FfRavK0k8+8Gk55yZvTpZAZM52HCp+N7Gps5aDRPpCMnXY\n\t nDOmw8PX/rbGikM4rN8leN1WFnd4FQr0CaY9vPjM3+XrmOVQojHztPhmZOQQpdzWpt\n\t /qZOJcF4NONUIAq0sCUqm/T0Jj4oA1LxIgPdQFslxN30bO5k9aD38eQKh2DbsrFi2s\n\t P+oAfb3y0RTSw==","Date":"Wed, 27 Sep 2023 11:00:12 +0200","From":"Mark Brown ","To":"Geert Uytterhoeven ","Cc":"Fabrizio Castro ,\n\tRob Herring ,\n\tKrzysztof Kozlowski ,\n\tConor Dooley ,\n\tMagnus Damm , linux-spi@vger.kernel.org,\n\tdevicetree@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-renesas-soc@vger.kernel.org,\n\tChris Paterson ,\n\tBiju Das ,\n\tLad Prabhakar ","Subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Message-ID":"","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>\n ","Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","Content-Type":"multipart/signed; micalg=pgp-sha512;\n\tprotocol=\"application/pgp-signature\"; boundary=\"QiSZLj8ULhE5uAOG\"","Content-Disposition":"inline","In-Reply-To":"\n ","X-Cookie":"Save energy: Drive a smaller shell."}},{"id":3188566,"web_url":"http://patchwork.ozlabs.org/comment/3188566/","msgid":"","list_archive_url":null,"date":"2023-09-27T09:10:58","subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":703,"url":"http://patchwork.ozlabs.org/api/people/703/","name":"Geert Uytterhoeven","email":"geert@linux-m68k.org"},"content":"Hi Mark,\n\nOn Wed, Sep 27, 2023 at 11:00 AM Mark Brown wrote:\n> On Wed, Sep 27, 2023 at 09:59:05AM +0200, Geert Uytterhoeven wrote:\n> > On Tue, Sep 26, 2023 at 11:08 PM Fabrizio Castro\n> > > + type: boolean\n> > > + description:\n> > > + Use CSI Slave Selection (SS) pin to enable transmission and reception when\n> > > + in slave mode.\n>\n> > Can't this be done in a more generic way? I had expected that the\n> > existing SPI_NO_CS flag can be set using a property in the \"slave\" subnode,\n> > but apparently there is no \"spi-no-cs\" property defined yet.\n>\n> The description is clearly saying there is a chip select, _NO_CS seems\n> entirely inappropriate. It's not specified in the device tree because\n> when there's no chip select for a device it's a fundamental property of\n> how the device is controlled and we don't need any information beyond\n> the compatible.\n\nIn host mode, it indeed doesn't matter, as you can have only a single\ndevice connected with SPI_NO_CS.\nIn device mode, the device needs to know if it must monitor the chip\nselect line or not.\n\nIn hindsight, I should have kept the question I had written initially,\nbut deleted after having read the documentation for the corresponding\nRZ/V2M register bits:\n\n What does it mean if this is false? That there is no chip select?\n\nSo \"spi-no-cs\" would be the inverse of \"renesas,csi-ss\".\n\nGr{oetje,eeting}s,\n\n Geert","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":"legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3733-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org\n [IPv6:2604:1380:45e3:2400::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwW7Z3mKKz1ypJ\n\tfor ;\n Wed, 27 Sep 2023 19:11:22 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id 83F3C281FD4\n\tfor ; Wed, 27 Sep 2023 09:11:19 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 89B18200D1;\n\tWed, 27 Sep 2023 09:11:17 +0000 (UTC)","from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net\n [23.128.96.19])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 13A5C290B\n\tfor ; Wed, 27 Sep 2023 09:11:16 +0000 (UTC)","from mail-yw1-f173.google.com (mail-yw1-f173.google.com\n [209.85.128.173])\n\tby lindbergh.monkeyblade.net (Postfix) with ESMTPS id 67F86BF;\n\tWed, 27 Sep 2023 02:11:13 -0700 (PDT)","by mail-yw1-f173.google.com with SMTP id\n 00721157ae682-59c0d329a8bso131600407b3.1;\n Wed, 27 Sep 2023 02:11:13 -0700 (PDT)","from mail-yw1-f171.google.com (mail-yw1-f171.google.com.\n [209.85.128.171])\n by smtp.gmail.com with ESMTPSA id\n gq10-20020a05690c444a00b0059f5828346csm2374946ywb.3.2023.09.27.02.11.11\n (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128);\n Wed, 27 Sep 2023 02:11:11 -0700 (PDT)","by mail-yw1-f171.google.com with SMTP id\n 00721157ae682-59bc956b029so131375657b3.2;\n Wed, 27 Sep 2023 02:11:11 -0700 (PDT)"],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20230601; t=1695805872; x=1696410672;\n h=content-transfer-encoding:cc:to:subject:message-id:date:from\n :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc\n :subject:date:message-id:reply-to;\n bh=kT5xQELVLk7uusSSsznnnd5tIH8qDLjO1UEAzy07OsA=;\n b=Yuo+bPsfxjO0oPNDawp8TCWQLa85WrM80BTN0oAutD1V3p0Osszngb0DyEtD2+n7r7\n 0AgXQRBfOF6JkDknZQV2r+SJwJkbdhDtjviDQPjMib2p/wvWkXOkr4a39fpArHTH8tWb\n f21MPgznaoIZDNjxQCxbyRvHBOMW2emxqYtIkbzb30itOVv0WmQjwCT0It5OLyO2Yzda\n 768qZAYKAxqqgoqsU3vKLIKpnOCi5JZgqNdMgn/PMUlb4aco/qAK7EQwaHjuHWp9lyip\n 68qooiqSefN9R5N10HDQ5EYhL7gq+5u8D6FFqb39avt0bS7PnzwUe4XCLmhx5jMzhoNP\n EF7w==","X-Gm-Message-State":"AOJu0Yw/RWvtbj6Zy9xIHwyR+qola6TqgimVK80q8U0w6FmwT1sjSaEE\n\t2EU3sKl+uTgpBBTeewUh7k0Fj/84UJ6LHg==","X-Google-Smtp-Source":"\n AGHT+IFJ74w9GvEdlwxKPeVi2H1pLKC9BHDPLx0a2JuDv/hhfQKppSj6zXfXnv9pu61WI/PN78QHKA==","X-Received":["by 2002:a0d:d415:0:b0:569:479f:6d7f with SMTP id\n w21-20020a0dd415000000b00569479f6d7fmr1702501ywd.43.1695805872377;\n Wed, 27 Sep 2023 02:11:12 -0700 (PDT)","by 2002:a0d:c946:0:b0:59b:fb69:1639 with SMTP id\n l67-20020a0dc946000000b0059bfb691639mr1846689ywd.32.1695805871578; Wed, 27\n Sep 2023 02:11:11 -0700 (PDT)"],"Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>\n \n ","In-Reply-To":"","From":"Geert Uytterhoeven ","Date":"Wed, 27 Sep 2023 11:10:58 +0200","X-Gmail-Original-Message-ID":"\n ","Message-ID":"\n ","Subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","To":"Mark Brown ","Cc":"Fabrizio Castro ,\n Rob Herring ,\n\tKrzysztof Kozlowski ,\n Conor Dooley ,\n\tMagnus Damm , linux-spi@vger.kernel.org,\n\tdevicetree@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-renesas-soc@vger.kernel.org,\n\tChris Paterson ,\n Biju Das ,\n\tLad Prabhakar ","Content-Type":"text/plain; charset=\"UTF-8\"","Content-Transfer-Encoding":"quoted-printable","X-Spam-Status":"No, score=-1.4 required=5.0 tests=BAYES_00,\n\tFREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,\n\tRCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_PASS\n\tautolearn=no autolearn_force=no version=3.4.6","X-Spam-Checker-Version":"SpamAssassin 3.4.6 (2021-04-09) on\n\tlindbergh.monkeyblade.net"}},{"id":3188576,"web_url":"http://patchwork.ozlabs.org/comment/3188576/","msgid":"","list_archive_url":null,"date":"2023-09-27T09:21:54","subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":24878,"url":"http://patchwork.ozlabs.org/api/people/24878/","name":"Mark Brown","email":"broonie@kernel.org"},"content":"On Wed, Sep 27, 2023 at 11:10:58AM +0200, Geert Uytterhoeven wrote:\n> On Wed, Sep 27, 2023 at 11:00 AM Mark Brown wrote:\n\n> > The description is clearly saying there is a chip select, _NO_CS seems\n> > entirely inappropriate. It's not specified in the device tree because\n> > when there's no chip select for a device it's a fundamental property of\n> > how the device is controlled and we don't need any information beyond\n> > the compatible.\n\n> In host mode, it indeed doesn't matter, as you can have only a single\n> device connected with SPI_NO_CS.\n> In device mode, the device needs to know if it must monitor the chip\n> select line or not.\n\n> In hindsight, I should have kept the question I had written initially,\n> but deleted after having read the documentation for the corresponding\n> RZ/V2M register bits:\n\n> What does it mean if this is false? That there is no chip select?\n\n> So \"spi-no-cs\" would be the inverse of \"renesas,csi-ss\".\n\nI see. Is there any control over what the chip select is when there is\none, in which case we could just look to see if there's one specified?\n\nI'm a bit nervous about a generic property that maps onto _NO_CS since\nit's likely that people will start using that in device bindings when\nthey shouldn't.","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=cpKxZGCH;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3742-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)"],"Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org\n [IPv6:2604:1380:45e3:2400::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwWMt0SF9z1ypJ\n\tfor ;\n Wed, 27 Sep 2023 19:22:02 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id A77E2281BEC\n\tfor ; Wed, 27 Sep 2023 09:22:00 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 74B2120B1C;\n\tWed, 27 Sep 2023 09:21:59 +0000 (UTC)","from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org\n [10.30.226.201])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 517DE20B0F\n\tfor ; Wed, 27 Sep 2023 09:21:57 +0000 (UTC)","by smtp.kernel.org (Postfix) with ESMTPSA id 40E62C433D9;\n\tWed, 27 Sep 2023 09:21:57 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org;\n\ts=k20201202; t=1695806517;\n\tbh=I448svr5ZDYp4rTesI5QISMQCcJhofO8M8peXBBCVFM=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=cpKxZGCHxET7gWuWcExH6CsMZ9pfGLXCTxYP/6YtBpQYujCM4n+xPTZfWZt+hhP0+\n\t NXnwHc+pAdxCGnMaHqG/Fe0XfyZ7hnrvR9yUvw5WO7wnbI2+AAbR5mEarzdMpY4ZrU\n\t Q29GnboTaxtoij/admCVD36krYP8urbjTRMsMmGABzccsys3ZA/Ny0/47irocy/c1H\n\t kwy9x7vYNoO9GgSvezOMjMyX1J+5JH9ldG3965MD6GB6quP81N1kV3L5vIp+dYPh66\n\t aRBHYrFHWg5mxLkDxWgfrWMMQpw32GaL4V4LoXSvjgjHS40bbRBm3cj0Mf5hKGUQu9\n\t 2N2HvVBwnyXrA==","Date":"Wed, 27 Sep 2023 11:21:54 +0200","From":"Mark Brown ","To":"Geert Uytterhoeven ","Cc":"Fabrizio Castro ,\n\tRob Herring ,\n\tKrzysztof Kozlowski ,\n\tConor Dooley ,\n\tMagnus Damm , linux-spi@vger.kernel.org,\n\tdevicetree@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-renesas-soc@vger.kernel.org,\n\tChris Paterson ,\n\tBiju Das ,\n\tLad Prabhakar ","Subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Message-ID":"","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>\n \n \n ","Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","Content-Type":"multipart/signed; micalg=pgp-sha512;\n\tprotocol=\"application/pgp-signature\"; boundary=\"Pm+YXDPO8WF2t5nM\"","Content-Disposition":"inline","In-Reply-To":"\n ","X-Cookie":"Save energy: Drive a smaller shell."}},{"id":3188592,"web_url":"http://patchwork.ozlabs.org/comment/3188592/","msgid":"","list_archive_url":null,"date":"2023-09-27T09:36:29","subject":"RE: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":80106,"url":"http://patchwork.ozlabs.org/api/people/80106/","name":"Fabrizio Castro","email":"fabrizio.castro.jz@renesas.com"},"content":"Hi Geert,\n\nThanks for your reply!\n\n> From: Geert Uytterhoeven \n> Subject: Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n> properties\n> \n> Hi Fabrizio,\n> \n> On Tue, Sep 26, 2023 at 11:08 PM Fabrizio Castro\n> wrote:\n> > The CSI IP found inside the Renesas RZ/V2M SoC can also work\n> > in SPI slave mode.\n> > When working in slave mode, the IP can make use of the SS\n> > (Slave Select) pin, with \"low\" as default active level.\n> > The active level of SS can be changed to \"high\" upon configuration.\n> > This patch adds two new properties, one to make use of the\n> > SS pin when in slave mode, and one to make the SS pin active high.\n> >\n> > Signed-off-by: Fabrizio Castro \n> \n> Thanks for your patch!\n> \n> > --- a/Documentation/devicetree/bindings/spi/renesas,rzv2m-csi.yaml\n> > +++ b/Documentation/devicetree/bindings/spi/renesas,rzv2m-csi.yaml\n> > @@ -39,6 +39,17 @@ properties:\n> > power-domains:\n> > maxItems: 1\n> >\n> > + renesas,csi-ss:\n> > + type: boolean\n> > + description:\n> > + Use CSI Slave Selection (SS) pin to enable transmission and\n> reception when\n> > + in slave mode.\n> \n> Can't this be done in a more generic way? I had expected that the\n> existing SPI_NO_CS flag can be set using a property in the \"slave\"\n> subnode,\n> but apparently there is no \"spi-no-cs\" property defined yet.\n\nI couldn't find a generic way to address that in a generic way.\n\n> \n> > +\n> > + renesas,csi-ss-high:\n> > + type: boolean\n> > + description:\n> > + The SS pin is active high (by default the SS pin is active\n> low).\n> \n> Can't you use the \"spi-cs-high\" property in the \"slave\" subnode\n> instead?\n\nI could. I didn't go for it for the first version of this series\nafter reading the automatic handling of SPI_CS_HIGH (which for\nexample doesn't get automatically added to the mode for targets\nusing a GPIO as CS). Even though nothing prevents you from\nexplicitly adding SPI_CS_HIGH to the mode and using \"spi-cs-high\"\nin the device tree, I thought that could be confusing, but\nmaybe it's not?\n\nThanks,\nFab\n\n> \n> > +\n> > required:\n> > - compatible\n> > - reg\n> \n> Gr{oetje,eeting}s,\n> \n> Geert\n> \n> --\n> Geert Uytterhoeven -- There's lots of Linux beyond ia32 --\n> geert@linux-m68k.org\n> \n> In personal conversations with technical people, I call myself a\n> hacker. But\n> when I'm talking to journalists I just say \"programmer\" or something\n> like that.\n> -- Linus Torvalds","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=renesas.com header.i=@renesas.com header.a=rsa-sha256\n header.s=selector1 header.b=RsbgNHUO;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3750-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","dkim=none (message not signed)\n header.d=none;dmarc=none action=none header.from=renesas.com;"],"Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org\n [IPv6:2604:1380:45e3:2400::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwWhn2PFYz1yp8\n\tfor ;\n Wed, 27 Sep 2023 19:36:41 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id 8394F281E78\n\tfor ; Wed, 27 Sep 2023 09:36:39 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 7D3732377F;\n\tWed, 27 Sep 2023 09:36:37 +0000 (UTC)","from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net\n [23.128.96.19])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id C54ED1D6B4\n\tfor ; Wed, 27 Sep 2023 09:36:35 +0000 (UTC)","from JPN01-TYC-obe.outbound.protection.outlook.com\n (mail-tycjpn01on2123.outbound.protection.outlook.com [40.107.114.123])\n\tby lindbergh.monkeyblade.net (Postfix) with ESMTPS id DCD4F1FF5;\n\tWed, 27 Sep 2023 02:36:32 -0700 (PDT)","from TYWPR01MB8775.jpnprd01.prod.outlook.com (2603:1096:400:169::11)\n by TY3PR01MB10143.jpnprd01.prod.outlook.com (2603:1096:400:1d8::5) with\n Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6838.21; Wed, 27 Sep\n 2023 09:36:30 +0000","from TYWPR01MB8775.jpnprd01.prod.outlook.com\n ([fe80::600a:9708:18f:2528]) by TYWPR01MB8775.jpnprd01.prod.outlook.com\n ([fe80::600a:9708:18f:2528%3]) with mapi id 15.20.6838.016; Wed, 27 Sep 2023\n 09:36:30 +0000"],"ARC-Seal":"i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=XjASV+Pb1nwQ3GKrqN07MH08KQD99aueKZKCLRV3R4giExdTF0fFQmySD1F0HZS4v9PLO6wiNmYw5aoG8MXmuZsf/6tVe2U0brhmkRL9fUtEUuOOYK3FyYtM2m45zAS+S2gccomHNk3nnbcSKV1vzbLqFpyUXVQ1PJLC5H2ORLCnE0a0WtErf3WDy8BwMgaoYMF0VicZ8DFPADBxBc+VUcuhAXHOKNfS9S5C/TLPIEi7OY+RRrDQeZR6N3jZU96E0We1LCzS9cupZhUrqEdCBftjqN67fQm8wudwTPoOo9AUgjPY6QGm8W0QbFIQkzTZmcjGUPOEK9yFfoXHAuJ7zg==","ARC-Message-Signature":"i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector9901;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;\n bh=Uzy7x3GBsc9VwKt/2P6KPU4zqA+/XRH6D5dVMAm+6zE=;\n b=jR4BpNqzej7/9R4dq5DUvRgp6mMUsxt+ScD6AAgZ7htlo6OJ4G1TdfUdukJm1IWtIWGx1GNhXtIVHN7ctrDtHlFzgp6MdHzSbJKRW8/rZmoNcGTAI/Qn4hSIcaNmFLMLF8hFZupw0Wx9Q6eJWF0uao2/p2L/xWsh+zsDEes/7li8RR1qTYCX2UEKNUazXoTmLqGh1CBJmgrAwmgUS1tk+AiWv0CwP7FpQt2RVLHOTUMd3qLc6p+BDnkmp+1e5AhRKicLUOd9OorMumraYOOGj4N4cpbgSNgWQE+JifPfgDnTvK/xWfJ+OEOWhz3kU9YBVgOk92chNVKMqgUdrR4f8w==","ARC-Authentication-Results":"i=1; mx.microsoft.com 1; spf=pass\n smtp.mailfrom=renesas.com; dmarc=pass action=none header.from=renesas.com;\n dkim=pass header.d=renesas.com; arc=none","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=renesas.com;\n s=selector1;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=Uzy7x3GBsc9VwKt/2P6KPU4zqA+/XRH6D5dVMAm+6zE=;\n b=RsbgNHUONrUJzhTceRaq2QUgOoFGT4/GIXx4QQglkQfwCcyk9mBbgdXgbvkDEcs0QBcFCmCt5XEkhbdjqRJ9lQpNYjUp7MsjSEA0zUcGf5srlhRZnJXFTIKLk50v/kdY2MUl4u7mGSvkTDzY0o9Lx6g60po36BV6M1FSo4b0bTI=","From":"Fabrizio Castro ","To":"Geert Uytterhoeven ","CC":"Mark Brown , Rob Herring ,\n\tKrzysztof Kozlowski , Conor Dooley\n\t, Magnus Damm ,\n\t\"linux-spi@vger.kernel.org\" ,\n\t\"devicetree@vger.kernel.org\" ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"linux-renesas-soc@vger.kernel.org\" ,\n\tChris Paterson , Biju Das\n\t, Prabhakar Mahadev Lad\n\t","Subject":"RE: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Thread-Topic":"[PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Thread-Index":"AQHZ8L2blakiXIKx8k6DD60xX0IJ3LAuT32AgAAaGRA=","Date":"Wed, 27 Sep 2023 09:36:29 +0000","Message-ID":"\n ","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>\n ","In-Reply-To":"\n ","Accept-Language":"en-GB, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","authentication-results":["legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=renesas.com header.i=@renesas.com header.a=rsa-sha256\n header.s=selector1 header.b=RsbgNHUO;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3750-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","dkim=none (message not signed)\n header.d=none;dmarc=none action=none header.from=renesas.com;"],"x-ms-publictraffictype":"Email","x-ms-traffictypediagnostic":"TYWPR01MB8775:EE_|TY3PR01MB10143:EE_","x-ms-office365-filtering-correlation-id":"7e9bdaf0-8557-41e7-9478-08dbbf3d3a9c","x-ms-exchange-senderadcheck":"1","x-ms-exchange-antispam-relay":"0","x-microsoft-antispam":"BCL:0;","x-microsoft-antispam-message-info":"\n dqkfnbioB7YBFkkXZacyxJgYeoxcLT/2Aqltpey1w3iLpKDTf2TyzCItJ1Mj8rjd4dqcAkwr2+AW0UVASD5zUGUN2sqHvPw0mt3SmGxgLI5aTIIAmpvY0eKcNs+BUbc/DGNGu6QRrEXQSU/broISDvFB847CxFhsmW4ZdWcUs9v52Hj4ZrHbOSWIG9ivKiQy9hJGH36jzlg8maZaneBElVQKUSJgX6EwuPQVGgfO+SEeFohbJzLfb85e75Pr3zT+bPyAo5xnnfoeiuhLqLLu7WozJNcr2GERsbPKStv3FjwM9/yvpawPsSISkwMJHIjiYiJGWqWLWADrNFUo/FFYu0LbqMZSFXhRW/tgxuwPqaVpkWjx7DMdAt6aRcaD6PK99Fi6hw4wEsydVk5a5OUkIGaszS/bF/UfKnzD/cq30uVce9/tF0L0BAewN9AcbUji7L/XMKWR0ApgpQNbZ3dMlqtl1EKUYhuz2UFr4jQO9tORgoBE96Cwn25qG1kutsRkwesP7YIbprRtrTzXlBbsEJE7M47J92atHLvWACYQpj9qYRJYS2VdSR184o/KHTS/zfU3RZH4ODVz63R1hHYfnO1MRMU1kCUpEECM+9A+1B2Ec/ayD12JCsaDZQWSkdDZ","x-forefront-antispam-report":"\n CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TYWPR01MB8775.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(366004)(396003)(39860400002)(136003)(376002)(346002)(230922051799003)(186009)(1800799009)(451199024)(2906002)(7416002)(8936002)(8676002)(4326008)(5660300002)(52536014)(64756008)(54906003)(66476007)(66556008)(66946007)(41300700001)(6916009)(316002)(76116006)(66446008)(478600001)(71200400001)(6506007)(33656002)(7696005)(53546011)(9686003)(86362001)(55016003)(26005)(107886003)(38070700005)(38100700002)(122000001);DIR:OUT;SFP:1102;","x-ms-exchange-antispam-messagedata-chunkcount":"1","x-ms-exchange-antispam-messagedata-0":"=?utf-8?q?XpEiF5VflddEP9p7t2hJ+jqidmwr?=\n\t=?utf-8?q?7EI7aZLlVKPXVSJTGWg0TbpbMHWVYacPOwK3T5UhdPs2zhuDRM9ObQ6wFIYhL26Cs?=\n\t=?utf-8?q?yErP09ppWMQPoVHgs7CHrqKCqki5ULkcopLMBtn4dowhddl/G/aOBcWRLiCQJZzDi?=\n\t=?utf-8?q?Eqwsn3LInQvsYMSm6CmdYeIbQonSfd9cB1qwLYY0jSTRXNMiL+yZtZyyHNMNHp0Zw?=\n\t=?utf-8?q?w3lRvH5knXERkmuzPJs3enftN6mTzqVpeBDABtV8zKhlmJ7ey5l0iTawRNnDLkMCX?=\n\t=?utf-8?q?YeFk867PL+MqaJ1Btic6SGSKGpey2HJqK7Us37pLxuydFAbB9bEh6rnE5WZwFaS4T?=\n\t=?utf-8?q?x3qyXpr9CAK+fesibHiQXbp0EaghBS4xou4YJX3AjX+Z9W5weor48uQ456R2eoC6+?=\n\t=?utf-8?q?bkJAtCQ3eEtvGxnYmBkkt4uhcsoKJy7sHlj1lWzHFfJ7G3+BtGBRGX7iN4XJ1HXxb?=\n\t=?utf-8?q?AazvI39w1j0LQfhsFdNZMPrO0au7Mdnem7GkeAcrrFFeWXSHFsRdYB9DAduY8LhQ1?=\n\t=?utf-8?q?s7qE06PXqM96Dv1ekO3HrL1ei3kjBJyrUSX1CkxuL1qVl+tHEn3sAN6Ar7lXHrgXS?=\n\t=?utf-8?q?vriIAfnYdy/d+bl4UYYIY7ygZkpnZ0655kqF/z02mIZms3uOzx+8Te66LPE/xQ6My?=\n\t=?utf-8?q?bQrhtx62KJrpgOuQReuSLiI9AhTJfu2rRkLAaV4anDnGrvMgIy6LvcTy+puF5seAk?=\n\t=?utf-8?q?dmnHMHsG//47OtqAD5x22T9sleHGGOEOPZ4wgbNINZfEx/5FXCj7zJNsDE8F15P0a?=\n\t=?utf-8?q?Ejt6WTV4pTtOfGI614+9yp4iOw+Cr53qCALQsM6e0R39TNcZsf8dZJN6YCrf6L/hO?=\n\t=?utf-8?q?tXPMFIxrccV2JYs8m5Kn1Ob2S3+a5z1sl60R4YIYf72uHjm7IpDacusL3oMf4CsD+?=\n\t=?utf-8?q?Ned3GMLzozjiiFJuDymM1PvP55JF1j9xdCKnpfGERM/FBq7gyu1NaU5SJRxkSsvM0?=\n\t=?utf-8?q?kM5ycmWvk4Iok8wLO1jv7hGDXSWJ+XLE5FyirkPMRcMjko6j6wXO6ORkUtYVdG1VM?=\n\t=?utf-8?q?HWW6GLv2rKBdg+oB2TJ+4eOY3E2x78I9f4NjWgxV0EjSSSMUbpPs/Obi2zwlShYig?=\n\t=?utf-8?q?71xyQheNL5tAYaBoUaptBTcHK9LTYgaYJHAGX2F0jgixjH7tiRQvhs+6w1S7Wn/jT?=\n\t=?utf-8?q?yTXQ6S+KrQyZVfdCJ9bFwl2aJW24tVifsk4+S9vmj+tBiNf7b1ejyAKZnvDls+YWz?=\n\t=?utf-8?q?F+/vQ3JsekGKuf0Q/LnR+ozYIZIk2EFt1wrya9M6x1QXEpgZdz9yAZ3CUhDrjsrk4?=\n\t=?utf-8?q?FC6Na9Kjp/L+Hs4jONsMB/8rxzqfWYHX/5OwbGqsg7w4nUB17RXyq3P6R/6K3c4NQ?=\n\t=?utf-8?q?jz8NFfXUMwJaygjkeK9flyT4ZTsoejiKOgzBKJV+n7VaUTZDWzwfyaYnZAmpUXi/b?=\n\t=?utf-8?q?/SIIamGYBXQNJDoNqIRlknrW6E2rrP48TakDTyPnnrsglhVHskMIRNXVcWSxLInE3?=\n\t=?utf-8?q?DOlp+qdgQFEm96lkFjYwhXxPJOFWZEXOLQkPWX1rm5VmiY3XlaqiHfHbabcwqvp+X?=\n\t=?utf-8?q?qMiyBV377cGCMMgTurIgARWFSUeEZ5M37g=3D=3D?=","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","X-OriginatorOrg":"renesas.com","X-MS-Exchange-CrossTenant-AuthAs":"Internal","X-MS-Exchange-CrossTenant-AuthSource":"TYWPR01MB8775.jpnprd01.prod.outlook.com","X-MS-Exchange-CrossTenant-Network-Message-Id":"\n 7e9bdaf0-8557-41e7-9478-08dbbf3d3a9c","X-MS-Exchange-CrossTenant-originalarrivaltime":"27 Sep 2023 09:36:29.9703\n (UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"53d82571-da19-47e4-9cb4-625a166a4a2a","X-MS-Exchange-CrossTenant-mailboxtype":"HOSTED","X-MS-Exchange-CrossTenant-userprincipalname":"\n L9fSiQwK5l32iyVcUGHZgkHOuVPV0Rw3yCUwjOnhh3CSCX0beh5L0at5RlgOjOsCuc8PoX37vNiRa1L5tpoObPf8TmVzhRDLoiFKGCljboM=","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"TY3PR01MB10143","X-Spam-Status":"No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,\n\tDKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,\n\tRCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham\n\tautolearn_force=no version=3.4.6","X-Spam-Checker-Version":"SpamAssassin 3.4.6 (2021-04-09) on\n\tlindbergh.monkeyblade.net"}},{"id":3188596,"web_url":"http://patchwork.ozlabs.org/comment/3188596/","msgid":"","list_archive_url":null,"date":"2023-09-27T09:40:00","subject":"RE: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":80106,"url":"http://patchwork.ozlabs.org/api/people/80106/","name":"Fabrizio Castro","email":"fabrizio.castro.jz@renesas.com"},"content":"Hi Mark,\n\nThanks for your reply!\n\n> From: Mark Brown \n> Subject: Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n> properties\n> \n> On Tue, Sep 26, 2023 at 10:08:17PM +0100, Fabrizio Castro wrote:\n> > The CSI IP found inside the Renesas RZ/V2M SoC can also work\n> > in SPI slave mode.\n> > When working in slave mode, the IP can make use of the SS\n> > (Slave Select) pin, with \"low\" as default active level.\n> > The active level of SS can be changed to \"high\" upon configuration.\n> > This patch adds two new properties, one to make use of the\n> > SS pin when in slave mode, and one to make the SS pin active high.\n> \n> Please avoid the use of outdated terminology like this, prefer \"device\n> mode\" or similar.\n\nOkay, I'll resend with a more modern terminology (\"host\" and \"target\").\n\n\n> \n> > + renesas,csi-ss:\n> > + type: boolean\n> > + description:\n> > + Use CSI Slave Selection (SS) pin to enable transmission and\n> reception when\n> > + in slave mode.\n> \n> When would this ever not be true when in device mode?\n\n\nThis IP can do without the SS pin to start operations. In fact, the\nSS pin is disabled by default (I know...), it needs to be explicitly\nenabled, and when enabled the pin is active low by default.\n\nCheers,\nFab","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=renesas.com header.i=@renesas.com header.a=rsa-sha256\n header.s=selector1 header.b=YbHajPfD;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=139.178.88.99;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3752-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","dkim=none (message not signed)\n header.d=none;dmarc=none action=none header.from=renesas.com;"],"Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org [139.178.88.99])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwWmp5Lgjz1yp0\n\tfor ;\n Wed, 27 Sep 2023 19:40:10 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id 71010281E5C\n\tfor ; Wed, 27 Sep 2023 09:40:09 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id A934924214;\n\tWed, 27 Sep 2023 09:40:07 +0000 (UTC)","from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net\n [23.128.96.19])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id CEBCE1D6B4\n\tfor ; Wed, 27 Sep 2023 09:40:05 +0000 (UTC)","from JPN01-OS0-obe.outbound.protection.outlook.com\n (mail-os0jpn01on2120.outbound.protection.outlook.com [40.107.113.120])\n\tby lindbergh.monkeyblade.net (Postfix) with ESMTPS id EFB9FEB;\n\tWed, 27 Sep 2023 02:40:02 -0700 (PDT)","from TYWPR01MB8775.jpnprd01.prod.outlook.com (2603:1096:400:169::11)\n by TYWPR01MB10981.jpnprd01.prod.outlook.com (2603:1096:400:391::11) with\n Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6813.30; Wed, 27 Sep\n 2023 09:40:00 +0000","from TYWPR01MB8775.jpnprd01.prod.outlook.com\n ([fe80::600a:9708:18f:2528]) by TYWPR01MB8775.jpnprd01.prod.outlook.com\n ([fe80::600a:9708:18f:2528%3]) with mapi id 15.20.6838.016; Wed, 27 Sep 2023\n 09:40:00 +0000"],"ARC-Seal":"i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=MQYmBvQpQR+VZp/b5uIBXPbiC8YwkKFlDI0eFEANWe82wZRyx8AJnzhLqc5gcuv2ny5b35K2Ft9Z6IyLdRdS6yUFX7rVuXMivCnm1fALRCvHVbgTq55zDotdqDyUqvnBqTqZziO4744JD0BGaq1B1Wd+4INvE9bKzYy82eYElJosiIgcw5ExyeiIn3JYzcRgeBfPcHsUmAp3IDY9yd+A0MN89o8MdHvnQ6zvyfH/3QTwLBqjO2aumxVIPLdVYN6tOpJneYyE0Jir/JS7ai9XiyUlNLKZYHGzc4oMOc0aeloiXdXGSX9FMIZtBWCKXJujrjIz/rdQz3C6tWo4wPIAOw==","ARC-Message-Signature":"i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector9901;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;\n bh=qtKUZ6qZehfvvogFWl3TjTqi7p5DrqGpVfxXzDw2LOU=;\n b=WSI/AKfa63ry7xNJme5hnqcmq/2eb4RFwA2CxkDdETV0296vcqDsL7h2qlqlNOezaA1SDdqRj74VQdUVtoo/KGj7NS09ppNnq+XYXCwyr4G5wOJl1dTWHsQS2jqVBFFXmMjbCfyQrGArC/ssC9VG/x2aybWz6sQsgpIXh5Vgcd35u+4UWJa2cwV9OC0UlHVaQF8EHVyuQ3rDvlqnIjuzhGpwV5N4rkvtIXaUc7W6YMDbm0fPSzaPnZnlYNFhEbhvLD3bysulgOQqP/sW5Nrc2VSeF/cbYCk5nTuv6lntuMOjTbQLNRCa7sycvYaw4krD9GYTuBO56oRKly4WDjmyPg==","ARC-Authentication-Results":"i=1; mx.microsoft.com 1; spf=pass\n smtp.mailfrom=renesas.com; dmarc=pass action=none header.from=renesas.com;\n dkim=pass header.d=renesas.com; arc=none","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=renesas.com;\n s=selector1;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=qtKUZ6qZehfvvogFWl3TjTqi7p5DrqGpVfxXzDw2LOU=;\n b=YbHajPfDzEfS9CwccWeHcEYsxzxDOFHcI6WRA/JCdQB5xR0MVq+xZ2tHAiQH0b3+ke7SpDKJKH2G7kvZZBtyLS/B9E2gT3uyqqtfPtnj0srIwD/zJEoKLrfMa9iXArE9GhxSGPzF17VIYR4KZR6VjDrtSfGCAJpgzLWl8O/+B3I=","From":"Fabrizio Castro ","To":"Mark Brown ","CC":"Rob Herring , Krzysztof Kozlowski\n\t, Conor Dooley ,\n\tGeert Uytterhoeven , Magnus Damm\n\t, \"linux-spi@vger.kernel.org\"\n\t, \"devicetree@vger.kernel.org\"\n\t, \"linux-kernel@vger.kernel.org\"\n\t, \"linux-renesas-soc@vger.kernel.org\"\n\t, Chris Paterson\n\t, Biju Das , Prabhakar\n Mahadev Lad ","Subject":"RE: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Thread-Topic":"[PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Thread-Index":"AQHZ8L2blakiXIKx8k6DD60xX0IJ3LAuX5YAgAALJcA=","Date":"Wed, 27 Sep 2023 09:40:00 +0000","Message-ID":"\n ","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>\n ","In-Reply-To":"","Accept-Language":"en-GB, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","authentication-results":["legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=renesas.com header.i=@renesas.com header.a=rsa-sha256\n header.s=selector1 header.b=YbHajPfD;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=139.178.88.99;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3752-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","dkim=none (message not signed)\n header.d=none;dmarc=none action=none header.from=renesas.com;"],"x-ms-publictraffictype":"Email","x-ms-traffictypediagnostic":"TYWPR01MB8775:EE_|TYWPR01MB10981:EE_","x-ms-office365-filtering-correlation-id":"b1d46e4c-2138-4fbc-7d45-08dbbf3db7d7","x-ms-exchange-senderadcheck":"1","x-ms-exchange-antispam-relay":"0","x-microsoft-antispam":"BCL:0;","x-microsoft-antispam-message-info":"\n 8ncdOtJI+Csx5AaLu62D6LTcGKGYS+8aOI90aglL4kw+ZNv4Iaiu1ZVhnr39EZL9jpi6PE47JiamVL/yiTVmdrs66iWycHP3bB7bpv+jUAHqZGj/OWP9qWAxJ43wsYDhXp8m2m0hhgm4Y6zFzgktn7HNGUfQ7iVqDrTfv0txYBI2bwAAsetSYBrYOm1HeT4eB7skclhuvlNt/iR+MEEGyelPWEw10sOec9CNbJ+NaDVQEwk3p99IjOh2On9ukXa6KZdxgxGnjXpPAJbq+7zJfKi29vNMT2TCkXIO64tzTle5UsYYxA2q9wwaiqVc6BpkMQ/7/BgvPqnQEZLAQY0UUHCpT5GfFu1k1n+aW9Q8pG6ojnkuUQSQ06hv/81lSyoxEPPi8RtXxq7Aqcq1iaTWTwQDSgo6f1J4GLEr6uVmpxCssgD+Kdv0M/XYN4L5WPpeUnCX8ybjxyB6lY8Axu8zo6L/LI8VuH5t1gptRcdNTHxN0RrpaDsaGYfGGoFb3thXUjTihnnVhRewjp9kYELqJpWinojKKft9pgQMP6j6t0zkk74xREPcPhaCTGWMTM0Kyjxgy0vDiP6CK1O7CuqxpI7D2lG1PdqS6B/I74lvAfXm70Hy4PediQ08bmmEB3x5","x-forefront-antispam-report":"\n CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TYWPR01MB8775.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(366004)(396003)(346002)(136003)(376002)(39860400002)(230922051799003)(186009)(451199024)(1800799009)(86362001)(8936002)(8676002)(4326008)(52536014)(5660300002)(41300700001)(33656002)(2906002)(7416002)(38100700002)(38070700005)(122000001)(71200400001)(478600001)(83380400001)(54906003)(64756008)(26005)(76116006)(316002)(6506007)(66446008)(6916009)(66946007)(66476007)(7696005)(66556008)(107886003)(55016003)(9686003);DIR:OUT;SFP:1102;","x-ms-exchange-antispam-messagedata-chunkcount":"1","x-ms-exchange-antispam-messagedata-0":"\n LefwKnJ+EGKneedzR1hgn4RIiAdEjlbUH/EaUTlXK56YXOCoxkMjBe629dj1B7ydS0OkeH9FJ73Co+MI8Ljg0zo+9fZzran8uDllixbrt2qtFmoilBYBLD7iU0Nq6DNVS445NSUdbHKgzeHdOH3RSx61gEh5nstznXhnUrkIfV0ra6WKxzG3T17956I5T7Nblel1YTKE38dWS8qxHGgPyK86ujLkVqZ9ELIvSxfHwR45DtASYkWBBzZH79ejjGIuOG3eBy6fAKcRJxk7/GUB05hDwPq7aMQUt91hm7xKF3n3qY++frVcuEVIXQ7fuVvqSa9SGNDT9/DxZfPx88BuvrJ5zGFC8CM3Rvl4UanAH6i6QhX1C8F/a7yY1uwL4UT17uesl4m1ndloEZc5/HumcNZaawkwN9efW/ckdH1hUOPdQD92bkpPa1+mhI8jHbShaKhhL3tAMIkKJjKmNoK4For93wFG9NtDLzslWR8jAlEuAjSsHudTw38PNGfbZwZu1ZVhD5uTFn+GIklqSIvqhUNHcK/vVuwqpc++xhp5bxqAWs+lmbymzTUIc1tSXl3py8X4qrsI0q9EpVcZAcK+m3CwB7IdUtmktFCEnYmCqf8j2Pl2+VfIKGfv1LoneenX9UMQkRiXS9jL7xLpDZIoZ9O/MTdrycAtMMqbuBBPCGMNcENEcDkCwExd57h4ZCoMLtRA8DP4uPoLr8XdIoamxCVIRsARd1rp45NEKItICw5DUSwur4paO7NmdKQ87p3cFCbvqzfhbqAn01ns+RnHvVB+upUQxu/3sSzbznIWvuaVl7J+V7EDL9oo8u9bnbOxG8bVAJH4KTIGONoi+lS39g7diyarNCFr6YG/iclrQ1wdb2BeZOHn8NNXGIU/kC4RhdhZ5ufBqfsbtDh0A4dBvX61e8nUjoYZ+sVrIShtfLMYLofUEPWOVTK/rcF3SLw2OnqqnCiBitpZOHegSm9S6h2s1FmAm59LBsnKzfyEO+0Ppd0SElEp62JDxPCAhY/nD+KpqOFzQeTyTuS0ZGJ92D/nVDPbCNpCZkZCrvD4JsyIz1ZmbxwrCnrK4eH/AE6LGne6ZgskpBj8PR+8NxyozkGqE6ylsTMnXggfpWT+dYnBME635LlpPf6D6SiTFd8zrCt9H5W8fMEX00KY4zHr9vZF7QpiQKbQVqjdC+vGlIxS81orJJbkCF0p1n2qCXEI+NfPHpFYzZ87XgPrcCBL+pGEjym+qqLiMzbowjExXxJ6oRSfEqeZUCsL28kJeq+Y+dxaPEgabrR81Y2VvSPl1mk8kug4Hu+EycVm04ZB4M64IvjDKLbowEC8qYwSj7esu0i3GVtz7lGy7gdnRZJ2gIUDK8rBYjGDUUqwsMsuQMXxtXu4PwJkdbq9GMXGf8yaCDPG2axpxtV/cwvNFvWG1r+bqVA9CK+/mbcNr5BSaMS2SYg9bLjRkLKnC5wtwQkV3P6LvxrOmi1O73KVcggstCCGUnQPJBWYQCYlOh4eyFk7Xy3bQHDrc+yalotqOEaziMZNhzbSSvm5iPa6OYCMF2OYpURmxu+0ZzeFzmKyAii8nMlF44aOvQIyptRdfoR60oKasziOckOXtHjhijwM4w==","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"quoted-printable","Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","X-OriginatorOrg":"renesas.com","X-MS-Exchange-CrossTenant-AuthAs":"Internal","X-MS-Exchange-CrossTenant-AuthSource":"TYWPR01MB8775.jpnprd01.prod.outlook.com","X-MS-Exchange-CrossTenant-Network-Message-Id":"\n b1d46e4c-2138-4fbc-7d45-08dbbf3db7d7","X-MS-Exchange-CrossTenant-originalarrivaltime":"27 Sep 2023 09:40:00.1203\n (UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"53d82571-da19-47e4-9cb4-625a166a4a2a","X-MS-Exchange-CrossTenant-mailboxtype":"HOSTED","X-MS-Exchange-CrossTenant-userprincipalname":"\n auBuol/c5PMMub7kRkLMA402m90xnz6yZD+CzUgJ1bPpkCT8zhAUdCTlFFg7jwB7hCApcdZioGYjnGlrko7/L4y9HjPDkXL6D+YawA42lkY=","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"TYWPR01MB10981","X-Spam-Status":"No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,\n\tDKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,\n\tRCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham\n\tautolearn_force=no version=3.4.6","X-Spam-Checker-Version":"SpamAssassin 3.4.6 (2021-04-09) on\n\tlindbergh.monkeyblade.net"}},{"id":3188599,"web_url":"http://patchwork.ozlabs.org/comment/3188599/","msgid":"","list_archive_url":null,"date":"2023-09-27T09:43:38","subject":"RE: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":80106,"url":"http://patchwork.ozlabs.org/api/people/80106/","name":"Fabrizio Castro","email":"fabrizio.castro.jz@renesas.com"},"content":"Hi Geert,\n\nThanks for your reply!\n\n> From: Geert Uytterhoeven \n> Subject: Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n> properties\n> \n> Hi Mark,\n> \n> On Wed, Sep 27, 2023 at 11:00 AM Mark Brown wrote:\n> > On Wed, Sep 27, 2023 at 09:59:05AM +0200, Geert Uytterhoeven wrote:\n> > > On Tue, Sep 26, 2023 at 11:08 PM Fabrizio Castro\n> > > > + type: boolean\n> > > > + description:\n> > > > + Use CSI Slave Selection (SS) pin to enable transmission\n> and reception when\n> > > > + in slave mode.\n> >\n> > > Can't this be done in a more generic way? I had expected that the\n> > > existing SPI_NO_CS flag can be set using a property in the \"slave\"\n> subnode,\n> > > but apparently there is no \"spi-no-cs\" property defined yet.\n> >\n> > The description is clearly saying there is a chip select, _NO_CS\n> seems\n> > entirely inappropriate. It's not specified in the device tree\n> because\n> > when there's no chip select for a device it's a fundamental property\n> of\n> > how the device is controlled and we don't need any information\n> beyond\n> > the compatible.\n> \n> In host mode, it indeed doesn't matter, as you can have only a single\n> device connected with SPI_NO_CS.\n> In device mode, the device needs to know if it must monitor the chip\n> select line or not.\n> \n> In hindsight, I should have kept the question I had written initially,\n> but deleted after having read the documentation for the corresponding\n> RZ/V2M register bits:\n> \n> What does it mean if this is false? That there is no chip select?\n\nYes, that's what it means. The IP would just use the clock line to\nshift data in and out.\nClearly, that could lead to \"misunderstandings\" as changing\nthe active level of the clock line on the host side may lead to \nextra clock cycles interpreted by the slave.\n\n> \n> So \"spi-no-cs\" would be the inverse of \"renesas,csi-ss\".\n\nYes, exactly.\n\nCheers,\nFab\n\n> \n> Gr{oetje,eeting}s,\n> \n> Geert\n> \n> --\n> Geert Uytterhoeven -- There's lots of Linux beyond ia32 --\n> geert@linux-m68k.org\n> \n> In personal conversations with technical people, I call myself a\n> hacker. But\n> when I'm talking to journalists I just say \"programmer\" or something\n> like that.\n> -- Linus Torvalds","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=renesas.com header.i=@renesas.com header.a=rsa-sha256\n header.s=selector1 header.b=FVjWTYoJ;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3753-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","dkim=none (message not signed)\n header.d=none;dmarc=none action=none header.from=renesas.com;"],"Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org\n [IPv6:2604:1380:45e3:2400::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwWs12wGqz1yp0\n\tfor ;\n Wed, 27 Sep 2023 19:43:49 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id EC0F9281EAC\n\tfor ; Wed, 27 Sep 2023 09:43:47 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 3A35926282;\n\tWed, 27 Sep 2023 09:43:45 +0000 (UTC)","from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net\n [23.128.96.19])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 82F4720B22\n\tfor ; Wed, 27 Sep 2023 09:43:43 +0000 (UTC)","from JPN01-OS0-obe.outbound.protection.outlook.com\n (mail-os0jpn01on2136.outbound.protection.outlook.com [40.107.113.136])\n\tby lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2C32DF3;\n\tWed, 27 Sep 2023 02:43:41 -0700 (PDT)","from TYWPR01MB8775.jpnprd01.prod.outlook.com (2603:1096:400:169::11)\n by OS7PR01MB11454.jpnprd01.prod.outlook.com (2603:1096:604:246::12) with\n Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6838.21; Wed, 27 Sep\n 2023 09:43:38 +0000","from TYWPR01MB8775.jpnprd01.prod.outlook.com\n ([fe80::600a:9708:18f:2528]) by TYWPR01MB8775.jpnprd01.prod.outlook.com\n ([fe80::600a:9708:18f:2528%3]) with mapi id 15.20.6838.016; Wed, 27 Sep 2023\n 09:43:38 +0000"],"ARC-Seal":"i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=DrKcVoNu4XA40rQns0uLvSAQmyexPwEh1H7P6/yWoMQ/iuBi0YTg7WsgtkFI5A5vnGkXysa6EaeLRRkBDFsDO2QewjbyilhY6ssSMCvLuFaxrS2+O0aClm2y5mCI1ov85E2nwwRT2Cg1V4QaXKWlncpDq2YvhuIOXt1GxehDRwLatcomqj4b9S8cz/Zd7Al40WTq9cqILFF5kxsQHD7TEdDp8Umpbrhz9QP31SN+OteLnhZIXfmjjNTuDVCTzbAFETNYvD1qCyQlQNLPKhLtvx7nfJNcW9vzeuDR8iSo6tdVAqI1i0kpSZrlTjNEMQaIa1en3YxGXfele9iO1M31pA==","ARC-Message-Signature":"i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector9901;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;\n bh=I9roojSoPrcpkWZ2814mbkl8QGDYXYGwNq44Q8Ye3hU=;\n b=PX5Fng2hgxZtTzj4BcVxVPmMQs0sx36qGbeWkcgcW4X/hNb6F57Y2NYlk+yt+qWGlLm+3qtOWMaJDx6s+d8FsyPCzQqkXeIWsTCd3WZaR4tprwUOACmRL2JoWMM4NjF8e/vamfGR2w6k7Sqbkrg7zFth20IoBhvOENoBL9xSETVPBIPxI1zGl75sYbN5ZUfmMO89PS2k4Jt9trfzQOELroDjH6xXWTS/S1ktlevIpwVNPrsWFjFjA+1PLeFSt9LSVQMGLyypiN2GOynASs42xL6ShLVeaxvoj7ho2BEH63BUKI1W7cr+omP//vxR5Y6MMb3svLOLeKaUbopN83641g==","ARC-Authentication-Results":"i=1; mx.microsoft.com 1; spf=pass\n smtp.mailfrom=renesas.com; dmarc=pass action=none header.from=renesas.com;\n dkim=pass header.d=renesas.com; arc=none","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=renesas.com;\n s=selector1;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=I9roojSoPrcpkWZ2814mbkl8QGDYXYGwNq44Q8Ye3hU=;\n b=FVjWTYoJ3bymR6neyPpv0bjTqWO/ZFKLof9UTBBRHR2muBHMlfRbidKnn0auPlSNmqnscbfX5Ofw0VOXxW7jshI9xqmlnpqH/AyLtUyL9iVLn9iV3fc7/R3SD7WkA9r1zCH0OtVfTGusAKoVKW30aljZ9MfYpwnbbgfrtwkVO1Y=","From":"Fabrizio Castro ","To":"Geert Uytterhoeven , Mark Brown ","CC":"Rob Herring , Krzysztof Kozlowski\n\t, Conor Dooley ,\n\tMagnus Damm , \"linux-spi@vger.kernel.org\"\n\t, \"devicetree@vger.kernel.org\"\n\t, \"linux-kernel@vger.kernel.org\"\n\t, \"linux-renesas-soc@vger.kernel.org\"\n\t, Chris Paterson\n\t, Biju Das , Prabhakar\n Mahadev Lad ","Subject":"RE: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Thread-Topic":"[PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Thread-Index":"AQHZ8L2blakiXIKx8k6DD60xX0IJ3LAuT32AgAAREwCAAAMCAIAACF3w","Date":"Wed, 27 Sep 2023 09:43:38 +0000","Message-ID":"\n ","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>\n \n \n ","In-Reply-To":"\n ","Accept-Language":"en-GB, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","authentication-results":["legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=renesas.com header.i=@renesas.com header.a=rsa-sha256\n header.s=selector1 header.b=FVjWTYoJ;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3753-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","dkim=none (message not signed)\n header.d=none;dmarc=none action=none header.from=renesas.com;"],"x-ms-publictraffictype":"Email","x-ms-traffictypediagnostic":"TYWPR01MB8775:EE_|OS7PR01MB11454:EE_","x-ms-office365-filtering-correlation-id":"3a749a8a-6005-455e-4cec-08dbbf3e39ba","x-ms-exchange-senderadcheck":"1","x-ms-exchange-antispam-relay":"0","x-microsoft-antispam":"BCL:0;","x-microsoft-antispam-message-info":"\n GLpNYCXAVzgCXA1RthZb7sIvJ2cojwb0mSIdDciWBhqqpKRN5CLaGE70cUHXKc8k/zByOHAdtlVZ3YcWv8rcGGkRwY1E7kHEIHKAo5i7tmwHI9PUkf1eoPy6V/DIzGbO45zINGXB0/2OFll9H0XPUNm0k5CGqhizBO7rQ+oCxllItxLXjko2Uth8mpTyBYHcm5lLsHY8RVDCSrROW/y4ybgT3gybwWzYd7+HAuw/hrIQHVVbyiz07/QhiUxrHjGvkHOQlEYlvanBcWSyaUDRJtUDql4irk8tlEfH3YQy9WfRhBiP2JYVrwJigeMi4bItfACqLlSho1/Pf5DMD7CWMu60bEzR33DZFMSCB6UCQF8kibXYa9IsvspocHOTeJmE6BiK6dtrM4U+zjquni+/ngG74D1bN+PP0Tc6kYs5fkImVq6hyO+fQ3KOaNGAEhZmB4Uks+RLreskggLJQIDUgmHtE4Koc9eaAi33byC6zVmaMgnhpnNW9k6iYcWFhzm7+Xj2j0PDmPbzPWtpB2S/loX4YKQI1O4I2sCKDta3yoEgOaoJOqOB/++8Yl6lwBO6Wlg322Cf4tuTpooCg4OCEFP1edZMCn+W6XcNiNRB6U17cdrL7tToThWHHHjVVrlZ","x-forefront-antispam-report":"\n CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TYWPR01MB8775.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(346002)(39860400002)(366004)(396003)(376002)(136003)(230922051799003)(451199024)(1800799009)(186009)(2906002)(8936002)(7416002)(54906003)(8676002)(52536014)(41300700001)(66476007)(5660300002)(316002)(4326008)(64756008)(66946007)(66556008)(66446008)(76116006)(110136005)(478600001)(71200400001)(6506007)(7696005)(53546011)(9686003)(107886003)(26005)(83380400001)(55016003)(86362001)(122000001)(33656002)(38070700005)(38100700002);DIR:OUT;SFP:1102;","x-ms-exchange-antispam-messagedata-chunkcount":"1","x-ms-exchange-antispam-messagedata-0":"=?utf-8?q?89oMkc5x7SZGaNEAXYaaWnJPISTJ?=\n\t=?utf-8?q?i5uv5XFWRs/zIebN26Nfid+i/2LNaAkn5Hd/uuoJDg8hLMeOe815lrfQZh+zPFeJM?=\n\t=?utf-8?q?Ax+YfGzJr/E4++2eiIlkO3vEBKgW/GlDSa5W5Ebmn5+RoJRje38HxysSUVVHmXOYl?=\n\t=?utf-8?q?/pU7Z+KgHqNjZX9vY5botJ+fqfB/Mj3va8DqBw+Mh2f5gY4euXSfNlp0/HpKCw/nJ?=\n\t=?utf-8?q?2Au6PUq5q0vGdfPRSytvIjg5C8kTahg6EAtntHCiEdm1JIeFhwPViKWXDMMD6GPqk?=\n\t=?utf-8?q?veKQSDjoePdZqMqRk1zlwtfIf5UyNk2ieckVXJZKdUUbZA2yefI7KPH0ra95MuDZN?=\n\t=?utf-8?q?BVz+Nlei4wIZVOayIcSti0QKGPlwN3lb3zV7X+eOjRAdlj+RlbWAIXgITpeBb5cGQ?=\n\t=?utf-8?q?NDfJwreA4uKC7/8C4mU/dYPMKuWbAxC8GNdvwvR0V+/ML+MIH4SkT0MrPt3r9nMUz?=\n\t=?utf-8?q?Iiu7FkJkjPnxVoH5bor/2SVBBxBIdIJPP7KkklZm/a1j9dOxM2orbCAydoMuLLUrI?=\n\t=?utf-8?q?mXOUtNjtxSzP8vfx8LXKWvNMBek3sWlwudm0+MWQYRIQniZu6jcRXHqkYVcn3DFev?=\n\t=?utf-8?q?uVQ7tBmaXpHmxQoZb/S6jmY4LYpIylBWys+M3JnfaH90sRntrbdKptUANmLmq3+UT?=\n\t=?utf-8?q?uV7XPqAIyr5B0rffwy+vY9CbGUFUrDL8cWFmN1KE0iu0Px1EObinwrqqOsBZvwQXA?=\n\t=?utf-8?q?uNN9hMPh49fipecNBllf3tgxlyx/veA5RWakPWD0ar6w83HIGnN9y/hsSaacOcddY?=\n\t=?utf-8?q?zScUPB7EylYwZ5/rJA1Pv9ufwWDlrm1Gmb0FrTnwhya8GEZ3i366xz869CoKeJUcy?=\n\t=?utf-8?q?OBh4c3BdaejUfgkWUxKers3weM1McYczb0Eo0nDRzK7ERrudDdmnIJZeKzba9MZfh?=\n\t=?utf-8?q?hxb0YD9xSP3f3IdZ5fsI4DL7J9dpK/tMgl6kIwU9k8RPMEoENs00h157lnY3AQmF5?=\n\t=?utf-8?q?yccEUiRo718mm4SzGQX9AnbULbAhGci4fXjW4Sz2j+wTwmccZKWoZWG9dFOTicERH?=\n\t=?utf-8?q?wHWVbq+y90SYrRcF2SjPUutaKkkuUlq6WlpgIy4L6xtlkiYmf2/JbDLQ742QRhBMn?=\n\t=?utf-8?q?NACNd3rV6e6IQxNkvShHiWerbYsbMtLmzNA4NyJdM+BwDGZe9SzkEnSHHro7LDOT/?=\n\t=?utf-8?q?GVz1w9CN/jtRAbVKe0PyoHNwwcYtz1Nju2Jgo8D3jdMtF6qAgJ0MiNpXXFbnKZ7jG?=\n\t=?utf-8?q?Yi030P2jyvlPwhY2OOjduS2Me+AOobbdWd85IoQsJLyySfcLgxLqAxEgMfV/ijgoh?=\n\t=?utf-8?q?aBhmKBr6VU5E8LH77syfqfealBviCoF/tWnUMNj/V5mOPgcEd2irh0oue3vWubbdR?=\n\t=?utf-8?q?TLckBdzMFysyj8lKm7efbVgzCexbCwMfQ54AimfJfvlt/UtyZOl1Jtnsoyy/oK/58?=\n\t=?utf-8?q?TdNw4dStAe4ItDw0YEEFwGGIJnFHC7gUW9cDOQK1cctkQ7OmHGuGmUCC/aZyhDu87?=\n\t=?utf-8?q?A4IWB7IjWjCRaxy8+wdEbDbQL4XFN7kByEcjTkBzKtStRMvO+6r0gujztOdVsD7Vh?=\n\t=?utf-8?q?y+P8QxdRlURN5niMqibJmFkuY4d5Xmogaw=3D=3D?=","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","X-OriginatorOrg":"renesas.com","X-MS-Exchange-CrossTenant-AuthAs":"Internal","X-MS-Exchange-CrossTenant-AuthSource":"TYWPR01MB8775.jpnprd01.prod.outlook.com","X-MS-Exchange-CrossTenant-Network-Message-Id":"\n 3a749a8a-6005-455e-4cec-08dbbf3e39ba","X-MS-Exchange-CrossTenant-originalarrivaltime":"27 Sep 2023 09:43:38.0412\n (UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"53d82571-da19-47e4-9cb4-625a166a4a2a","X-MS-Exchange-CrossTenant-mailboxtype":"HOSTED","X-MS-Exchange-CrossTenant-userprincipalname":"\n IoeRHKY5zurVeFu8S6qBfu+C1G7zOUUtbZaLp6AibTf8G++RpeTZmxgh9mY1fiZCnjKXy/m+saXoGcu663qnbNMaaXXHOHwJ+am4Zl4ffz0=","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"OS7PR01MB11454","X-Spam-Status":"No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,\n\tDKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,\n\tRCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham\n\tautolearn_force=no version=3.4.6","X-Spam-Checker-Version":"SpamAssassin 3.4.6 (2021-04-09) on\n\tlindbergh.monkeyblade.net"}},{"id":3188600,"web_url":"http://patchwork.ozlabs.org/comment/3188600/","msgid":"","list_archive_url":null,"date":"2023-09-27T09:44:17","subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":703,"url":"http://patchwork.ozlabs.org/api/people/703/","name":"Geert Uytterhoeven","email":"geert@linux-m68k.org"},"content":"Hi Mark,\n\nOn Wed, Sep 27, 2023 at 11:21 AM Mark Brown wrote:\n> On Wed, Sep 27, 2023 at 11:10:58AM +0200, Geert Uytterhoeven wrote:\n> > On Wed, Sep 27, 2023 at 11:00 AM Mark Brown wrote:\n>\n> > > The description is clearly saying there is a chip select, _NO_CS seems\n> > > entirely inappropriate. It's not specified in the device tree because\n> > > when there's no chip select for a device it's a fundamental property of\n> > > how the device is controlled and we don't need any information beyond\n> > > the compatible.\n>\n> > In host mode, it indeed doesn't matter, as you can have only a single\n> > device connected with SPI_NO_CS.\n> > In device mode, the device needs to know if it must monitor the chip\n> > select line or not.\n>\n> > In hindsight, I should have kept the question I had written initially,\n> > but deleted after having read the documentation for the corresponding\n> > RZ/V2M register bits:\n>\n> > What does it mean if this is false? That there is no chip select?\n>\n> > So \"spi-no-cs\" would be the inverse of \"renesas,csi-ss\".\n>\n> I see. Is there any control over what the chip select is when there is\n> one, in which case we could just look to see if there's one specified?\n\nOn RZ/V2M there isn't, as there is only a single hardware chip select.\n\nOn MSIOF, there are 3 hardware chip selects, but apparently only the\nprimary one can be used in target mode.\n\nI have to admit I never thought about this before (commit\ncf9e4784f3bde3e4 (\"spi: sh-msiof: Add slave mode support\") also predates\ncommit 9cce882bedd2768d (\"spi: sh-msiof: Extend support to 3 native chip\nselects\")). Hence the SPI target DT bindings use a single \"slave\" subnode,\nwithout a unit address, thus assuming no explicit (or a default)\nchip select configuration.\n\nGr{oetje,eeting}s,\n\n Geert","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":"legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3754-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org\n [IPv6:2604:1380:45e3:2400::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwWsy4Rjjz1yp0\n\tfor ;\n Wed, 27 Sep 2023 19:44:38 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id 55985281FBC\n\tfor ; Wed, 27 Sep 2023 09:44:37 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id C23ED2629A;\n\tWed, 27 Sep 2023 09:44:35 +0000 (UTC)","from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net\n [23.128.96.19])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 38D0920B22\n\tfor ; Wed, 27 Sep 2023 09:44:34 +0000 (UTC)","from mail-yw1-f179.google.com (mail-yw1-f179.google.com\n [209.85.128.179])\n\tby lindbergh.monkeyblade.net (Postfix) with ESMTPS id B6414E6;\n\tWed, 27 Sep 2023 02:44:32 -0700 (PDT)","by mail-yw1-f179.google.com with SMTP id\n 00721157ae682-59f55c276c3so98479607b3.2;\n Wed, 27 Sep 2023 02:44:32 -0700 (PDT)","from mail-yw1-f177.google.com (mail-yw1-f177.google.com.\n [209.85.128.177])\n by smtp.gmail.com with ESMTPSA id\n y4-20020a5b0f44000000b00d7e3e42d0c4sm259519ybr.53.2023.09.27.02.44.30\n (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128);\n Wed, 27 Sep 2023 02:44:30 -0700 (PDT)","by mail-yw1-f177.google.com with SMTP id\n 00721157ae682-59f6041395dso93370477b3.1;\n Wed, 27 Sep 2023 02:44:30 -0700 (PDT)"],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20230601; t=1695807872; x=1696412672;\n h=content-transfer-encoding:cc:to:subject:message-id:date:from\n :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc\n :subject:date:message-id:reply-to;\n bh=bCBtKjtOUbRzWwc+gk7sBNQfwK7ouTGHrKXSYvXMAhM=;\n b=F3+bmwXyNVKPf8Ix/anOjpY2mvIV3Ifb5c4pmdwpg8fvzBXpkC9tAspFKJPagrmqvx\n jh5nO0S4XNRv5bg7Lqj1FYBlMWCSyGYqZS6QS9QRHGzBC6xLsP3/isN5ArnhAtd/kq9Q\n dicqZAe4TAsbZHtoaNvlxt5OSaYtVXE/TnZWgHK5woiL9Ko5dfEf1w0ekzd1yAACYjZB\n prOhMjTl26TJxbuHRe7ENdraZQ4JKe4Ti/wYpeKdebjTmBy+fcFpy2z0N2IOMuBSd4cQ\n 4HHrxsQ8HQGol4u9m3khgnWG9HSinPFzMePylpplPAvm5rCfo62VBCSjb12AmP6cu7bA\n BftA==","X-Gm-Message-State":"AOJu0YxokLw9wILe2gbfW7QQoOZ5t9tWuZ3l/GciS+0Epvv9w2LCYUOV\n\tqU6s/ZdC6hFqSAPPpQq3e3FYfHMEi8+8Rg==","X-Google-Smtp-Source":"\n AGHT+IF8ltCzWRnIznlHP/+1Jk/toif12fLxW6sw3FLMWQs7FntXvo06tTxvGSiGNr1ZfOdE6ijbng==","X-Received":["by 2002:a25:c287:0:b0:d86:b9d5:124e with SMTP id\n s129-20020a25c287000000b00d86b9d5124emr1374845ybf.18.1695807871771;\n Wed, 27 Sep 2023 02:44:31 -0700 (PDT)","by 2002:a81:80c1:0:b0:59f:4bc3:3e9 with SMTP id\n q184-20020a8180c1000000b0059f4bc303e9mr1707738ywf.46.1695807870658; Wed, 27\n Sep 2023 02:44:30 -0700 (PDT)"],"Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>\n \n \n \n ","In-Reply-To":"","From":"Geert Uytterhoeven ","Date":"Wed, 27 Sep 2023 11:44:17 +0200","X-Gmail-Original-Message-ID":"\n ","Message-ID":"\n ","Subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","To":"Mark Brown ","Cc":"Fabrizio Castro ,\n Rob Herring ,\n\tKrzysztof Kozlowski ,\n Conor Dooley ,\n\tMagnus Damm , linux-spi@vger.kernel.org,\n\tdevicetree@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-renesas-soc@vger.kernel.org,\n\tChris Paterson ,\n Biju Das ,\n\tLad Prabhakar ","Content-Type":"text/plain; charset=\"UTF-8\"","Content-Transfer-Encoding":"quoted-printable","X-Spam-Status":"No, score=-1.4 required=5.0 tests=BAYES_00,\n\tFREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,\n\tRCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_PASS\n\tautolearn=no autolearn_force=no version=3.4.6","X-Spam-Checker-Version":"SpamAssassin 3.4.6 (2021-04-09) on\n\tlindbergh.monkeyblade.net"}},{"id":3188615,"web_url":"http://patchwork.ozlabs.org/comment/3188615/","msgid":"","list_archive_url":null,"date":"2023-09-27T10:05:24","subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":24878,"url":"http://patchwork.ozlabs.org/api/people/24878/","name":"Mark Brown","email":"broonie@kernel.org"},"content":"On Wed, Sep 27, 2023 at 11:44:17AM +0200, Geert Uytterhoeven wrote:\n> On Wed, Sep 27, 2023 at 11:21 AM Mark Brown wrote:\n\n> > I see. Is there any control over what the chip select is when there is\n> > one, in which case we could just look to see if there's one specified?\n\n> On RZ/V2M there isn't, as there is only a single hardware chip select.\n\n> On MSIOF, there are 3 hardware chip selects, but apparently only the\n> primary one can be used in target mode.\n\nOK, it sounds like we do need a property then. Like I say I'd rather\nnot have one that just works for _NO_CS in order to avoid confusion for\npeople writing SPI device drivers, either something in the generic\ntarget binding or a device specific one.","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=R68dpNLM;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3763-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)"],"Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org\n [IPv6:2604:1380:45e3:2400::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwXL230tLz1yp8\n\tfor ;\n Wed, 27 Sep 2023 20:05:30 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id C022F2829AB\n\tfor ; Wed, 27 Sep 2023 10:05:28 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id CB344273CF;\n\tWed, 27 Sep 2023 10:05:27 +0000 (UTC)","from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org\n [10.30.226.201])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id A12901F95A\n\tfor ; Wed, 27 Sep 2023 10:05:27 +0000 (UTC)","by smtp.kernel.org (Postfix) with ESMTPSA id AA3D2C433C7;\n\tWed, 27 Sep 2023 10:05:26 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org;\n\ts=k20201202; t=1695809127;\n\tbh=qnrP8nXG0KaxINYqeddDcSnJIQu51tz/Syj8SawhtkM=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=R68dpNLMbyY2PFKqG//gkhL0S9WMOiIJmABfcTzH+R3JKAUF6upvQaXIWnaqrUEw4\n\t bCfeBJjbBX3oA+6zTyJs9qxdoxoFuDiEDBOwY3bAWJ1iKsbuLXRT/1vYelgcK4Lt9g\n\t FSVFS1ndrTTZL7wiCnShwY/kRJ5eTvVE+pQCq8ONBhA/3QNm0gFRxtWqqDBaMJHlBB\n\t fYB/CzqOAnFN2oBTDLbwPPbRbT3gSRPLnTe+ciyASz6FDjSVexb7+n/c+dzot+imJB\n\t n2k/IP2NoLy7pyC0JgCi1ORtnrddd+8JD8Y/sMMN3ME9Ggp9n8hdSMYzCAu1AbMZvQ\n\t CN7Byr+An48Nw==","Date":"Wed, 27 Sep 2023 12:05:24 +0200","From":"Mark Brown ","To":"Geert Uytterhoeven ","Cc":"Fabrizio Castro ,\n\tRob Herring ,\n\tKrzysztof Kozlowski ,\n\tConor Dooley ,\n\tMagnus Damm , linux-spi@vger.kernel.org,\n\tdevicetree@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-renesas-soc@vger.kernel.org,\n\tChris Paterson ,\n\tBiju Das ,\n\tLad Prabhakar ","Subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Message-ID":"","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>\n \n \n \n \n ","Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","Content-Type":"multipart/signed; micalg=pgp-sha512;\n\tprotocol=\"application/pgp-signature\"; boundary=\"nmWLiBkxLUwi0tfY\"","Content-Disposition":"inline","In-Reply-To":"\n ","X-Cookie":"Save energy: Drive a smaller shell."}},{"id":3188627,"web_url":"http://patchwork.ozlabs.org/comment/3188627/","msgid":"","list_archive_url":null,"date":"2023-09-27T10:18:57","subject":"RE: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":80106,"url":"http://patchwork.ozlabs.org/api/people/80106/","name":"Fabrizio Castro","email":"fabrizio.castro.jz@renesas.com"},"content":"Hi Mark,\n\nThanks for your reply!\n\n> From: Mark Brown \n> Subject: Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n> properties\n> \n> On Wed, Sep 27, 2023 at 11:44:17AM +0200, Geert Uytterhoeven wrote:\n> > On Wed, Sep 27, 2023 at 11:21 AM Mark Brown \n> wrote:\n> \n> > > I see. Is there any control over what the chip select is when\n> there is\n> > > one, in which case we could just look to see if there's one\n> specified?\n> \n> > On RZ/V2M there isn't, as there is only a single hardware chip\n> select.\n> \n> > On MSIOF, there are 3 hardware chip selects, but apparently only the\n> > primary one can be used in target mode.\n> \n> OK, it sounds like we do need a property then. Like I say I'd rather\n> not have one that just works for _NO_CS in order to avoid confusion\n> for\n> people writing SPI device drivers, either something in the generic\n> target binding or a device specific one.\n\nShall I invert the logic then? What I mean is I could drop property\n\"renesas,csi-ss\" and add property \"renesas,csi-no-ss\" instead, therefore\nwithout \"renesas,csi-no-ss\" pin SS will be used, with \"renesas,csi-no-ss\"\npin SS won't be used.\nWhat do you think?\n\nAlso, I could drop \"renesas,csi-ss-high\" and use \"spi-cs-high\" instead?\n\nGeert, any thoughts on the above?\n\nThanks,\nFab","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=renesas.com header.i=@renesas.com header.a=rsa-sha256\n header.s=selector1 header.b=h/jSZiR3;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3785-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","dkim=none (message not signed)\n header.d=none;dmarc=none action=none header.from=renesas.com;"],"Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org\n [IPv6:2604:1380:45e3:2400::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwXdr0jTBz1ypJ\n\tfor ;\n Wed, 27 Sep 2023 20:19:12 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id 753F3281CAD\n\tfor ; Wed, 27 Sep 2023 10:19:09 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 9C897273F3;\n\tWed, 27 Sep 2023 10:19:07 +0000 (UTC)","from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net\n [23.128.96.19])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id F2CD71FD1\n\tfor ; Wed, 27 Sep 2023 10:19:05 +0000 (UTC)","from JPN01-OS0-obe.outbound.protection.outlook.com\n (mail-os0jpn01on2135.outbound.protection.outlook.com [40.107.113.135])\n\tby lindbergh.monkeyblade.net (Postfix) with ESMTPS id 318C64218;\n\tWed, 27 Sep 2023 03:19:02 -0700 (PDT)","from TYWPR01MB8775.jpnprd01.prod.outlook.com (2603:1096:400:169::11)\n by TYCPR01MB11071.jpnprd01.prod.outlook.com (2603:1096:400:3a7::14) with\n Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6792.27; Wed, 27 Sep\n 2023 10:18:59 +0000","from TYWPR01MB8775.jpnprd01.prod.outlook.com\n ([fe80::600a:9708:18f:2528]) by TYWPR01MB8775.jpnprd01.prod.outlook.com\n ([fe80::600a:9708:18f:2528%3]) with mapi id 15.20.6838.016; Wed, 27 Sep 2023\n 10:18:57 +0000"],"ARC-Seal":"i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=dHmCVc4p1hTdCAAmu59xjUznCsBPQ6KY3uE1k76Az3QUJHaplkLA4xRktJDc+ImAvbGK5Im05fD/yNcZWqpc8fcTn3OkhG91vJyBad1ZBK/ig8sbUgWIVtVMMn+faxu9Ki44ajQdzldmSt6x0rfXuuOBFDn5JqE+RkqjH1CQHUBy3oT2BN5D3v8jLF9Ne9OriBFuYWAe1sTBfgMpN37950cmrMW03UkLuVsxnYB+IUlI5l3MZbEPhyinV6xRzl8V6UaJ7pH63jVJRUd7vsqYHgbdWuFnX4qKR5gs6u5CoKfRpUgXVC2piV4O/9PG7Ktg/u9iU2c1eF/1FrtQC8ZxEg==","ARC-Message-Signature":"i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector9901;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;\n bh=06c27kec8Kz3dXyX2OKNZ+mujdjvb/lhLDR2W85lv8s=;\n b=ZQeUsHlLHPNQqtGcKg5lnGM0Auht8vqFiYpixC84S2o/iZcZDTXXdOP6afuxA829yBrSn9XbslkipsDZsRNWb1CxbKitZzefxxNTV6l3KX34a651uAkqs9fn1gfhLNPd+0tCIsigqU606qjE5Ydn8horoTVsMafHDWCTB3ArMhtj/L7wgaVv/UDYMvjoxzJH/0WyMU4dmvPeFz2fjU7dsxAQMwl7rrsTgqm8T1ijddlkRW+vZIhzs8LmFMz0cHxSmAIiDe3GpVJtGrHpsIa/tdg8XNUjFBglGZglHVyC/IR+J8yUtjObBw6tj3A3gVQ2w6lja84ial0jO86jafFcXw==","ARC-Authentication-Results":"i=1; mx.microsoft.com 1; spf=pass\n smtp.mailfrom=renesas.com; dmarc=pass action=none header.from=renesas.com;\n dkim=pass header.d=renesas.com; arc=none","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=renesas.com;\n s=selector1;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=06c27kec8Kz3dXyX2OKNZ+mujdjvb/lhLDR2W85lv8s=;\n b=h/jSZiR3QcXbYlVdQHR1TW0RDN0FBGA3zQ1d/rR+XMMGnfz9ocfS+C8Io/CXCFp246QZKRPua8OrCgPFSSQfnJX+6VmrNJbI1O0nabunKk4apDK5MWpatHqOTAs67qZ/N2JMZIEbe1l/BIIPRN7MAM7dIqHCxAt8WjeOtYCSltc=","From":"Fabrizio Castro ","To":"Mark Brown , Geert Uytterhoeven ","CC":"Rob Herring , Krzysztof Kozlowski\n\t, Conor Dooley ,\n\tMagnus Damm , \"linux-spi@vger.kernel.org\"\n\t, \"devicetree@vger.kernel.org\"\n\t, \"linux-kernel@vger.kernel.org\"\n\t, \"linux-renesas-soc@vger.kernel.org\"\n\t, Chris Paterson\n\t, Biju Das , Prabhakar\n Mahadev Lad ","Subject":"RE: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Thread-Topic":"[PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Thread-Index":"\n AQHZ8L2blakiXIKx8k6DD60xX0IJ3LAuT32AgAAREwCAAAMCAIAAAw4AgAAGQYCAAAXnAIAAAe0A","Date":"Wed, 27 Sep 2023 10:18:57 +0000","Message-ID":"\n ","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>\n \n \n \n \n \n ","In-Reply-To":"","Accept-Language":"en-GB, en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","authentication-results":["legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=renesas.com header.i=@renesas.com header.a=rsa-sha256\n header.s=selector1 header.b=h/jSZiR3;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3785-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","dkim=none (message not signed)\n header.d=none;dmarc=none action=none header.from=renesas.com;"],"x-ms-publictraffictype":"Email","x-ms-traffictypediagnostic":"TYWPR01MB8775:EE_|TYCPR01MB11071:EE_","x-ms-office365-filtering-correlation-id":"3f9604fd-20ca-4334-f2c5-08dbbf4328be","x-ms-exchange-senderadcheck":"1","x-ms-exchange-antispam-relay":"0","x-microsoft-antispam":"BCL:0;","x-microsoft-antispam-message-info":"\n TEdsqewl2vGYXW+Cpvk3hL8VrAE2Fqj8NEftGmx/gUwNyJSakSNaDDKXyD5Lo4RWUgMo+aZWLZTPMjnFHRToYpekMn4AP87Nx2+ajKqu++PVSfcbplajI4MA372R2VZ735wWb671vyyisux8bq2lDU4v1aUFQ5AdXMbtj2Zo8cd7hSpj325eSr47k9G72mLxhIQ7UyPM+9fmLHLpLy1kqX287EyVli0FmPXvU76/TRmf3H3lDOGdrTVXVUTZJdv5OxWIM6V4r66bccF1dlRAyrNW1FtPZD8/Mz2hzHUtL/ffITVO7su7m39uNrOWzY0DWB8NvEz/8Hg6SaHw98/PR+KafPvAarlvV2RP7dJxDuvUp9jv8rJ+7ftb0s2Q3TMNeukUpQMo2vipJazL0ypQ32K44eXiCVo9+P9dGjgC8wk5b22OyLXZENdCsIXV6Yiem7iBsxaxJsGQi2yBoCnSO9ihlfrmZsvTSbSD+nuIb9Nrkx6Si+gJ+70LLNnJXAdlNzLFE9jUB/tqM9KAtJGbSTFRFWb9sp25aOrcnZjozN9Bzjfnp1n6dQjJQfRpOSe62nQ2q5nVGblgCZAH90PrJez/UZWwO1gPiRyaJvf5QpfZADWO5jW5F9TDak3LgEIB","x-forefront-antispam-report":"\n CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TYWPR01MB8775.jpnprd01.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(136003)(39860400002)(396003)(346002)(376002)(366004)(230922051799003)(1800799009)(186009)(451199024)(5660300002)(107886003)(83380400001)(76116006)(26005)(54906003)(66446008)(9686003)(316002)(66476007)(7416002)(64756008)(41300700001)(8676002)(478600001)(52536014)(7696005)(55016003)(110136005)(71200400001)(66946007)(66556008)(2906002)(8936002)(6506007)(53546011)(4326008)(38070700005)(33656002)(122000001)(38100700002)(86362001);DIR:OUT;SFP:1102;","x-ms-exchange-antispam-messagedata-chunkcount":"1","x-ms-exchange-antispam-messagedata-0":"=?utf-8?q?5KjluxwU91vjNTaa54XihjYjmNK1?=\n\t=?utf-8?q?4TzlvVU0GR66YhvwndDXhYByholq6JDn7JmFw/23VD+Gl2dapO6FXuyPe70Ui9iJ3?=\n\t=?utf-8?q?Rldbir4NLXoCgCD8Wg4GUozOkrQ49wwIiOmqkEjL9Nev6Yq1Vw5d0ZcVspefUTXDf?=\n\t=?utf-8?q?GfM6ATpVJfrk2y1HErIRyh7qoHqF1zqBkPNfZoHqbWnHoxV6qSRqwwq9sOkSqjTn8?=\n\t=?utf-8?q?EgNMihovXY1Cc3NYHre1iWp79LPO8s42MgCavMDfFGYgEKN8KLlpseUZlZC761ESp?=\n\t=?utf-8?q?Cb01h4HqXg7MbB4PoAtqx8s94m/Qtw4naVxhIPd1OOLYF103OGJY2cVDv0wpTz5rG?=\n\t=?utf-8?q?MGDPUZ7OpYFnq3zMqyvbznhMRJZgp5x548hxNmTNA15o6zzPuYpUhdn+GbOZHOMUp?=\n\t=?utf-8?q?dD3BmUN16+Ta/14rffwHhEXTzO2iHa2tTHOCkw7RGr61xcwT1bMkMu7B86M9n5hts?=\n\t=?utf-8?q?ri2sJ6Yf15irv53p0bvqh5wDncC2F7gH/Jy4SvoxCuHPiZycoAylSWePBNWyi93Ip?=\n\t=?utf-8?q?Y6/W0s4P0GW7dYRKtbz2Gfz9/ELSYFdCbMvgsIsze5h1KSDACRdja5a7V0rgOJz0a?=\n\t=?utf-8?q?D/qAWOUNz+D/dFjSYwRfP2Hwka+i7hqExJtOUb58vVkLHe+P7dDrcWITXu5Tngy64?=\n\t=?utf-8?q?WcNQ1bfO1j2nts9IL5hVdPRlmNLi6y8eGT1NVnYjU4IZ3Mha3qVyGJnw/VlYNgYu6?=\n\t=?utf-8?q?0ZHrxay9RLsjorgwuVOe+IogYs4za0raO+Asv+4lw1ozmWNRexiK2MCE1PcKvyvjE?=\n\t=?utf-8?q?EGimUGs/ySR0J439mD0/YYF/I1mvVlxS+MsfGetvCC6mHqFm11noyELDtBGln6MGT?=\n\t=?utf-8?q?ZE/HyppkCQmq5vDcAybPel49rBXOSUOV+RXY0c+/BziIAiLbQo1rkz3zq2MPFrewq?=\n\t=?utf-8?q?RCbkFc4IWzYKNIDeZ/+21nRLga55BC9df+oviGmNzYvPw3s6DCavsCTrjzGTVmbYH?=\n\t=?utf-8?q?eppYSpP6OZCBrolhaOwyV7xNcTSVPUk74kQ16CP+HTo6xraygDSE3w7Cm+wCDszo6?=\n\t=?utf-8?q?B+ag/GkciAKzP5vhyKEwxHrPSQXnhoeTFegSErAMuubQqBLknjZ27v8vw3pi62t1d?=\n\t=?utf-8?q?6zJm5c4m+OYbF7Ku22lsgALnMNOGYQcoAT08tyBrC9P4L7ROJOkbJA9Z+OY4X3mqV?=\n\t=?utf-8?q?01tuyIi8K5N/HoG7fVdasK+RO5Pdcdf9XiMQYIjNRbd1bwRRTgPFDtrkbIJ97aa6m?=\n\t=?utf-8?q?byOAxd0vQ0g5UFqtg/88IemlRh3l6L5A708IcFb72IWP9LEhIcLGJ2vHyrFILmkhu?=\n\t=?utf-8?q?WL5Z+hSHpMmL9fp4UdpFHtbgEopnpCQ6RakFN/ZNB2FEboNtS435JzN1ENaOZ7V30?=\n\t=?utf-8?q?u7vr6YgYI061Rn+QXHjBKl4fp+z3WPTpesq7WZbqD78RTiIge/RXxbqy80GrgKtL4?=\n\t=?utf-8?q?xtNh4SC1rMt19WPZaTcI8Ts16aki6WSkjdn/iyt/0k9RiwZfSAJqGrbUCOjkIkZA5?=\n\t=?utf-8?q?XPUNr7P/VTJKTVr6o0P5FvXe/X1sUpnNQ9RcybCcUEa+NUd/klEKFvs7wK/qwjkGz?=\n\t=?utf-8?q?XfRhV5uuuIEqhXtRGGEjGGiDONNlJCkwbQ=3D=3D?=","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","X-OriginatorOrg":"renesas.com","X-MS-Exchange-CrossTenant-AuthAs":"Internal","X-MS-Exchange-CrossTenant-AuthSource":"TYWPR01MB8775.jpnprd01.prod.outlook.com","X-MS-Exchange-CrossTenant-Network-Message-Id":"\n 3f9604fd-20ca-4334-f2c5-08dbbf4328be","X-MS-Exchange-CrossTenant-originalarrivaltime":"27 Sep 2023 10:18:57.0301\n (UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"53d82571-da19-47e4-9cb4-625a166a4a2a","X-MS-Exchange-CrossTenant-mailboxtype":"HOSTED","X-MS-Exchange-CrossTenant-userprincipalname":"\n PBWBt1d3WmGdK93eCGDLJ+YFn/zNpzHLsftkfUGog7M8RcjHsTLkjFMUrVxxLictgEC/Rrnw3vnzyeBb447gnJuix/1dJ9Zjt7+xjjBjtbI=","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"TYCPR01MB11071","X-Spam-Status":"No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,\n\tDKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,\n\tRCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham\n\tautolearn_force=no version=3.4.6","X-Spam-Checker-Version":"SpamAssassin 3.4.6 (2021-04-09) on\n\tlindbergh.monkeyblade.net"}},{"id":3188630,"web_url":"http://patchwork.ozlabs.org/comment/3188630/","msgid":"","list_archive_url":null,"date":"2023-09-27T10:22:59","subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","submitter":{"id":24878,"url":"http://patchwork.ozlabs.org/api/people/24878/","name":"Mark Brown","email":"broonie@kernel.org"},"content":"On Wed, Sep 27, 2023 at 10:18:57AM +0000, Fabrizio Castro wrote:\n> > From: Mark Brown \n\n> > OK, it sounds like we do need a property then. Like I say I'd rather\n> > not have one that just works for _NO_CS in order to avoid confusion\n> > for\n> > people writing SPI device drivers, either something in the generic\n> > target binding or a device specific one.\n\n> Shall I invert the logic then? What I mean is I could drop property\n> \"renesas,csi-ss\" and add property \"renesas,csi-no-ss\" instead, therefore\n> without \"renesas,csi-no-ss\" pin SS will be used, with \"renesas,csi-no-ss\"\n> pin SS won't be used.\n> What do you think?\n\nThat sounds fine for me, I guess we could add a further property if some\nnew IP allows multiple options for the chip select in target mode.\n\n> Also, I could drop \"renesas,csi-ss-high\" and use \"spi-cs-high\" instead?\n\nI think that's OK but I looked less at that bit.","headers":{"Return-Path":"\n ","X-Original-To":["incoming-dt@patchwork.ozlabs.org","devicetree@vger.kernel.org"],"Delivered-To":"patchwork-incoming-dt@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256\n header.s=k20201202 header.b=aQ9ZoLYP;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=permerror (SPF Permanent Error: More than 10 MX records returned)\n smtp.mailfrom=vger.kernel.org (client-ip=2604:1380:45e3:2400::1;\n helo=sv.mirrors.kernel.org;\n envelope-from=devicetree+bounces-3787-incoming-dt=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)"],"Received":["from sv.mirrors.kernel.org (sv.mirrors.kernel.org\n [IPv6:2604:1380:45e3:2400::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature ECDSA (secp384r1))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4RwXky3G45z1ynX\n\tfor ;\n Wed, 27 Sep 2023 20:23:38 +1000 (AEST)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sv.mirrors.kernel.org (Postfix) with ESMTP id E87872826AA\n\tfor ; Wed, 27 Sep 2023 10:23:35 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id F396026E34;\n\tWed, 27 Sep 2023 10:23:33 +0000 (UTC)","from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org\n [10.30.226.201])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id CBC75F9CD\n\tfor ; Wed, 27 Sep 2023 10:23:33 +0000 (UTC)","by smtp.kernel.org (Postfix) with ESMTPSA id BAD11C433C7;\n\tWed, 27 Sep 2023 10:23:32 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org;\n\ts=k20201202; t=1695810213;\n\tbh=mJtkcoqAxjc3capOnsYp/Ti3db2yjrAid5Uj7rgquYY=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=aQ9ZoLYPAOcGdFlIzX7JdymQVx3fL2BNysG9Ry+JkQAVBnVz/pwO2aFE2WYKOAwrr\n\t k8mlfs9Ga7f6h1BC/igfCBLh9QrrIXnyjg83ujenCmhNour0TIbsK7viHMYNp0oO53\n\t b1CLzyZA2F1obO6tv5UDXUaohmt8ucVsgUVrcfB2CdZe7e0q5kPXCe8WxvS1pIgoeU\n\t CyhUDp4lxRMELKWNEOrUjiA753FvvRyYRJUPk6waiVtewxvFXwysd7D+U25AkY2Yly\n\t aDDx365GMO0SVNhlthr6fiEaAG7KBdz6FQEBujmrHTZ23ywUQ6FwPypbohm0RktKEL\n\t l6kiAZQAJnVLQ==","Date":"Wed, 27 Sep 2023 12:22:59 +0200","From":"Mark Brown ","To":"Fabrizio Castro ","Cc":"Geert Uytterhoeven ,\n\tRob Herring ,\n\tKrzysztof Kozlowski ,\n\tConor Dooley ,\n\tMagnus Damm ,\n\t\"linux-spi@vger.kernel.org\" ,\n\t\"devicetree@vger.kernel.org\" ,\n\t\"linux-kernel@vger.kernel.org\" ,\n\t\"linux-renesas-soc@vger.kernel.org\" ,\n\tChris Paterson ,\n\tBiju Das ,\n\tPrabhakar Mahadev Lad ","Subject":"Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related\n properties","Message-ID":"","References":"<20230926210818.197356-1-fabrizio.castro.jz@renesas.com>\n <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>\n \n \n \n \n \n \n ","Precedence":"bulk","X-Mailing-List":"devicetree@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","Content-Type":"multipart/signed; micalg=pgp-sha512;\n\tprotocol=\"application/pgp-signature\"; boundary=\"wJt9p1k0SYjlzPnn\"","Content-Disposition":"inline","In-Reply-To":"\n ","X-Cookie":"Save energy: Drive a smaller shell."}}]