Re: [PATCH] acpi: add newline in power.c message

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 29, 2012 at 11:25 AM, Randy Dunlap <rdunlap@xxxxxxxxxxxx> wrote:
> From: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
>
> Add newline to printk so that the message is on a line
> by itself and not merged with something unrelated to it.
>
> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
> Reported-by: Toralf Förster <toralf.foerster@xxxxxx>
> Cc:     Len Brown <lenb@xxxxxxxxxx>
> Cc:     Rafael J. Wysocki <rjw@xxxxxxx>
> ---
>  drivers/acpi/power.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- lnx-37-rc3.orig/drivers/acpi/power.c
> +++ lnx-37-rc3/drivers/acpi/power.c
> @@ -473,7 +473,7 @@ int acpi_power_resource_register_device(
>         return ret;
>
>  no_power_resource:
> -       printk(KERN_DEBUG PREFIX "Invalid Power Resource to register!");
> +       printk(KERN_DEBUG PREFIX "Invalid Power Resource to register!\n");

It'd be nice if this and the other similar printks in this file
included a hint about the context, e.g., the device to which it's
related.

>         return -ENODEV;
>  }
>  EXPORT_SYMBOL_GPL(acpi_power_resource_register_device);
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux