Messages in this thread | | | Date | Tue, 29 Oct 2013 08:50:40 -0700 | From | Greg KH <> | Subject | Re: [PATCH] drivers: staging: bcm: Removed a developer debug statement. |
| |
On Thu, Oct 24, 2013 at 01:02:45AM +0300, Dan Carpenter wrote: > On Wed, Oct 23, 2013 at 03:11:20PM -0400, Chuong Ngo wrote: > > Removed a developer debug statement per the TODO list. Additionally, removed braces for the if-statement to match coding style. > > Line wrap the changelog at 72 characters. You don't really need to > mention the removed braces because that's too obvious to bother > commenting on. > > My guess is that Greg will probably be willing to apply this as is so > don't worry about resending unless he requests it. (He normally seems > to let newbies get by with one line wrapping warning before he starts > making you redo it).
Yes, I'll fix it this time, but Chuong, please be more careful next time.
greg k-h
| |