[<prev] [next>] [day] [month] [year] [list]
Message-ID: <952720eb-e302-8a9e-a8ae-1ca7c3dd7e1c@infradead.org>
Date: Tue, 9 Nov 2021 21:41:38 -0800
From: Randy Dunlap <rdunlap@...radead.org>
To: Lh Kuo 郭力豪 <lh.Kuo@...plus.com>,
"LH.Kuo" <lhjeff911@...il.com>,
"p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
"broonie@...nel.org" <broonie@...nel.org>,
"robh+dt@...nel.org" <robh+dt@...nel.org>,
"linux-spi@...r.kernel.org" <linux-spi@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Cc: "dvorkin@...bo.com" <dvorkin@...bo.com>,
"qinjian@...lus1.com" <qinjian@...lus1.com>,
Wells Lu 呂芳騰 <wells.lu@...plus.com>
Subject: Re: [PATCH v2 1/2] SPI: Add SPI driver for Sunplus SP7021
On 11/9/21 9:39 PM, Lh Kuo 郭力豪 wrote:
> Hi
>
>> -----Original Message-----
>> From: Randy Dunlap <rdunlap@...radead.org>
>> Sent: Wednesday, November 10, 2021 12:55 AM
>> To: LH.Kuo <lhjeff911@...il.com>; p.zabel@...gutronix.de;
>> broonie@...nel.org; robh+dt@...nel.org; linux-spi@...r.kernel.org;
>> devicetree@...r.kernel.org; linux-kernel@...r.kernel.org
>> Cc: dvorkin@...bo.com; qinjian@...lus1.com; Wells Lu 呂芳騰
>> <wells.lu@...plus.com>; Lh Kuo 郭力豪 <lh.Kuo@...plus.com>
>> Subject: Re: [PATCH v2 1/2] SPI: Add SPI driver for Sunplus SP7021
>>
>> On 11/9/21 1:01 AM, LH.Kuo wrote:
>>> diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig index
>>> 596705d..30ce0ed 100644
>>> --- a/drivers/spi/Kconfig
>>> +++ b/drivers/spi/Kconfig
>>> @@ -866,6 +866,17 @@ config SPI_SUN6I
>>> help
>>> This enables using the SPI controller on the Allwinner A31 SoCs.
>>>
>>> +config SPI_SUNPLUS_SP7021
>>> + tristate "Sunplus SP7021 SPI controller"
>>> + depends on SOC_SP7021
>>> + help
>>> + This enable Sunplus SP7021 spi controller driver on the SP7021 SoCs.
>>
>> enables the Sunplus SP021 SPI
>>
>>> + This driver can also be built as a module. If so, the module will be
>>> + called as spi-sunplus-sp7021.
>>> +
>>> + If you have a Sunplus SP7021 platform say Y here.
>>
>> have a Sunplus
>> (i.e., drop one space)
>>
>>> + If unsure, say N.
>>
>>
>
> I will make change as below is it OK ?
>
> config SPI_SUNPLUS_SP7021
> tristate "Sunplus SP7021 SPI controller"
> depends on SOC_SP7021
> help
> This enable Sunplus SP7021 spi controller driver on the SP7021 SoCs.
enables SPI
> This driver can also be built as a module. If so, the module will be
> called as spi-sunplus-sp7021.
>
> If you have a Sunplus SP7021 platform say Y here.
drop one space: a Sunplus
>
> If unsure, say N.
>
>> --
>> ~Randy
--
~Randy
Powered by blists - more mailing lists