linux-kernel - Re: [PATCH 10/16] ARM: vt8500: move platform

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1347397033.2141.4.camel@gitbox>
Date:	Tue, 11 Sep 2012 20:54:03 +0000
From:	Tony Prisk <linux@...sktech.co.nz>
To:	"arnd@...db.de" <arnd@...db.de>
CC:	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux@....linux.org.uk" <linux@....linux.org.uk>,
	"FlorianSchandinat@....de" <FlorianSchandinat@....de>
Subject: Re: [PATCH 10/16] ARM: vt8500: move platform_data definitions

On Tue, 2012-09-11 at 14:59 +0200, Arnd Bergmann wrote:
> Platform data for device drivers should be defined in
> include/linux/platform_data/*.h, not in the architecture
> and platform specific directories.
> 
> This moves such data out of the vt8500 include directories
> 
> Signed-off-by: Arnd Bergmann <arnd@...db.de>
> Cc: Tony Prisk <linux@...sktech.co.nz>
> Cc: Florian Tobias Schandinat <FlorianSchandinat@....de>
> ---
>  arch/arm/mach-vt8500/devices.c                                       |    2 +-
>  drivers/video/vt8500lcdfb.c                                          |    2 +-
>  drivers/video/wm8505fb.c                                             |    2 +-
>  .../vt8500fb.h => include/linux/platform_data/video-vt8500lcdfb.h    |    0
>  4 files changed, 3 insertions(+), 3 deletions(-)
>  rename arch/arm/mach-vt8500/include/mach/vt8500fb.h => include/linux/platform_data/video-vt8500lcdfb.h (100%)
> 
> diff --git a/arch/arm/mach-vt8500/devices.c b/arch/arm/mach-vt8500/devices.c
> index 1fcdc36..82b4bce 100644
> --- a/arch/arm/mach-vt8500/devices.c
> +++ b/arch/arm/mach-vt8500/devices.c
> @@ -23,7 +23,7 @@
>  
>  #include <asm/mach/arch.h>
>  
> -#include <mach/vt8500fb.h>
> +#include <linux/platform_data/video-vt8500lcdfb.h>
>  #include <mach/i8042.h>
>  #include "devices.h"
>  
> diff --git a/drivers/video/vt8500lcdfb.c b/drivers/video/vt8500lcdfb.c
> index 2a5fe6e..66a74f9 100644
> --- a/drivers/video/vt8500lcdfb.c
> +++ b/drivers/video/vt8500lcdfb.c
> @@ -30,7 +30,7 @@
>  #include <linux/platform_device.h>
>  #include <linux/wait.h>
>  
> -#include <mach/vt8500fb.h>
> +#include <linux/platform_data/video-vt8500lcdfb.h>
>  
>  #include "vt8500lcdfb.h"
>  #include "wmt_ge_rops.h"
> diff --git a/drivers/video/wm8505fb.c b/drivers/video/wm8505fb.c
> index c8703bd..ffeff48 100644
> --- a/drivers/video/wm8505fb.c
> +++ b/drivers/video/wm8505fb.c
> @@ -29,7 +29,7 @@
>  #include <linux/platform_device.h>
>  #include <linux/wait.h>
>  
> -#include <mach/vt8500fb.h>
> +#include <linux/platform_data/video-vt8500lcdfb.h>
>  
>  #include "wm8505fb_regs.h"
>  #include "wmt_ge_rops.h"
> diff --git a/arch/arm/mach-vt8500/include/mach/vt8500fb.h b/include/linux/platform_data/video-vt8500lcdfb.h
> similarity index 100%
> rename from arch/arm/mach-vt8500/include/mach/vt8500fb.h
> rename to include/linux/platform_data/video-vt8500lcdfb.h

Acked-by: Tony Prisk <linux@...sktech.co.nz>

Be aware that my current patch series (which has now been finalized)
drops mach-vt8500/devices.c so you might get a conflict depending on the
patch order.