Re: [PATCH] toshiba_acpi: Only register backlight device when interface is read/write

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

 



On Thu, May 03, 2012 at 09:32:57PM +0100, Matthew Garrett wrote:
> Merged for -next, thanks. I fixed up the conflicts with the the 
> transflective display patch, but someone should probably verify them.

At a glance I see a couple of obvious mistakes, one of which will fail
to build.

>  static int __get_lcd_brightness(struct toshiba_acpi_dev *dev)
>  {
>  	u32 hci_result;
>  	u32 value;
> +	int brightness;

brightness needs to be initialized to 0. As-is it should generate a
warning about using an uninitialized variable.

>  static const struct backlight_ops toshiba_backlight_data = {
> +	.iotns = BL_CORE_SUSPENDRESUME,

Should be options.
--
To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux