[PATCH v2] staging: media: fix spelling mistakes

Hridesh MG posted 1 patch 1 month, 3 weeks ago
drivers/staging/media/ipu3/ipu3-css-params.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[PATCH v2] staging: media: fix spelling mistakes
Posted by Hridesh MG 1 month, 3 weeks ago
Fix three minor spelling/grammar issues:
	chunck -> chunk
	procotol -> protocol
	follow -> following

Signed-off-by: Hridesh MG <hridesh699@gmail.com>
---
Changelog
	v1 -> v2: Move the changes from meta notes to commit
	message.
---
 drivers/staging/media/ipu3/ipu3-css-params.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/media/ipu3/ipu3-css-params.c b/drivers/staging/media/ipu3/ipu3-css-params.c
index 34f574b0b521..af4205f4b038 100644
--- a/drivers/staging/media/ipu3/ipu3-css-params.c
+++ b/drivers/staging/media/ipu3/ipu3-css-params.c
@@ -639,7 +639,7 @@ static int imgu_css_osys_calc_frame_and_stripe_params(
 				/*
 				 * FW workaround for a HW bug: if the first
 				 * chroma pixel is generated exactly at the end
-				 * of chunck scaler HW may not output the pixel
+				 * of chunk scaler HW may not output the pixel
 				 * for downscale factors smaller than 1.5
 				 * (timing issue).
 				 */
@@ -1416,7 +1416,7 @@ imgu_css_shd_ops_calc(struct imgu_abi_shd_intra_frame_operations_data *ops,
 }
 
 /*
- * The follow handshake procotol is the same for AF, AWB and AWB FR.
+ * The following handshake protocol is the same for AF, AWB and AWB FR.
  *
  * for n sets of meta-data, the flow is:
  * --> init
-- 
2.46.1
Re: [PATCH v2] staging: media: fix spelling mistakes
Posted by Hans Verkuil 1 month, 3 weeks ago
On 02/10/2024 17:22, Hridesh MG wrote:
> Fix three minor spelling/grammar issues:
> 	chunck -> chunk
> 	procotol -> protocol
> 	follow -> following
> 
> Signed-off-by: Hridesh MG <hridesh699@gmail.com>

Please always include the driver name in the Subject:

staging: media: ipu3:

That way the ipu3 maintainer will notice the patch.

Regards,

	Hans

> ---
> Changelog
> 	v1 -> v2: Move the changes from meta notes to commit
> 	message.
> ---
>  drivers/staging/media/ipu3/ipu3-css-params.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/media/ipu3/ipu3-css-params.c b/drivers/staging/media/ipu3/ipu3-css-params.c
> index 34f574b0b521..af4205f4b038 100644
> --- a/drivers/staging/media/ipu3/ipu3-css-params.c
> +++ b/drivers/staging/media/ipu3/ipu3-css-params.c
> @@ -639,7 +639,7 @@ static int imgu_css_osys_calc_frame_and_stripe_params(
>  				/*
>  				 * FW workaround for a HW bug: if the first
>  				 * chroma pixel is generated exactly at the end
> -				 * of chunck scaler HW may not output the pixel
> +				 * of chunk scaler HW may not output the pixel
>  				 * for downscale factors smaller than 1.5
>  				 * (timing issue).
>  				 */
> @@ -1416,7 +1416,7 @@ imgu_css_shd_ops_calc(struct imgu_abi_shd_intra_frame_operations_data *ops,
>  }
>  
>  /*
> - * The follow handshake procotol is the same for AF, AWB and AWB FR.
> + * The following handshake protocol is the same for AF, AWB and AWB FR.
>   *
>   * for n sets of meta-data, the flow is:
>   * --> init
Re: [PATCH v2] staging: media: fix spelling mistakes
Posted by Sakari Ailus 1 month, 3 weeks ago
On Thu, Oct 03, 2024 at 08:52:02AM +0200, Hans Verkuil wrote:
> On 02/10/2024 17:22, Hridesh MG wrote:
> > Fix three minor spelling/grammar issues:
> > 	chunck -> chunk
> > 	procotol -> protocol
> > 	follow -> following
> > 
> > Signed-off-by: Hridesh MG <hridesh699@gmail.com>
> 
> Please always include the driver name in the Subject:
> 
> staging: media: ipu3:
> 
> That way the ipu3 maintainer will notice the patch.

Well I was cc'd. ;)

I can fix that while applying, no need for v3...

Hridesh: it's a good idea to check which subject prefix is used for a
driver in the commit log, too.

-- 
Sakari Ailus
Re: [PATCH v2] staging: media: fix spelling mistakes
Posted by Hridesh MG 1 month, 3 weeks ago
On 24/10/03 08:37AM, Sakari Ailus wrote:
>I can fix that while applying, no need for v3...
>
>Hridesh: it's a good idea to check which subject prefix is used for a
>driver in the commit log, too.

Will do, thanks!
Re: [PATCH v2] staging: media: fix spelling mistakes
Posted by Dan Carpenter 1 month, 3 weeks ago
On Wed, Oct 02, 2024 at 08:52:30PM +0530, Hridesh MG wrote:
> Fix three minor spelling/grammar issues:
> 	chunck -> chunk
> 	procotol -> protocol
> 	follow -> following
> 
> Signed-off-by: Hridesh MG <hridesh699@gmail.com>
> ---

Thanks!

Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>

regards,
dan carpenter