2023-03-06 01:54:26

by Jaewon Kim

[permalink] [raw]
Subject: [PATCH] spi: s3c64xx: add no_cs description

This patch adds missing variable no_cs descriptions.

Signed-off-by: Jaewon Kim <[email protected]>
---
include/linux/platform_data/spi-s3c64xx.h | 1 +
1 file changed, 1 insertion(+)

diff --git a/include/linux/platform_data/spi-s3c64xx.h b/include/linux/platform_data/spi-s3c64xx.h
index 5df1ace6d2c9..3101152ce449 100644
--- a/include/linux/platform_data/spi-s3c64xx.h
+++ b/include/linux/platform_data/spi-s3c64xx.h
@@ -29,6 +29,7 @@ struct s3c64xx_spi_csinfo {
* struct s3c64xx_spi_info - SPI Controller defining structure
* @src_clk_nr: Clock source index for the CLK_CFG[SPI_CLKSEL] field.
* @num_cs: Number of CS this controller emulates.
+ * @no_cs: Used when CS line is not connected.
* @cfg_gpio: Configure pins for this SPI controller.
*/
struct s3c64xx_spi_info {
--
2.17.1



2023-03-06 06:26:46

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] spi: s3c64xx: add no_cs description

On 06/03/2023 02:42, Jaewon Kim wrote:
> This patch adds missing variable no_cs descriptions.
>
> Signed-off-by: Jaewon Kim <[email protected]>


Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof


2023-03-06 09:36:33

by Andi Shyti

[permalink] [raw]
Subject: Re: [PATCH] spi: s3c64xx: add no_cs description

Hi Jaewon,

On Mon, Mar 06, 2023 at 10:42:39AM +0900, Jaewon Kim wrote:
> This patch adds missing variable no_cs descriptions.
>
> Signed-off-by: Jaewon Kim <[email protected]>

Reviewed-by: Andi Shyti <[email protected]>

Thanks,
Andi

> ---
> include/linux/platform_data/spi-s3c64xx.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/include/linux/platform_data/spi-s3c64xx.h b/include/linux/platform_data/spi-s3c64xx.h
> index 5df1ace6d2c9..3101152ce449 100644
> --- a/include/linux/platform_data/spi-s3c64xx.h
> +++ b/include/linux/platform_data/spi-s3c64xx.h
> @@ -29,6 +29,7 @@ struct s3c64xx_spi_csinfo {
> * struct s3c64xx_spi_info - SPI Controller defining structure
> * @src_clk_nr: Clock source index for the CLK_CFG[SPI_CLKSEL] field.
> * @num_cs: Number of CS this controller emulates.
> + * @no_cs: Used when CS line is not connected.
> * @cfg_gpio: Configure pins for this SPI controller.
> */
> struct s3c64xx_spi_info {
> --
> 2.17.1

2023-03-30 13:44:14

by Jaewon Kim

[permalink] [raw]
Subject: Re: [PATCH] spi: s3c64xx: add no_cs description

Hello Mark


Could you apply this patch?

I missed you in the mail recipient.


https://lkml.org/lkml/2023/3/6/34

https://lkml.org/lkml/2023/3/6/232


On 23. 3. 6. 10:42, Jaewon Kim wrote:
> This patch adds missing variable no_cs descriptions.
>
> Signed-off-by: Jaewon Kim <[email protected]>
> ---
> include/linux/platform_data/spi-s3c64xx.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/include/linux/platform_data/spi-s3c64xx.h b/include/linux/platform_data/spi-s3c64xx.h
> index 5df1ace6d2c9..3101152ce449 100644
> --- a/include/linux/platform_data/spi-s3c64xx.h
> +++ b/include/linux/platform_data/spi-s3c64xx.h
> @@ -29,6 +29,7 @@ struct s3c64xx_spi_csinfo {
> * struct s3c64xx_spi_info - SPI Controller defining structure
> * @src_clk_nr: Clock source index for the CLK_CFG[SPI_CLKSEL] field.
> * @num_cs: Number of CS this controller emulates.
> + * @no_cs: Used when CS line is not connected.
> * @cfg_gpio: Configure pins for this SPI controller.
> */
> struct s3c64xx_spi_info {


Thanks

2023-03-30 14:49:34

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] spi: s3c64xx: add no_cs description

On 30/03/2023 15:40, Jaewon Kim wrote:
> Hello Mark
>
>
> Could you apply this patch?
>
> I missed you in the mail recipient.
>
>
> https://lkml.org/lkml/2023/3/6/34
>
> https://lkml.org/lkml/2023/3/6/232

I think you should resend. This time with entire output of
scripts/get_maintainers.pl

Best regards,
Krzysztof

2023-03-30 20:25:32

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH] spi: s3c64xx: add no_cs description

On Mon, 06 Mar 2023 10:42:39 +0900, Jaewon Kim wrote:
> This patch adds missing variable no_cs descriptions.
>
>

Applied to

broonie/spi.git for-next

Thanks!

[1/1] spi: s3c64xx: add no_cs description
commit: e3d53ded577328f2b26d361f2e62fc70e85ab6a3

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

2023-03-31 01:15:40

by Jaewon Kim

[permalink] [raw]
Subject: Re: [PATCH] spi: s3c64xx: add no_cs description

Hi Krzysztof


I always appreciate your kind guide.


I used 'get_maintainers.pl' but the Mark was missing.

Because the changes in this patch are only in
"include/linux/platform_data/".

There was no touch in SPI driver. So, script did not outputting the SPI
Maintainer.


Thanks

Jaewon Kim

On 23. 3. 30. 23:46, Krzysztof Kozlowski wrote:
> On 30/03/2023 15:40, Jaewon Kim wrote:
>> Hello Mark
>>
>>
>> Could you apply this patch?
>>
>> I missed you in the mail recipient.
>>
>>
>> https://protect2.fireeye.com/v1/url?k=62c6672d-034d7217-62c7ec62-74fe4860008a-45c729ef051007d6&q=1&e=03453742-6822-4fdc-9ee7-bdb6bdb0fe4f&u=https%3A%2F%2Flkml.org%2Flkml%2F2023%2F3%2F6%2F34
>>
>> https://protect2.fireeye.com/v1/url?k=41253bbb-20ae2e81-4124b0f4-74fe4860008a-f6ba8ee97b64390d&q=1&e=03453742-6822-4fdc-9ee7-bdb6bdb0fe4f&u=https%3A%2F%2Flkml.org%2Flkml%2F2023%2F3%2F6%2F232
> I think you should resend. This time with entire output of
> scripts/get_maintainers.pl
>
> Best regards,
> Krzysztof
>
>