Thread (10 messages) 10 messages, 5 authors, 2021-10-19

Re: [PATCH] staging: vt6655: Fix line wrapping in rf.c file

From: Greg KH <gregkh@linuxfoundation.org>
Date: 2021-10-19 11:12:36
Also in: lkml

On Tue, Oct 19, 2021 at 11:59:56AM +0100, Karolina Drobnik wrote:
Hi,

Thank you very much for your comments.

On Mon, 2021-10-18 at 17:12 +0200, Greg KH wrote:
quoted
Also, these are all just fine as-is for now.  A better way to make
these lines smaller is to use better variable and function names 
that are shorter and make sense :)
I have v2 ready but I'm not sure, given the Joe's patch, if my solution
is a satisfactory one. I didn't jump on such refactoring as I'm still
learning about the codebase/process and didn't want to muddle the
waters (...more than I do already).

Greg, what would you prefer? Should I back up with my patch, pick
something else and let Joe's patch be merged?
Joe hasn't submitted it in a format that I can take it in, so that is up
to you, I can't tell you want to work on :)

good luck!

greg k-h
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help