Re: [PATCH 26/41] ACPI: video: Fix null pointer in appledisplay driver

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

 



On Tuesday 13 February 2007 00:49, Len Brown wrote:
> From: Michael Hanselmann <linux-kernel@xxxxxxxxx>
> 
> commit "ACPI: video: Add dev argument for backlight_device_register"
> 519ab5f2be65b72cf12ae99c89752bbe79b44df6
> broke the apple display driver.
> 
> Signed-off-by: Michael Hanselmann <linux-kernel@xxxxxxxxx>
> Signed-off-by: Len Brown <len.brown@xxxxxxxxx>
> ---
>  drivers/usb/misc/appledisplay.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/usb/misc/appledisplay.c b/drivers/usb/misc/appledisplay.c
> index a7932a7..31ab83c 100644
> --- a/drivers/usb/misc/appledisplay.c
> +++ b/drivers/usb/misc/appledisplay.c
> @@ -281,7 +281,7 @@ static int appledisplay_probe(struct usb_interface *iface,
>  	/* Register backlight device */
>  	snprintf(bl_name, sizeof(bl_name), "appledisplay%d",
>  		atomic_inc_return(&count_displays) - 1);
> -	pdata->bd = backlight_device_register(bl_name, NULL, NULL,
> +	pdata->bd = backlight_device_register(bl_name, NULL, pdata
>  						&appledisplay_bl_data);
>  	if (IS_ERR(pdata->bd)) {
>  		err("appledisplay: Backlight registration failed");

huh, this version of the patch is missing a comma.
anyway, it is fixed now.

cheers,
-Len
-
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