Re: [PATCH v2] Fix spelling errors in file drivers/usb/gadget/udc/m66592-udc.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Sep 16, 2024 at 01:35:19AM -0600, Everest K.C. wrote:
> On Mon, Sep 16, 2024 at 12:52 AM Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> >
> > On Sun, Sep 15, 2024 at 11:23:56PM -0600, Everest K.C. wrote:
> > > Fixed spelling errors in error message and comments that
> > > were reported by codespell as follows:
> > >       unexpect  --> unexpected
> > >       workaound --> workaround
> > >
> > > Signed-off-by: Everest K.C. <everestkc@xxxxxxxxxxxxxxxx>
> > > ---
> > >  v1 --> v2: Removed the coding style fix. This patch now only
> > >           fixes spelling errors. The coding style will be fixed
> > >           in a separate patch.
> > >
> > >  drivers/usb/gadget/udc/m66592-udc.c | 18 +++++++++---------
> >
> > If you do:
> >         git log --oneline drivers/usb/gadget/udc/m66592-udc.c
> > you will see the format for how to properly write the Subject: line
> > here.
> >
> > As this is your first kernel contributions, I recommend working in an
> > area that is designed for new developers to help work through these
> > types of issues, drivers/staging/  Get some experience there before
> > moving out into other areas of the kernel.  Also note that many
> > subsystems do not care about things like spelling fixes, so be careful.
> >
> > thanks,
> >
> > greg k-h
> 
> Hi Greg,
> Thank you for pointing out my mistake with the subject line. I have noted
> and will keep that in mind when I submit my next patch.
> Also, do you mean the fixes like this need to be done in gregkh/staging.git
> repo? Do you recommend that I resubmit this patch with corrections or
> discard it ?

I mean start working in drivers/staging/ on issues there, not in
drivers/usb/, until you get the process working well for you, then worry
about moving to other portions of the kernel.

good luck!

greg k-h




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux