Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) client-ip=23.128.96.32;
From:   Fabrizio Castro <fabrizio.castro.jz@renesas.com>
To:     Mark Brown <broonie@kernel.org>
CC:     Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        Geert Uytterhoeven <geert+renesas@glider.be>,
        Magnus Damm <magnus.damm@gmail.com>,
        "linux-spi@vger.kernel.org" <linux-spi@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-renesas-soc@vger.kernel.org" 
        <linux-renesas-soc@vger.kernel.org>,
        Chris Paterson <Chris.Paterson2@renesas.com>,
        Biju Das <biju.das@bp.renesas.com>,
        Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>
Subject: RE: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related
 properties
Thread-Topic: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related
 properties
Thread-Index: AQHZ8L2blakiXIKx8k6DD60xX0IJ3LAuX5YAgAALJcA=
Date:   Wed, 27 Sep 2023 09:40:00 +0000
Message-ID: <TYWPR01MB877515D00099BE7FB167605FC2C2A@TYWPR01MB8775.jpnprd01.prod.outlook.com>
References: <20230926210818.197356-1-fabrizio.castro.jz@renesas.com>
 <20230926210818.197356-2-fabrizio.castro.jz@renesas.com>
 <ZRPuSidGd5zO6pXf@finisterre.sirena.org.uk>
