Re: [PATCH 3/3] iw_cxgb4: remove the stid on listen create failure

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

 



On Fri, Sep 15, 2017 at 01:51:17PM -0500, Steve Wise wrote:
> > Subject: Re: [PATCH 3/3] iw_cxgb4: remove the stid on listen create failure
> > 
> > On Tue, Sep 05, 2017 at 11:52:34AM -0700, Steve Wise wrote:
> > > Signed-off-by: Steve Wise <swise@xxxxxxxxxxxxxxxxxxxxx>
> > > ---
> > >  drivers/infiniband/hw/cxgb4/cm.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > I know I sure don't take patches without changelog text, and for sure
> > that could never go into a stable tree...
> 
> I thought the one line commit log was sufficient.  I'll be more verbose.

All you are saying in the subject was _what_ you did, which does nothing
to explain _why_ you did it.  Especially for something you feel needs to
be fixed in older kernels you need to explain why it is all needed.

Changelog descriptions are hard, usually harder than writing the
original patch.  Welcome to good software engineering practices :)

thanks,

greg k-h



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]