[PATCH] staging: dgnc: Remove non-standard APR/RAPR printing	macros
    Daniel Baluta 
    daniel.baluta at gmail.com
       
    Thu Oct  2 10:16:51 EEST 2014
    
    
  
On Thu, Oct 2, 2014 at 12:42 AM, Roberta Dobrescu
<roberta.dobrescu at gmail.com> wrote:
>> @@ -278,7 +274,7 @@ static int dgnc_start(void)
>>        */
>>       rc = register_chrdev(0, "dgnc", &dgnc_BoardFops);
>>       if (rc <= 0) {
>> -             APR(("Can't register dgnc driver device (%d)\n", rc));
>> +             pr_err(DRVSTR": Can't register dgnc driver device (%d)\n", rc);
>>               return -ENXIO;
>>       }
>
> I'm not sure about these changes from APR to pr_err.
>
> At this point there is no struct device object to pass to dev_dbg.
> I thought about removing the message completely, since there is
> a return err code, but I'm not sure this is the right thing to do.
>
> A second opinion would be very welcome!
Looks good to me.
    
    
More information about the firefly
mailing list