In-Reply-To: <ZRPuSidGd5zO6pXf@finisterre.sirena.org.uk>
Accept-Language: en-GB, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?LefwKnJ+EGKneedzR1hgn4RIiAdEjlbUH/EaUTlXK56YXOCoxkMjBe629dj1?=
 =?us-ascii?Q?B7ydS0OkeH9FJ73Co+MI8Ljg0zo+9fZzran8uDllixbrt2qtFmoilBYBLD7i?=
 =?us-ascii?Q?U0Nq6DNVS445NSUdbHKgzeHdOH3RSx61gEh5nstznXhnUrkIfV0ra6WKxzG3?=
 =?us-ascii?Q?T17956I5T7Nblel1YTKE38dWS8qxHGgPyK86ujLkVqZ9ELIvSxfHwR45DtAS?=
 =?us-ascii?Q?YkWBBzZH79ejjGIuOG3eBy6fAKcRJxk7/GUB05hDwPq7aMQUt91hm7xKF3n3?=
 =?us-ascii?Q?qY++frVcuEVIXQ7fuVvqSa9SGNDT9/DxZfPx88BuvrJ5zGFC8CM3Rvl4UanA?=
 =?us-ascii?Q?H6i6QhX1C8F/a7yY1uwL4UT17uesl4m1ndloEZc5/HumcNZaawkwN9efW/ck?=
 =?us-ascii?Q?dH1hUOPdQD92bkpPa1+mhI8jHbShaKhhL3tAMIkKJjKmNoK4For93wFG9NtD?=
 =?us-ascii?Q?LzslWR8jAlEuAjSsHudTw38PNGfbZwZu1ZVhD5uTFn+GIklqSIvqhUNHcK/v?=
 =?us-ascii?Q?Vuwqpc++xhp5bxqAWs+lmbymzTUIc1tSXl3py8X4qrsI0q9EpVcZAcK+m3Cw?=
 =?us-ascii?Q?B7IdUtmktFCEnYmCqf8j2Pl2+VfIKGfv1LoneenX9UMQkRiXS9jL7xLpDZIo?=
 =?us-ascii?Q?Z9O/MTdrycAtMMqbuBBPCGMNcENEcDkCwExd57h4ZCoMLtRA8DP4uPoLr8Xd?=
 =?us-ascii?Q?IoamxCVIRsARd1rp45NEKItICw5DUSwur4paO7NmdKQ87p3cFCbvqzfhbqAn?=
 =?us-ascii?Q?01ns+RnHvVB+upUQxu/3sSzbznIWvuaVl7J+V7EDL9oo8u9bnbOxG8bVAJH4?=
 =?us-ascii?Q?KTIGONoi+lS39g7diyarNCFr6YG/iclrQ1wdb2BeZOHn8NNXGIU/kC4RhdhZ?=
 =?us-ascii?Q?5ufBqfsbtDh0A4dBvX61e8nUjoYZ+sVrIShtfLMYLofUEPWOVTK/rcF3SLw2?=
 =?us-ascii?Q?OnqqnCiBitpZOHegSm9S6h2s1FmAm59LBsnKzfyEO+0Ppd0SElEp62JDxPCA?=
 =?us-ascii?Q?hY/nD+KpqOFzQeTyTuS0ZGJ92D/nVDPbCNpCZkZCrvD4JsyIz1ZmbxwrCnrK?=
 =?us-ascii?Q?4eH/AE6LGne6ZgskpBj8PR+8NxyozkGqE6ylsTMnXggfpWT+dYnBME635Llp?=
 =?us-ascii?Q?Pf6D6SiTFd8zrCt9H5W8fMEX00KY4zHr9vZF7QpiQKbQVqjdC+vGlIxS81or?=
 =?us-ascii?Q?JJbkCF0p1n2qCXEI+NfPHpFYzZ87XgPrcCBL+pGEjym+qqLiMzbowjExXxJ6?=
 =?us-ascii?Q?oRSfEqeZUCsL28kJeq+Y+dxaPEgabrR81Y2VvSPl1mk8kug4Hu+EycVm04ZB?=
 =?us-ascii?Q?4M64IvjDKLbowEC8qYwSj7esu0i3GVtz7lGy7gdnRZJ2gIUDK8rBYjGDUUqw?=
 =?us-ascii?Q?sMsuQMXxtXu4PwJkdbq9GMXGf8yaCDPG2axpxtV/cwvNFvWG1r+bqVA9CK+/?=
 =?us-ascii?Q?mbcNr5BSaMS2SYg9bLjRkLKnC5wtwQkV3P6LvxrOmi1O73KVcggstCCGUnQP?=
 =?us-ascii?Q?JBWYQCYlOh4eyFk7Xy3bQHDrc+yalotqOEaziMZNhzbSSvm5iPa6OYCMF2OY?=
 =?us-ascii?Q?pURmxu+0ZzeFzmKyAii8nMlF44aOvQIyptRdfoR60oKasziOckOXtHjhijwM?=
 =?us-ascii?Q?4w=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
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: b1d46e4c-2138-4fbc-7d45-08dbbf3db7d7
X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Sep 2023 09:40:00.1203
 (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: auBuol/c5PMMub7kRkLMA402m90xnz6yZD+CzUgJ1bPpkCT8zhAUdCTlFFg7jwB7hCApcdZioGYjnGlrko7/L4y9HjPDkXL6D+YawA42lkY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYWPR01MB10981
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Wed, 27 Sep 2023 02:40:18 -0700 (PDT)

Hi Mark,

Thanks for your reply!

> From: Mark Brown <broonie@kernel.org>
> Subject: Re: [PATCH 1/2] spi: renesas,rzv2m-csi: Add SPI Slave related
> properties
>=20
> On Tue, Sep 26, 2023 at 10:08:17PM +0100, Fabrizio Castro wrote:
> > The CSI IP found inside the Renesas RZ/V2M SoC can also work
> > in SPI slave mode.
> > When working in slave mode, the IP can make use of the SS
> > (Slave Select) pin, with "low" as default active level.
> > The active level of SS can be changed to "high" upon configuration.
> > This patch adds two new properties, one to make use of the
> > SS pin when in slave mode, and one to make the SS pin active high.
>=20
> Please avoid the use of outdated terminology like this, prefer "device
> mode" or similar.

Okay, I'll resend with a more modern terminology ("host" and "target").


>=20
> > +  renesas,csi-ss:
> > +    type: boolean
> > +    description:
> > +      Use CSI Slave Selection (SS) pin to enable transmission and
> reception when
> > +      in slave mode.
>=20
> When would this ever not be true when in device mode?


This IP can do without the SS pin to start operations. In fact, the
SS pin is disabled by default (I know...), it needs to be explicitly
enabled, and when enabled the pin is active low by default.

Cheers,
Fab