diff mbox series

media: tm6000: Spelling s/diconencted/diconnected/

Message ID 20190731134121.18473-1-geert+renesas@glider.be (mailing list archive)
State New, archived
Headers show
Series media: tm6000: Spelling s/diconencted/diconnected/ | expand

Commit Message

Geert Uytterhoeven July 31, 2019, 1:41 p.m. UTC
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/media/usb/tm6000/tm6000-cards.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Antti Palosaari July 31, 2019, 6:36 p.m. UTC | #1
On 7/31/19 4:41 PM, Geert Uytterhoeven wrote:
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>   drivers/media/usb/tm6000/tm6000-cards.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/media/usb/tm6000/tm6000-cards.c b/drivers/media/usb/tm6000/tm6000-cards.c
> index 23df50aa0a4af6da..9a0ffe678524987c 100644
> --- a/drivers/media/usb/tm6000/tm6000-cards.c
> +++ b/drivers/media/usb/tm6000/tm6000-cards.c
> @@ -1328,7 +1328,7 @@ static int tm6000_usb_probe(struct usb_interface *interface,
>   
>   /*
>    * tm6000_usb_disconnect()
> - * called when the device gets diconencted
> + * called when the device gets diconnected

disconnected ?

For the both patches.

>    * video device will be unregistered on v4l2_close in case it is still open
>    */
>   static void tm6000_usb_disconnect(struct usb_interface *interface)
> 


regards
Antti
Geert Uytterhoeven July 31, 2019, 6:43 p.m. UTC | #2
Hi Antti,

On Wed, Jul 31, 2019 at 8:36 PM Antti Palosaari <crope@iki.fi> wrote:
> On 7/31/19 4:41 PM, Geert Uytterhoeven wrote:
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > ---
> >   drivers/media/usb/tm6000/tm6000-cards.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/media/usb/tm6000/tm6000-cards.c b/drivers/media/usb/tm6000/tm6000-cards.c
> > index 23df50aa0a4af6da..9a0ffe678524987c 100644
> > --- a/drivers/media/usb/tm6000/tm6000-cards.c
> > +++ b/drivers/media/usb/tm6000/tm6000-cards.c
> > @@ -1328,7 +1328,7 @@ static int tm6000_usb_probe(struct usb_interface *interface,
> >
> >   /*
> >    * tm6000_usb_disconnect()
> > - * called when the device gets diconencted
> > + * called when the device gets diconnected
>
> disconnected ?

Thanks, will fix!

> For the both patches.

How embarassing...
/me hides in a brown paper bag...

Gr{oetje,eeting}s,

                        Geert
Hans Verkuil Aug. 7, 2019, 9:33 a.m. UTC | #3
On 7/31/19 8:43 PM, Geert Uytterhoeven wrote:
> Hi Antti,
> 
> On Wed, Jul 31, 2019 at 8:36 PM Antti Palosaari <crope@iki.fi> wrote:
>> On 7/31/19 4:41 PM, Geert Uytterhoeven wrote:
>>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>>> ---
>>>   drivers/media/usb/tm6000/tm6000-cards.c | 2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/media/usb/tm6000/tm6000-cards.c b/drivers/media/usb/tm6000/tm6000-cards.c
>>> index 23df50aa0a4af6da..9a0ffe678524987c 100644
>>> --- a/drivers/media/usb/tm6000/tm6000-cards.c
>>> +++ b/drivers/media/usb/tm6000/tm6000-cards.c
>>> @@ -1328,7 +1328,7 @@ static int tm6000_usb_probe(struct usb_interface *interface,
>>>
>>>   /*
>>>    * tm6000_usb_disconnect()
>>> - * called when the device gets diconencted
>>> + * called when the device gets diconnected
>>
>> disconnected ?
> 
> Thanks, will fix!

Can you also add a commit message when you post v2? Currently that's
empty and Linus doesn't like empty commit messages :-)

Regards,

	Hans

> 
>> For the both patches.
> 
> How embarassing...
> /me hides in a brown paper bag...
> 
> Gr{oetje,eeting}s,
> 
>                         Geert
>
Geert Uytterhoeven Aug. 7, 2019, 10:53 a.m. UTC | #4
Hi Hans,

On Wed, Aug 7, 2019 at 11:33 AM Hans Verkuil <hverkuil@xs4all.nl> wrote:
> On 7/31/19 8:43 PM, Geert Uytterhoeven wrote:
> > On Wed, Jul 31, 2019 at 8:36 PM Antti Palosaari <crope@iki.fi> wrote:
> >> On 7/31/19 4:41 PM, Geert Uytterhoeven wrote:
> >>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >>> ---
> >>>   drivers/media/usb/tm6000/tm6000-cards.c | 2 +-
> >>>   1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>> diff --git a/drivers/media/usb/tm6000/tm6000-cards.c b/drivers/media/usb/tm6000/tm6000-cards.c
> >>> index 23df50aa0a4af6da..9a0ffe678524987c 100644
> >>> --- a/drivers/media/usb/tm6000/tm6000-cards.c
> >>> +++ b/drivers/media/usb/tm6000/tm6000-cards.c
> >>> @@ -1328,7 +1328,7 @@ static int tm6000_usb_probe(struct usb_interface *interface,
> >>>
> >>>   /*
> >>>    * tm6000_usb_disconnect()
> >>> - * called when the device gets diconencted
> >>> + * called when the device gets diconnected
> >>
> >> disconnected ?
> >
> > Thanks, will fix!
>
> Can you also add a commit message when you post v2? Currently that's
> empty and Linus doesn't like empty commit messages :-)

Even if everything to be said is in the one-line summary?

Gr{oetje,eeting}s,

                        Geert
Hans Verkuil Aug. 7, 2019, 11:05 a.m. UTC | #5
On 8/7/19 12:53 PM, Geert Uytterhoeven wrote:
> Hi Hans,
> 
> On Wed, Aug 7, 2019 at 11:33 AM Hans Verkuil <hverkuil@xs4all.nl> wrote:
>> On 7/31/19 8:43 PM, Geert Uytterhoeven wrote:
>>> On Wed, Jul 31, 2019 at 8:36 PM Antti Palosaari <crope@iki.fi> wrote:
>>>> On 7/31/19 4:41 PM, Geert Uytterhoeven wrote:
>>>>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>>>>> ---
>>>>>   drivers/media/usb/tm6000/tm6000-cards.c | 2 +-
>>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/drivers/media/usb/tm6000/tm6000-cards.c b/drivers/media/usb/tm6000/tm6000-cards.c
>>>>> index 23df50aa0a4af6da..9a0ffe678524987c 100644
>>>>> --- a/drivers/media/usb/tm6000/tm6000-cards.c
>>>>> +++ b/drivers/media/usb/tm6000/tm6000-cards.c
>>>>> @@ -1328,7 +1328,7 @@ static int tm6000_usb_probe(struct usb_interface *interface,
>>>>>
>>>>>   /*
>>>>>    * tm6000_usb_disconnect()
>>>>> - * called when the device gets diconencted
>>>>> + * called when the device gets diconnected
>>>>
>>>> disconnected ?
>>>
>>> Thanks, will fix!
>>
>> Can you also add a commit message when you post v2? Currently that's
>> empty and Linus doesn't like empty commit messages :-)
> 
> Even if everything to be said is in the one-line summary?

Yes. In fact, checkpatch warns if the commit description is empty.

In this case, just say 'Fix typo.', that's sufficient.

Regards,

	Hans

> 
> Gr{oetje,eeting}s,
> 
>                         Geert
>
Geert Uytterhoeven Aug. 7, 2019, 12:12 p.m. UTC | #6
Hi Hans,

On Wed, Aug 7, 2019 at 1:05 PM Hans Verkuil <hverkuil@xs4all.nl> wrote:
> On 8/7/19 12:53 PM, Geert Uytterhoeven wrote:
> > On Wed, Aug 7, 2019 at 11:33 AM Hans Verkuil <hverkuil@xs4all.nl> wrote:
> >> On 7/31/19 8:43 PM, Geert Uytterhoeven wrote:
> >>> On Wed, Jul 31, 2019 at 8:36 PM Antti Palosaari <crope@iki.fi> wrote:
> >>>> On 7/31/19 4:41 PM, Geert Uytterhoeven wrote:
> >>>>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >>>>> ---
> >>>>>   drivers/media/usb/tm6000/tm6000-cards.c | 2 +-
> >>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
> >>>>>
> >>>>> diff --git a/drivers/media/usb/tm6000/tm6000-cards.c b/drivers/media/usb/tm6000/tm6000-cards.c
> >>>>> index 23df50aa0a4af6da..9a0ffe678524987c 100644
> >>>>> --- a/drivers/media/usb/tm6000/tm6000-cards.c
> >>>>> +++ b/drivers/media/usb/tm6000/tm6000-cards.c
> >>>>> @@ -1328,7 +1328,7 @@ static int tm6000_usb_probe(struct usb_interface *interface,
> >>>>>
> >>>>>   /*
> >>>>>    * tm6000_usb_disconnect()
> >>>>> - * called when the device gets diconencted
> >>>>> + * called when the device gets diconnected
> >>>>
> >>>> disconnected ?
> >>>
> >>> Thanks, will fix!
> >>
> >> Can you also add a commit message when you post v2? Currently that's
> >> empty and Linus doesn't like empty commit messages :-)
> >
> > Even if everything to be said is in the one-line summary?
>
> Yes. In fact, checkpatch warns if the commit description is empty.

Quoting the checkpatch.pl maintainer:

   "checkpatch is a stupid script and doesn't forbid anything.
    It's just a suggestion guide."

https://lore.kernel.org/lkml/20190801124037.11dcac71@gandalf.local.home/

> In this case, just say 'Fix typo.', that's sufficient.

If that makes you happy...

Gr{oetje,eeting}s,

                        Geert
diff mbox series

Patch

diff --git a/drivers/media/usb/tm6000/tm6000-cards.c b/drivers/media/usb/tm6000/tm6000-cards.c
index 23df50aa0a4af6da..9a0ffe678524987c 100644
--- a/drivers/media/usb/tm6000/tm6000-cards.c
+++ b/drivers/media/usb/tm6000/tm6000-cards.c
@@ -1328,7 +1328,7 @@  static int tm6000_usb_probe(struct usb_interface *interface,
 
 /*
  * tm6000_usb_disconnect()
- * called when the device gets diconencted
+ * called when the device gets diconnected
  * video device will be unregistered on v4l2_close in case it is still open
  */
 static void tm6000_usb_disconnect(struct usb_interface *interface)