> -----Original Message-----
> From: Andrew Lunn [mailto:[email protected]]
> Sent: Monday, June 6, 2016 9:41 AM
> To: Limonciello, Mario <[email protected]>
> Cc: [email protected]; LKML <[email protected]>; Netdev
> <[email protected]>; Linux USB <[email protected]>;
> [email protected]; [email protected]; Greg KH
> <[email protected]>
> Subject: Re: [PATCH v3] r8152: Add support for setting pass through MAC
> address on RTL8153-AD
> 
> > +   /* returns _AUXMAC_#AABBCCDDEEFF# */
> > +   status = acpi_evaluate_object(NULL, "\\_SB.AMAC", NULL, &buffer);
> > +   obj = (union acpi_object *)buffer.pointer;
> > +   if (ACPI_SUCCESS(status)) {
> > +           if (obj->type != ACPI_TYPE_BUFFER ||
> > +               obj->string.length != 0x17) {
> > +                   pr_warn("r8152: get_passthru_addr: Invalid buffer");
> 
> Please don't use pr_warn() when you can use the better alternatives
> like dev_warn().
> 
>      Andrew

OK thanks, I'll adjust that to use a different warn.

Reply via email to