| Message ID | 20240923033032.2420839-1-yujiaoliang@vivo.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [v1] media: atomisp: Fix typos in comment | expand |
On 9/23/24 5:30 AM, Yu Jiaoliang wrote: > Corrected typos: > componnet->component, > cofiguration->configuration, > doubtfull->doubtful, > consisit->consist, > coppied->copied. > > These changes fix the typos in the comment, > without affecting the functionality. > > Signed-off-by: Yu Jiaoliang <yujiaoliang@vivo.com> Hi Jiaoliang, Please make your "Subject" line more unique. Consider that we may end up with having dozen of commits like yours, all of them referring to different removals and all without the necessary information to tell what they differ in (except the driver/subsystem). So it would help if you add the changed file or function to make it more unique. Please improve your patch description. Better omit what you are changing as this can be seen in the change itself. Do focus on why this patch makes the code better. Your sentence is fine. Just remove what you change. If you send in a second version of this patch please use a change history. Description from Dan under: https://staticthinking.wordpress.com/2022/07/27/how-to-send-a-v2-patch/ Thanks Bye Philipp
On 9/23/24 5:30 AM, Yu Jiaoliang wrote: > Corrected typos: > componnet->component, > cofiguration->configuration, > doubtfull->doubtful, > consisit->consist, > coppied->copied. > > These changes fix the typos in the comment, > without affecting the functionality. > > Signed-off-by: Yu Jiaoliang <yujiaoliang@vivo.com> > --- > drivers/staging/media/atomisp/pci/atomisp_v4l2.c | 2 +- > .../media/atomisp/pci/hive_isp_css_common/host/input_system.c | 2 +- > drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c | 2 +- > drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c | 2 +- > drivers/staging/media/atomisp/pci/sh_css_mipi.c | 2 +- > 5 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c > index c9984f1557b0..eb46ba4f4f13 100644 > --- a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c > +++ b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c > @@ -55,7 +55,7 @@ > /* G-Min addition: pull this in from intel_mid_pm.h */ > #define CSTATE_EXIT_LATENCY_C1 1 > > -/* cross componnet debug message flag */ > +/* cross component debug message flag */ > int dbg_level; > module_param(dbg_level, int, 0644); > MODULE_PARM_DESC(dbg_level, "debug message level (default:0)"); > diff --git a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c > index 8f2f4e8eddd9..c18f62fe66b3 100644 > --- a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c > +++ b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c > @@ -406,7 +406,7 @@ input_system_err_t input_system_configuration_reset(void) > config.unallocated_ib_mem_words = IB_CAPACITY_IN_WORDS; > //config.acq_allocated_ib_mem_words = 0; > > - // Set the start of the session cofiguration. > + // Set the start of the session configuration. > config.session_flags = INPUT_SYSTEM_CFG_FLAG_REQUIRED; > > return INPUT_SYSTEM_ERR_NO_ERROR; > diff --git a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c > index 7ce2b2d6da11..3bfaf52c5cdd 100644 > --- a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c > +++ b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c > @@ -951,7 +951,7 @@ int ia_css_binary_find(struct ia_css_binary_descr *descr, struct ia_css_binary * > unsigned int i; > > assert(descr); > - /* MW: used after an error check, may accept NULL, but doubtfull */ > + /* MW: used after an error check, may accept NULL, but doubtful */ > assert(binary); > > dev_dbg(atomisp_dev, "ia_css_binary_find() enter: descr=%p, (mode=%d), binary=%p\n", > diff --git a/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c b/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c > index 2d7fddb114f6..0ab83e1b51b8 100644 > --- a/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c > +++ b/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c > @@ -362,7 +362,7 @@ void ia_css_frame_free_multiple(unsigned int num_frames, > int ia_css_frame_allocate_with_buffer_size(struct ia_css_frame **frame, > const unsigned int buffer_size_bytes) > { > - /* AM: Body coppied from frame_allocate_with_data(). */ > + /* AM: Body copied from frame_allocate_with_data(). */ > int err; > struct ia_css_frame *me = frame_create(0, 0, > IA_CSS_FRAME_FORMAT_NUM,/* Not valid format yet */ > diff --git a/drivers/staging/media/atomisp/pci/sh_css_mipi.c b/drivers/staging/media/atomisp/pci/sh_css_mipi.c > index 80f0395cc560..710744ff0b24 100644 > --- a/drivers/staging/media/atomisp/pci/sh_css_mipi.c > +++ b/drivers/staging/media/atomisp/pci/sh_css_mipi.c > @@ -169,7 +169,7 @@ ia_css_mipi_frame_calculate_size(const unsigned int width, > /* ceil(words_per_odd_line/8); mem_word = 32 bytes, 8 words */ > mem_words_for_first_line = (words_for_first_line + 7) >> 3; > mem_words_per_even_line = (words_per_even_line + 7) >> 3; > - mem_words_for_EOF = 1; /* last line consisit of the optional (EOL) and EOF */ > + mem_words_for_EOF = 1; /* last line consist of the optional (EOL) and EOF */ > > mem_words = ((embedded_data_size_words + 7) >> 3) + > mem_words_for_first_line + Hi Jiaoliang, I cannot apply your patch: Applying: media: atomisp: Fix typos in comment error: patch failed: drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c:951 error: drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c: patch does not apply Patch failed at 0001 media: atomisp: Fix typos in comment Are you using the right git repo? git remote show origin * remote origin Fetch URL: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git ... git branch -a my branch: staging-testing Please rebase. Thanks. Bye Philipp
On Mon, Sep 23, 2024 at 07:57:00AM +0200, Philipp Hortmann wrote: > On 9/23/24 5:30 AM, Yu Jiaoliang wrote: ... > I cannot apply your patch: > Fetch URL: > git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git You used the wrong branch. AtomISP has it's own tree/branch for development: https://git.kernel.org/pub/scm/linux/kernel/git/hansg/linux.git/log/?h=media-atomisp
On Mon, Sep 23, 2024 at 07:44:51AM +0200, Philipp Hortmann wrote: > On 9/23/24 5:30 AM, Yu Jiaoliang wrote: ... > Please make your "Subject" line more unique. Consider that we may end up > with having dozen of commits like yours, all of them referring to different > removals and all without the necessary information to tell what they differ > in (except the driver/subsystem). So it would help if you add the changed > file or function to make it more unique. For _this_ driver the Subject is good enough, no need to change. We all know that 1) AtomISP is in staging; 2) AtomISP is unique driver in the entire kernel. > Please improve your patch description. Better omit what you are changing as > this can be seen in the change itself. Do focus on why this patch makes the > code better. Your sentence is fine. Just remove what you change. > > If you send in a second version of this patch please use a change history. > Description from Dan under: > https://staticthinking.wordpress.com/2022/07/27/how-to-send-a-v2-patch/ The above is a good advice, thanks.
On Mon, Sep 23, 2024 at 11:30:23AM +0800, Yu Jiaoliang wrote: > Corrected typos: > componnet->component, > cofiguration->configuration, > doubtfull->doubtful, > consisit->consist, > coppied->copied. > > These changes fix the typos in the comment, > without affecting the functionality. ... > //config.acq_allocated_ib_mem_words = 0; > > - // Set the start of the session cofiguration. > + // Set the start of the session configuration. While at it, the comment style can be changed to C-like /* Set the start of the session configuration */ (no need to resend unless Hans asks for that, otherwise he might fix whilst applying).
On Mon, Sep 23, 2024 at 11:26:57AM +0300, Andy Shevchenko wrote: > On Mon, Sep 23, 2024 at 07:57:00AM +0200, Philipp Hortmann wrote: > > On 9/23/24 5:30 AM, Yu Jiaoliang wrote: ... > > I cannot apply your patch: > > > Fetch URL: > > git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git > > You used the wrong branch. > AtomISP has it's own tree/branch for development: > https://git.kernel.org/pub/scm/linux/kernel/git/hansg/linux.git/log/?h=media-atomisp Talking about this, Yu may use --base next time, so everyone, including CIs, will know exactly what should be used as a base.
Hi, On 23-Sep-24 5:30 AM, Yu Jiaoliang wrote: > Corrected typos: > componnet->component, > cofiguration->configuration, > doubtfull->doubtful, > consisit->consist, > coppied->copied. > > These changes fix the typos in the comment, > without affecting the functionality. > > Signed-off-by: Yu Jiaoliang <yujiaoliang@vivo.com> Thank you for your patch(es). I have merged this/these in my media-atomisp branch: https://git.kernel.org/pub/scm/linux/kernel/git/hansg/linux.git/log/?h=media-atomisp And this/these will be included in my next pull-request to Mauro (to media subsystem maintainer) Regards, Hans > --- > drivers/staging/media/atomisp/pci/atomisp_v4l2.c | 2 +- > .../media/atomisp/pci/hive_isp_css_common/host/input_system.c | 2 +- > drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c | 2 +- > drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c | 2 +- > drivers/staging/media/atomisp/pci/sh_css_mipi.c | 2 +- > 5 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c > index c9984f1557b0..eb46ba4f4f13 100644 > --- a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c > +++ b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c > @@ -55,7 +55,7 @@ > /* G-Min addition: pull this in from intel_mid_pm.h */ > #define CSTATE_EXIT_LATENCY_C1 1 > > -/* cross componnet debug message flag */ > +/* cross component debug message flag */ > int dbg_level; > module_param(dbg_level, int, 0644); > MODULE_PARM_DESC(dbg_level, "debug message level (default:0)"); > diff --git a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c > index 8f2f4e8eddd9..c18f62fe66b3 100644 > --- a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c > +++ b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c > @@ -406,7 +406,7 @@ input_system_err_t input_system_configuration_reset(void) > config.unallocated_ib_mem_words = IB_CAPACITY_IN_WORDS; > //config.acq_allocated_ib_mem_words = 0; > > - // Set the start of the session cofiguration. > + // Set the start of the session configuration. > config.session_flags = INPUT_SYSTEM_CFG_FLAG_REQUIRED; > > return INPUT_SYSTEM_ERR_NO_ERROR; > diff --git a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c > index 7ce2b2d6da11..3bfaf52c5cdd 100644 > --- a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c > +++ b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c > @@ -951,7 +951,7 @@ int ia_css_binary_find(struct ia_css_binary_descr *descr, struct ia_css_binary * > unsigned int i; > > assert(descr); > - /* MW: used after an error check, may accept NULL, but doubtfull */ > + /* MW: used after an error check, may accept NULL, but doubtful */ > assert(binary); > > dev_dbg(atomisp_dev, "ia_css_binary_find() enter: descr=%p, (mode=%d), binary=%p\n", > diff --git a/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c b/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c > index 2d7fddb114f6..0ab83e1b51b8 100644 > --- a/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c > +++ b/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c > @@ -362,7 +362,7 @@ void ia_css_frame_free_multiple(unsigned int num_frames, > int ia_css_frame_allocate_with_buffer_size(struct ia_css_frame **frame, > const unsigned int buffer_size_bytes) > { > - /* AM: Body coppied from frame_allocate_with_data(). */ > + /* AM: Body copied from frame_allocate_with_data(). */ > int err; > struct ia_css_frame *me = frame_create(0, 0, > IA_CSS_FRAME_FORMAT_NUM,/* Not valid format yet */ > diff --git a/drivers/staging/media/atomisp/pci/sh_css_mipi.c b/drivers/staging/media/atomisp/pci/sh_css_mipi.c > index 80f0395cc560..710744ff0b24 100644 > --- a/drivers/staging/media/atomisp/pci/sh_css_mipi.c > +++ b/drivers/staging/media/atomisp/pci/sh_css_mipi.c > @@ -169,7 +169,7 @@ ia_css_mipi_frame_calculate_size(const unsigned int width, > /* ceil(words_per_odd_line/8); mem_word = 32 bytes, 8 words */ > mem_words_for_first_line = (words_for_first_line + 7) >> 3; > mem_words_per_even_line = (words_per_even_line + 7) >> 3; > - mem_words_for_EOF = 1; /* last line consisit of the optional (EOL) and EOF */ > + mem_words_for_EOF = 1; /* last line consist of the optional (EOL) and EOF */ > > mem_words = ((embedded_data_size_words + 7) >> 3) + > mem_words_for_first_line +
diff --git a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c index c9984f1557b0..eb46ba4f4f13 100644 --- a/drivers/staging/media/atomisp/pci/atomisp_v4l2.c +++ b/drivers/staging/media/atomisp/pci/atomisp_v4l2.c @@ -55,7 +55,7 @@ /* G-Min addition: pull this in from intel_mid_pm.h */ #define CSTATE_EXIT_LATENCY_C1 1 -/* cross componnet debug message flag */ +/* cross component debug message flag */ int dbg_level; module_param(dbg_level, int, 0644); MODULE_PARM_DESC(dbg_level, "debug message level (default:0)"); diff --git a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c index 8f2f4e8eddd9..c18f62fe66b3 100644 --- a/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c +++ b/drivers/staging/media/atomisp/pci/hive_isp_css_common/host/input_system.c @@ -406,7 +406,7 @@ input_system_err_t input_system_configuration_reset(void) config.unallocated_ib_mem_words = IB_CAPACITY_IN_WORDS; //config.acq_allocated_ib_mem_words = 0; - // Set the start of the session cofiguration. + // Set the start of the session configuration. config.session_flags = INPUT_SYSTEM_CFG_FLAG_REQUIRED; return INPUT_SYSTEM_ERR_NO_ERROR; diff --git a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c index 7ce2b2d6da11..3bfaf52c5cdd 100644 --- a/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c +++ b/drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c @@ -951,7 +951,7 @@ int ia_css_binary_find(struct ia_css_binary_descr *descr, struct ia_css_binary * unsigned int i; assert(descr); - /* MW: used after an error check, may accept NULL, but doubtfull */ + /* MW: used after an error check, may accept NULL, but doubtful */ assert(binary); dev_dbg(atomisp_dev, "ia_css_binary_find() enter: descr=%p, (mode=%d), binary=%p\n", diff --git a/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c b/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c index 2d7fddb114f6..0ab83e1b51b8 100644 --- a/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c +++ b/drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c @@ -362,7 +362,7 @@ void ia_css_frame_free_multiple(unsigned int num_frames, int ia_css_frame_allocate_with_buffer_size(struct ia_css_frame **frame, const unsigned int buffer_size_bytes) { - /* AM: Body coppied from frame_allocate_with_data(). */ + /* AM: Body copied from frame_allocate_with_data(). */ int err; struct ia_css_frame *me = frame_create(0, 0, IA_CSS_FRAME_FORMAT_NUM,/* Not valid format yet */ diff --git a/drivers/staging/media/atomisp/pci/sh_css_mipi.c b/drivers/staging/media/atomisp/pci/sh_css_mipi.c index 80f0395cc560..710744ff0b24 100644 --- a/drivers/staging/media/atomisp/pci/sh_css_mipi.c +++ b/drivers/staging/media/atomisp/pci/sh_css_mipi.c @@ -169,7 +169,7 @@ ia_css_mipi_frame_calculate_size(const unsigned int width, /* ceil(words_per_odd_line/8); mem_word = 32 bytes, 8 words */ mem_words_for_first_line = (words_for_first_line + 7) >> 3; mem_words_per_even_line = (words_per_even_line + 7) >> 3; - mem_words_for_EOF = 1; /* last line consisit of the optional (EOL) and EOF */ + mem_words_for_EOF = 1; /* last line consist of the optional (EOL) and EOF */ mem_words = ((embedded_data_size_words + 7) >> 3) + mem_words_for_first_line +
Corrected typos: componnet->component, cofiguration->configuration, doubtfull->doubtful, consisit->consist, coppied->copied. These changes fix the typos in the comment, without affecting the functionality. Signed-off-by: Yu Jiaoliang <yujiaoliang@vivo.com> --- drivers/staging/media/atomisp/pci/atomisp_v4l2.c | 2 +- .../media/atomisp/pci/hive_isp_css_common/host/input_system.c | 2 +- drivers/staging/media/atomisp/pci/runtime/binary/src/binary.c | 2 +- drivers/staging/media/atomisp/pci/runtime/frame/src/frame.c | 2 +- drivers/staging/media/atomisp/pci/sh_css_mipi.c | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-)