lkml.org 
[lkml]   [2010]   [Jun]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 3/5]acpi:glue.c Fix warning: variable 'ret' set but not used
On 06/29/2010 08:47 AM, David Howells wrote:
> Justin P. Mattock<justinmattock@gmail.com> wrote:
>
>> + if (fn) {
>> + printk(KERN_WARNING "dev%d: Failed to create
>> firmware_node: %d\n", status, fn);
>> + }else if (pn) {
>> + printk(KERN_WARNING "dev%d: Failed to create
>> physical_node: %d\n", status, pn);
>> + return 0;
>> + }
>
> The if-statement should be correctly indented (it's inside another
> if-body, so needs to be one more tab over) and there needs to be a space
> before the else.
>
o.k., so it should look something like this:
if (fn) {
code...
} else if (pn) {


> You should probably split your printks up so they don't exceed 80 chars too,
> for example:
>
> printk(KERN_WARNING
> "dev%d: Failed to create physical_node: %d\n",
> status, pn);
>
> Also 'status' is probably the wrong thing to print as the number in "dev%d".
> If it worked, that should be unconditionally AE_OK, I think. Can you not use
> dev_warn() or similar instead or printk?
>
> David
>

alright, I'll look at this today. I'm not the best at making printks in
fact I'm more intimidated by them..(so with this in mind, I'm going to
sit and make myself learn this, so I atleast have a better idea of doing
these than I have now.)

Justin P. Mattock










\
 
 \ /
  Last update: 2010-06-29 19:17    [W:0.261 / U:0.644 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site