2021-03-10 15:04:05

by Xiaofeng Cao

[permalink] [raw]
Subject: [PATCH] Correct 'so'

In Kconfig it should be 'to'
In c8sectpfe-core.c it should be 'do'

Signed-off-by: Xiaofeng Cao <[email protected]>
---
drivers/media/Kconfig | 2 +-
drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/Kconfig b/drivers/media/Kconfig
index 6222b3ae220b..b07812657cee 100644
--- a/drivers/media/Kconfig
+++ b/drivers/media/Kconfig
@@ -134,7 +134,7 @@ config MEDIA_PLATFORM_SUPPORT
This is found on Embedded hardware (SoC), on V4L2 codecs and
on some GPU and newer CPU chipsets.

- Say Y when you want to be able so see such devices.
+ Say Y when you want to be able to see such devices.

config MEDIA_TEST_SUPPORT
bool
diff --git a/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c b/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
index a7a6ea666740..338b205ae3a7 100644
--- a/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
+++ b/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
@@ -655,7 +655,7 @@ static irqreturn_t c8sectpfe_error_irq_handler(int irq, void *priv)

/*
* TODO FIXME we should detect some error conditions here
- * and ideally so something about them!
+ * and ideally do something about them!
*/

return IRQ_HANDLED;
--
2.25.1


2021-03-10 17:21:52

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH] Correct 'so'

On 3/10/21 6:58 AM, Xiaofeng Cao wrote:
> In Kconfig it should be 'to'
> In c8sectpfe-core.c it should be 'do'
>
> Signed-off-by: Xiaofeng Cao <[email protected]>

The Subject: could use some work, but the patch contents look good.
Thanks.

Acked-by: Randy Dunlap <[email protected]>

> ---
> drivers/media/Kconfig | 2 +-
> drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/Kconfig b/drivers/media/Kconfig
> index 6222b3ae220b..b07812657cee 100644
> --- a/drivers/media/Kconfig
> +++ b/drivers/media/Kconfig
> @@ -134,7 +134,7 @@ config MEDIA_PLATFORM_SUPPORT
> This is found on Embedded hardware (SoC), on V4L2 codecs and
> on some GPU and newer CPU chipsets.
>
> - Say Y when you want to be able so see such devices.
> + Say Y when you want to be able to see such devices.
>
> config MEDIA_TEST_SUPPORT
> bool
> diff --git a/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c b/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
> index a7a6ea666740..338b205ae3a7 100644
> --- a/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
> +++ b/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
> @@ -655,7 +655,7 @@ static irqreturn_t c8sectpfe_error_irq_handler(int irq, void *priv)
>
> /*
> * TODO FIXME we should detect some error conditions here
> - * and ideally so something about them!
> + * and ideally do something about them!
> */
>
> return IRQ_HANDLED;
>


--
~Randy