mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] staging: media: atomisp: fix spelling in hmm_bo log messages
@ 2026-11-10 11:57 Kincses Benjámin
  2026-11-10 13:40 ` Dan Carpenter
  2026-11-10 13:46 ` Andy Shevchenko
  0 siblings, 2 replies; 3+ messages in thread
From: Kincses Benjámin @ 2026-11-10 11:57 UTC (permalink / raw)
  To: Hans de Goede, Mauro Carvalho Chehab
  Cc: Sakari Ailus, Andy Shevchenko, Greg Kroah-Hartman, linux-media,
	linux-staging, linux-kernel

Three hmm_bo log messages misspell the past tense of 'bind'.
Fix them to use 'bound', as listed in scripts/spelling.txt.
Only string literals are changed; no functional change.

Signed-off-by: Benjamin Kincses <kincses.benjamin2007@gmail.com>
---
 drivers/staging/media/atomisp/pci/hmm/hmm_bo.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/media/atomisp/pci/hmm/hmm_bo.c
b/drivers/staging/media/atomisp/pci/hmm/hmm_bo.c
index 0a8f401..1fc5c1e 100644
--- a/drivers/staging/media/atomisp/pci/hmm/hmm_bo.c
+++ b/drivers/staging/media/atomisp/pci/hmm/hmm_bo.c
@@ -448,7 +448,7 @@ void hmm_bo_release(struct hmm_buffer_object *bo)
        }

        if (bo->status & HMM_BO_BINDED) {
-               dev_warn(atomisp_dev, "the bo is still binded, unbind
it first...\n");
+               dev_warn(atomisp_dev, "the bo is still bound, unbind
it first...\n");
                hmm_bo_unbind(bo);
        }

@@ -830,7 +830,7 @@ int hmm_bo_bind(struct hmm_buffer_object *bo)

 status_err2:
        mutex_unlock(&bo->mutex);
-       dev_err(atomisp_dev, "buffer object already binded.\n");
+       dev_err(atomisp_dev, "buffer object already bound.\n");
        return -EINVAL;
 status_err1:
        mutex_unlock(&bo->mutex);
@@ -882,7 +882,7 @@ void hmm_bo_unbind(struct hmm_buffer_object *bo)
 status_err:
        mutex_unlock(&bo->mutex);
        dev_err(atomisp_dev,
-               "buffer vm or page not allocated or not binded yet.\n");
+               "buffer vm or page not allocated or not bound yet.\n");
 }

 int hmm_bo_binded(struct hmm_buffer_object *bo)

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] staging: media: atomisp: fix spelling in hmm_bo log messages
  2026-11-10 11:57 [PATCH] staging: media: atomisp: fix spelling in hmm_bo log messages Kincses Benjámin
@ 2026-11-10 13:40 ` Dan Carpenter
  2026-11-10 13:46 ` Andy Shevchenko
  1 sibling, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2026-11-10 13:40 UTC (permalink / raw)
  To: Kincses Benjámin
  Cc: Hans de Goede, Mauro Carvalho Chehab, Sakari Ailus,
	Andy Shevchenko, Greg Kroah-Hartman, linux-media, linux-staging,
	linux-kernel

On Tue, Nov 10, 2026 at 12:57:49PM +0100, Kincses Benjámin wrote:
> Three hmm_bo log messages misspell the past tense of 'bind'.
> Fix them to use 'bound', as listed in scripts/spelling.txt.
> Only string literals are changed; no functional change.
> 
> Signed-off-by: Benjamin Kincses <kincses.benjamin2007@gmail.com>
> ---

The patch is corrupted.  Read the first couple paragraphs of
Documentation/process/email-clients.rst

regards,
dan carpenter


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] staging: media: atomisp: fix spelling in hmm_bo log messages
  2026-11-10 11:57 [PATCH] staging: media: atomisp: fix spelling in hmm_bo log messages Kincses Benjámin
  2026-11-10 13:40 ` Dan Carpenter
@ 2026-11-10 13:46 ` Andy Shevchenko
  1 sibling, 0 replies; 3+ messages in thread
From: Andy Shevchenko @ 2026-11-10 13:46 UTC (permalink / raw)
  To: Kincses Benjámin
  Cc: Hans de Goede, Mauro Carvalho Chehab, Sakari Ailus,
	Andy Shevchenko, Greg Kroah-Hartman, linux-media, linux-staging,
	linux-kernel

On Tue, Nov 10, 2026 at 1:58 PM Kincses Benjámin
<kincses.benjamin2007@gmail.com> wrote:
>
> Three hmm_bo log messages misspell the past tense of 'bind'.
> Fix them to use 'bound', as listed in scripts/spelling.txt.
> Only string literals are changed; no functional change.

Don't be a meat proxy to some tools. While grammatically your patch is
correct, I think it will give wrong information as the used (wrong)
English word "binded" comes from the definition. I haven't checked if
HMM is a HW-related piece of software, but if it's, we can't change it
without checking on the datasheet. Otherwise this needs to change the
constants and the whole usage of "binded" in the whole driver.

-- 
With Best Regards,
Andy Shevchenko

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-11-10 13:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-11-10 11:57 [PATCH] staging: media: atomisp: fix spelling in hmm_bo log messages Kincses Benjámin
2026-11-10 13:40 ` Dan Carpenter
2026-11-10 13:46 ` Andy Shevchenko

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®