Re: [PATCH RESEND] fujitsu-laptop: Drop unneeded include

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

 



On Wednesday, March 19, 2014 07:38:46 PM Jean Delvare wrote:
> The fujitsu-laptop driver includes <linux/video_output.h> but doesn't
> call any of its functions. Drop the unneeded include to avoid
> unnecessary driver rebuilds.
> 
> Signed-off-by: Jean Delvare <jdelvare@xxxxxxx>
> Acked-by: Jonathan Woithe <jwoithe@xxxxxxxxxx>
> Cc: Matthew Garrett <matthew.garrett@xxxxxxxxxx>

Well, thanks, but why didn't you send it to the same CC list as the original
patch set?  This doesn't make it particularly convenient to handle ...

> ---
> Resending with Rafael in Cc so that he can include it in a larger
> clean-up patch series.
> 
>  drivers/platform/x86/fujitsu-laptop.c |    1 -
>  1 file changed, 1 deletion(-)
> 
> --- linux-3.14-rc6.orig/drivers/platform/x86/fujitsu-laptop.c	2014-03-17 11:31:04.273862743 +0100
> +++ linux-3.14-rc6/drivers/platform/x86/fujitsu-laptop.c	2014-03-17 11:31:19.961145958 +0100
> @@ -66,7 +66,6 @@
>  #include <linux/backlight.h>
>  #include <linux/input.h>
>  #include <linux/kfifo.h>
> -#include <linux/video_output.h>
>  #include <linux/platform_device.h>
>  #include <linux/slab.h>
>  #if defined(CONFIG_LEDS_CLASS) || defined(CONFIG_LEDS_CLASS_MODULE)
> 
> 
> 

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
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