drivers/staging/media/ipu3/ipu3-css-params.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
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
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
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
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!
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
© 2016 - 2024 Red Hat, Inc.