Re: [PATCH] eeepc-laptop.c: use pr_fmt and pr_<level>

[Alan Jenkins]

Joe Perches wrote:
> Convert the unusual printk(EEEPC_<level> uses to
> the more standard pr_fmt and pr_<level>(.
>
> Signed-off-by: Joe Perches <joe@xxxxxxxxxxx>
>
> diff --git a/drivers/platform/x86/eeepc-laptop.c b/drivers/platform/x86/eeepc-laptop.c
> index 353a898..94cacdd 100644
> --- a/drivers/platform/x86/eeepc-laptop.c
> +++ b/drivers/platform/x86/eeepc-laptop.c
> @@ -16,6 +16,8 @@
>   *  GNU General Public License for more details.
>   */
>  
> +#define pr_fmt(fmt) EEEPC_HOTK_FILE ": " fmt
>
>   

Is EEEPC_HOTK_FILE used anywhere else?  It would be clearer if you were 
able to just use "eeepc: " or whatever and remove the definition of 
EEEPC_HOTK_FILE.

Thanks for the cleanup
Alan

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/