On Sun, Oct 26, 2014 at 12:52 PM, Nicholas Krause xerofo...@gmail.com wrote:
Fix all opening and closing braces issues reported by checkpatch.
Signed-off-by: Nicholas Krause xerofo...@gmail.com
---
drivers/staging/rtl8723au/core/rtw_ap.c | 138
++--
1 file
Fix the curley braces that do not reside on the same line because
this does not follow the kernel coding style and causes checkpatch.pl
warnings.
Signed-off-by: Nicholas Krause xerofo...@gmail.com
---
drivers/staging/rtl8723au/core/rtw_ap.c | 138 ++--
1 file changed,
Hi Pranay,
Thanks a bunch for nice explanation.
Best Regards,
Krishna
On Mon, Oct 27, 2014 at 10:34 PM, Pranay Srivastava pran...@gmail.com
wrote:
On Mon, Oct 27, 2014 at 8:08 PM, Er Krishna erkris...@gmail.com wrote:
Hi Valdis,
Many thanks for the mail and quick answers. Regarding the
On Tue, Oct 28, 2014 at 4:24 AM, Jeff Kirsher
jeffrey.t.kirs...@intel.com wrote:
On Sun, Oct 26, 2014 at 12:52 PM, Nicholas Krause xerofo...@gmail.com wrote:
Fix all opening and closing braces issues reported by checkpatch.
Signed-off-by: Nicholas Krause xerofo...@gmail.com
---
On Tue, Oct 28, 2014 at 11:50 AM, Greg Freemyer greg.freem...@gmail.com wrote:
Lots of violations
checkpatch finds are intentionally left in place because correcting
them makes the code less readable, not more readable.
Yeah, but there are still hundreds of thousands of checkpatch
violations
On Tue, Oct 28, 2014 at 1:39 PM, Greg Donald gdon...@gmail.com wrote:
On Tue, Oct 28, 2014 at 11:50 AM, Greg Freemyer greg.freem...@gmail.com
wrote:
Lots of violations
checkpatch finds are intentionally left in place because correcting
them makes the code less readable, not more readable.
Hi,
I'm developing a device driver to drive a specific custom hardware,
that is implemented in FPGA. Since it is an application specific
device driver, where (in witch folder) should I put Its source in my
local kernel source tree? Is there any guideline or best practice
about it?
This device
On Tue, Oct 28, 2014 at 12:53 PM, Nick Krause xerofo...@gmail.com wrote:
I actually fixed this to improve code readability not for the kernel
rules for your information.
I know, good job.. and here's to hoping a maintainer picks up your
patch and applies it to their tree :)
But I would argue
-Original Message-
From: kernelnewbies-bounces+jharan=bytemobile@kernelnewbies.org
[mailto:kernelnewbies-
bounces+jharan=bytemobile@kernelnewbies.org] On Behalf Of Greg Donald
Sent: Tuesday, October 28, 2014 10:39 AM
To: Greg Freemyer
Cc: kernelnewbies; Jeff Kirsher;
it's pointless. Increase it to a 21st century value or kill it.
--
Greg Donald
But, but, but, what about all the kernel developers who are writing kernel
code on VT100s and storing their sources on 80 column punch cards?
Lol, I agree!
There's nothing 21st century about long
On Tue, 28 Oct 2014 12:39:17 -0500, Greg Donald said:
The WARNING line over 80 characters currently accounts for 216K of
the total violations. IMHO checkpatch should just stop complaining
about the 80 char limit since that's the main offender causing new
On the other hand, there's very good
On Tue, Oct 28, 2014 at 12:39:17PM -0500, Greg Donald wrote:
The WARNING line over 80 characters currently accounts for 216K of
the total violations. IMHO checkpatch should just stop complaining
about the 80 char limit since that's the main offender causing new
kernel developers to
Hey Greg,
I am trying to improve my code as much as possible now, I really am finally
understanding
how terrible my code was before and I hope never again to make patches that
shitty.
Cheers Nick
On 14-10-28 02:06 PM, Greg Donald wrote:
On Tue, Oct 28, 2014 at 12:53 PM, Nick Krause
On Tue, Oct 28, 2014 at 1:20 PM, Mandeep Sandhu
mandeepsandhu@gmail.com wrote:
Just
increase your font size so that it fits nicely in your editor from end to
end! :P
I see :)
Sounds good but no. Increasing the font size just causes a loss of
screen real estate vertically.
--
Greg
On Tue, Oct 28, 2014 at 2:52 PM, nick xerofo...@gmail.com wrote:
I am trying to improve my code as much as possible now, I really am finally
understanding
how terrible my code was before and I hope never again to make patches that
shitty.
I don't think your recent progress has gone
On Tue, Oct 28, 2014 at 03:55:53PM -0500, Greg Donald wrote:
On Tue, Oct 28, 2014 at 2:04 PM, John de la Garza j...@jjdev.com wrote:
It seems like you assuming the limit is based on terminal size?
Hmm, I thought it was because of 80 column punch cards and ancient
printers that didn't wrap.
Hello
I have a question about this structure. It has 2 fields. One is iov_base
the pointer of type void. Second is iov_len of type size_t.
This is interesting: iov_len has always a value of 8190. How does it impact
the iov_base? I mean does iov_base is built from other structures? If soo
where I
Greg,
Not picked up of yet. I would appreciate if this gets forwarded for me as this
may help it get picked up.
Further more this issues I am were causing were not technical but not listening
and that's why I decided
to state around and learn how to my patches properly.
Cheers Nick
On
On Tue, 2014-10-28 at 21:49 -0400, nick wrote:
Greg,
Not picked up of yet. I would appreciate if this gets forwarded for me as
this may help it get picked up.
Further more this issues I am were causing were not technical but not
listening and that's why I decided
to state around and learn
Greg,
That's fine, I was wondering how long Greg KH takes to get around to picking
this up as he is very busy with
other kernel work.
Cheers Nick
On 14-10-28 09:58 PM, Jeff Kirsher wrote:
On Tue, 2014-10-28 at 21:49 -0400, nick wrote:
Greg,
Not picked up of yet. I would appreciate if this
On Tue, 2014-10-28 at 22:01 -0400, nick wrote:
Greg,
That's fine, I was wondering how long Greg KH takes to get around to picking
this up as he is very busy with
other kernel work.
Cheers Nick
I am Jeff and I was the one that responded...
On 14-10-28 09:58 PM, Jeff Kirsher wrote:
On
Sorry Jeff,
My fault, mistyped.
Cheers Nick
On 14-10-28 10:04 PM, Jeff Kirsher wrote:
On Tue, 2014-10-28 at 22:01 -0400, nick wrote:
Greg,
That's fine, I was wondering how long Greg KH takes to get around to picking
this up as he is very busy with
other kernel work.
Cheers Nick
I am
從我的 HTC 寄出
- Reply message -
寄件者: Jeff Kirsher jeffrey.t.kirs...@intel.com
收件者: nick xerofo...@gmail.com
副本: Greg Donald gdon...@gmail.com, Greg Freemyer
greg.freem...@gmail.com, kernelnewbies kernelnewbies@kernelnewbies.org
主旨: [PATCH] staging: rtl8723au: Fix brace coding style issues
On Wed, Oct 29, 2014 at 7:31 AM, nick xerofo...@gmail.com wrote:
Greg,
That's fine, I was wondering how long Greg KH takes to get around to picking
this up as he is very busy with
other kernel work.
it might take a long long time. i think he is very busy now. I have
not seen his replies to
24 matches
Mail list logo