Re: [PATCH v2 3/3] [media] include/media: move platform_data to linux/platform_data/media

2015-11-17 Thread Mauro Carvalho Chehab 
Em Mon, 16 Nov 2015 13:28:10 +0100
Arnd Bergmann  escreveu:

> I think we can also move some of the existing platform data headers to the 
> same
> place, but that could be a separate patch:
> 
> $ git grep linux/platform_data drivers/media/
> drivers/media/platform/coda/coda-common.c:#include 
> 
> drivers/media/platform/soc_camera/mx2_camera.c:#include 
> 
> drivers/media/platform/soc_camera/mx3_camera.c:#include 
> 
> drivers/media/platform/soc_camera/mx3_camera.c:#include 
> 
> drivers/media/platform/soc_camera/pxa_camera.c:#include 
> 
> drivers/media/platform/soc_camera/rcar_vin.c:#include 
> 

Fair enough. Just sent a separate patch to address that.

Regards,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2 3/3] [media] include/media: move platform_data to linux/platform_data/media

2015-11-16 Thread Krzysztof Kozlowski 
W dniu 16.11.2015 o 20:00, Mauro Carvalho Chehab pisze:
> Let's not mix platform_data headers with the core headers. Instead, let's
> create a subdir at linux/platform_data and move the headers to that
> common place, adding it to MAINTAINERS.
> 
> The headers were moved with:
>   mkdir include/linux/platform_data/media/; git mv 
> include/media/gpio-ir-recv.h include/media/ir-rx51.h 
> include/media/mmp-camera.h include/media/omap1_camera.h 
> include/media/omap4iss.h include/media/s5p_hdmi.h include/media/si4713.h 
> include/media/sii9234.h include/media/smiapp.h include/media/soc_camera.h 
> include/media/soc_camera_platform.h include/media/timb_radio.h 
> include/media/timb_video.h include/linux/platform_data/media/
> 
> And the references fixed with this script:
> MAIN_DIR="linux/platform_data/"
> PREV_DIR="media/"
> DIRS="media/"
> 
> echo "Checking affected files" >&2
> for i in $DIRS; do
>   for j in $(find include/$MAIN_DIR/$i -type f -name '*.h'); do
>n=`basename $j`
>   git grep -l $n
>   done
> done|sort|uniq >files && (
>   echo "Handling files..." >&2;
>   echo "for i in \$(cat files|grep -v Documentation); do cat \$i | \\";
>   (
>   cd include/$MAIN_DIR;
>   for j in $DIRS; do
>   for i in $(ls $j); do
>   echo "perl -ne 's,(include 
> [\\\"\\<])$PREV_DIR($i)([\\\"\\>]),\1$MAIN_DIR$j\2\3,; print \$_' |\\";
>   done;
>   done;
>   echo "cat > a && mv a \$i; done";
>   );
>   echo "Handling documentation..." >&2;
>   echo "for i in MAINTAINERS \$(cat files); do cat \$i | \\";
>   (
>   cd include/$MAIN_DIR;
>   for j in $DIRS; do
>   for i in $(ls $j); do
>   echo "  perl -ne 
> 's,include/$PREV_DIR($i)\b,include/$MAIN_DIR$j\1,; print \$_' |\\";
>   done;
>   done;
>   echo "cat > a && mv a \$i; done"
>   );
> ) >script && . ./script
> 
> Signed-off-by: Mauro Carvalho Chehab 
> ---
>  Documentation/video4linux/omap4_camera.txt| 2 +-
>  Documentation/video4linux/si4713.txt  | 2 +-
>  MAINTAINERS   | 1 +
>  arch/arm/mach-omap1/include/mach/camera.h | 2 +-
>  arch/arm/mach-omap2/board-rx51-peripherals.c  | 4 ++--
>  arch/arm/plat-samsung/devs.c  | 2 +-

Acked-by: Krzysztof Kozlowski 

Best regards,
Krzysztof

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2 3/3] [media] include/media: move platform_data to linux/platform_data/media

2015-11-16 Thread Arnd Bergmann 
On Monday 16 November 2015 09:00:45 Mauro Carvalho Chehab wrote:
> Let's not mix platform_data headers with the core headers. Instead, let's
> create a subdir at linux/platform_data and move the headers to that
> common place, adding it to MAINTAINERS.
> 

Acked-by: Arnd Bergmann 

I think we can also move some of the existing platform data headers to the same
place, but that could be a separate patch:

$ git grep linux/platform_data drivers/media/
drivers/media/platform/coda/coda-common.c:#include 
drivers/media/platform/soc_camera/mx2_camera.c:#include 

drivers/media/platform/soc_camera/mx3_camera.c:#include 

drivers/media/platform/soc_camera/mx3_camera.c:#include 

drivers/media/platform/soc_camera/pxa_camera.c:#include 

drivers/media/platform/soc_camera/rcar_vin.c:#include 


Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2 3/3] [media] include/media: move platform_data to linux/platform_data/media

2015-11-16 Thread Lee Jones 
On Mon, 16 Nov 2015, Mauro Carvalho Chehab wrote:

> Let's not mix platform_data headers with the core headers. Instead, let's
> create a subdir at linux/platform_data and move the headers to that
> common place, adding it to MAINTAINERS.
> 
> The headers were moved with:
>   mkdir include/linux/platform_data/media/; git mv 
> include/media/gpio-ir-recv.h include/media/ir-rx51.h 
> include/media/mmp-camera.h include/media/omap1_camera.h 
> include/media/omap4iss.h include/media/s5p_hdmi.h include/media/si4713.h 
> include/media/sii9234.h include/media/smiapp.h include/media/soc_camera.h 
> include/media/soc_camera_platform.h include/media/timb_radio.h 
> include/media/timb_video.h include/linux/platform_data/media/
> 
> And the references fixed with this script:
> MAIN_DIR="linux/platform_data/"
> PREV_DIR="media/"
> DIRS="media/"
> 
> echo "Checking affected files" >&2
> for i in $DIRS; do
>   for j in $(find include/$MAIN_DIR/$i -type f -name '*.h'); do
>n=`basename $j`
>   git grep -l $n
>   done
> done|sort|uniq >files && (
>   echo "Handling files..." >&2;
>   echo "for i in \$(cat files|grep -v Documentation); do cat \$i | \\";
>   (
>   cd include/$MAIN_DIR;
>   for j in $DIRS; do
>   for i in $(ls $j); do
>   echo "perl -ne 's,(include 
> [\\\"\\<])$PREV_DIR($i)([\\\"\\>]),\1$MAIN_DIR$j\2\3,; print \$_' |\\";
>   done;
>   done;
>   echo "cat > a && mv a \$i; done";
>   );
>   echo "Handling documentation..." >&2;
>   echo "for i in MAINTAINERS \$(cat files); do cat \$i | \\";
>   (
>   cd include/$MAIN_DIR;
>   for j in $DIRS; do
>   for i in $(ls $j); do
>   echo "  perl -ne 
> 's,include/$PREV_DIR($i)\b,include/$MAIN_DIR$j\1,; print \$_' |\\";
>   done;
>   done;
>   echo "cat > a && mv a \$i; done"
>   );
> ) >script && . ./script
> 
> Signed-off-by: Mauro Carvalho Chehab 
> ---
>  Documentation/video4linux/omap4_camera.txt| 2 +-
>  Documentation/video4linux/si4713.txt  | 2 +-
>  MAINTAINERS   | 1 +
>  arch/arm/mach-omap1/include/mach/camera.h | 2 +-
>  arch/arm/mach-omap2/board-rx51-peripherals.c  | 4 ++--
>  arch/arm/plat-samsung/devs.c  | 2 +-
>  arch/sh/boards/mach-ap325rxa/setup.c  | 2 +-
>  drivers/media/platform/marvell-ccic/mmp-driver.c  | 2 +-
>  drivers/media/platform/s5p-tv/hdmi_drv.c  | 2 +-
>  drivers/media/platform/s5p-tv/sii9234_drv.c   | 2 +-
>  drivers/media/platform/soc_camera/omap1_camera.c  | 2 +-
>  drivers/media/platform/soc_camera/soc_camera_platform.c   | 2 +-
>  drivers/media/platform/timblogiw.c| 2 +-
>  drivers/media/radio/radio-timb.c  | 2 +-
>  drivers/media/radio/si4713/radio-usb-si4713.c | 2 +-
>  drivers/media/radio/si4713/si4713.h   | 2 +-
>  drivers/media/rc/gpio-ir-recv.c   | 2 +-
>  drivers/media/rc/ir-rx51.c| 2 +-
>  drivers/mfd/timberdale.c  | 4 ++--

Acked-by: Lee Jones 
  
>  drivers/staging/media/omap4iss/iss.h  | 2 +-
>  drivers/staging/media/omap4iss/iss_csiphy.h   | 2 +-
>  include/{ => linux/platform_data}/media/gpio-ir-recv.h| 1 -
>  include/{ => linux/platform_data}/media/ir-rx51.h | 0
>  include/{ => linux/platform_data}/media/mmp-camera.h  | 0
>  include/{ => linux/platform_data}/media/omap1_camera.h| 0
>  include/{ => linux/platform_data}/media/omap4iss.h| 0
>  include/{ => linux/platform_data}/media/s5p_hdmi.h| 1 -
>  include/{ => linux/platform_data}/media/si4713.h  | 2 +-
>  include/{ => linux/platform_data}/media/sii9234.h | 0
>  include/{ => linux/platform_data}/media/soc_camera_platform.h | 0
>  include/{ => linux/platform_data}/media/timb_radio.h  | 0
>  include/{ => linux/platform_data}/media/timb_video.h  | 0
>  32 files changed, 24 insertions(+), 25 deletions(-)
>  rename include/{ => linux/platform_data}/media/gpio-ir-recv.h (99%)
>  rename include/{ => linux/platform_data}/media/ir-rx51.h (100%)
>  rename include/{ => linux/platform_data}/media/mmp-camera.h (100%)
>  rename include/{ => linux/platform_data}/media/omap1_camera.h (100%)
>  rename include/{ => linux/platform_data}/media/omap4iss.h (100%)
>  rename include/{ => linux/platform_data}/media/s5p_hdmi.h (99%)
>  rename include/{

[PATCH v2 3/3] [media] include/media: move platform_data to linux/platform_data/media

2015-11-16 Thread Mauro Carvalho Chehab 
Let's not mix platform_data headers with the core headers. Instead, let's
create a subdir at linux/platform_data and move the headers to that
common place, adding it to MAINTAINERS.

The headers were moved with:
mkdir include/linux/platform_data/media/; git mv 
include/media/gpio-ir-recv.h include/media/ir-rx51.h include/media/mmp-camera.h 
include/media/omap1_camera.h include/media/omap4iss.h include/media/s5p_hdmi.h 
include/media/si4713.h include/media/sii9234.h include/media/smiapp.h 
include/media/soc_camera.h include/media/soc_camera_platform.h 
include/media/timb_radio.h include/media/timb_video.h 
include/linux/platform_data/media/

And the references fixed with this script:
MAIN_DIR="linux/platform_data/"
PREV_DIR="media/"
DIRS="media/"

echo "Checking affected files" >&2
for i in $DIRS; do
for j in $(find include/$MAIN_DIR/$i -type f -name '*.h'); do
 n=`basename $j`
git grep -l $n
done
done|sort|uniq >files && (
echo "Handling files..." >&2;
echo "for i in \$(cat files|grep -v Documentation); do cat \$i | \\";
(
cd include/$MAIN_DIR;
for j in $DIRS; do
for i in $(ls $j); do
echo "perl -ne 's,(include 
[\\\"\\<])$PREV_DIR($i)([\\\"\\>]),\1$MAIN_DIR$j\2\3,; print \$_' |\\";
done;
done;
echo "cat > a && mv a \$i; done";
);
echo "Handling documentation..." >&2;
echo "for i in MAINTAINERS \$(cat files); do cat \$i | \\";
(
cd include/$MAIN_DIR;
for j in $DIRS; do
for i in $(ls $j); do
echo "  perl -ne 
's,include/$PREV_DIR($i)\b,include/$MAIN_DIR$j\1,; print \$_' |\\";
done;
done;
echo "cat > a && mv a \$i; done"
);
) >script && . ./script

Signed-off-by: Mauro Carvalho Chehab 
---
 Documentation/video4linux/omap4_camera.txt| 2 +-
 Documentation/video4linux/si4713.txt  | 2 +-
 MAINTAINERS   | 1 +
 arch/arm/mach-omap1/include/mach/camera.h | 2 +-
 arch/arm/mach-omap2/board-rx51-peripherals.c  | 4 ++--
 arch/arm/plat-samsung/devs.c  | 2 +-
 arch/sh/boards/mach-ap325rxa/setup.c  | 2 +-
 drivers/media/platform/marvell-ccic/mmp-driver.c  | 2 +-
 drivers/media/platform/s5p-tv/hdmi_drv.c  | 2 +-
 drivers/media/platform/s5p-tv/sii9234_drv.c   | 2 +-
 drivers/media/platform/soc_camera/omap1_camera.c  | 2 +-
 drivers/media/platform/soc_camera/soc_camera_platform.c   | 2 +-
 drivers/media/platform/timblogiw.c| 2 +-
 drivers/media/radio/radio-timb.c  | 2 +-
 drivers/media/radio/si4713/radio-usb-si4713.c | 2 +-
 drivers/media/radio/si4713/si4713.h   | 2 +-
 drivers/media/rc/gpio-ir-recv.c   | 2 +-
 drivers/media/rc/ir-rx51.c| 2 +-
 drivers/mfd/timberdale.c  | 4 ++--
 drivers/staging/media/omap4iss/iss.h  | 2 +-
 drivers/staging/media/omap4iss/iss_csiphy.h   | 2 +-
 include/{ => linux/platform_data}/media/gpio-ir-recv.h| 1 -
 include/{ => linux/platform_data}/media/ir-rx51.h | 0
 include/{ => linux/platform_data}/media/mmp-camera.h  | 0
 include/{ => linux/platform_data}/media/omap1_camera.h| 0
 include/{ => linux/platform_data}/media/omap4iss.h| 0
 include/{ => linux/platform_data}/media/s5p_hdmi.h| 1 -
 include/{ => linux/platform_data}/media/si4713.h  | 2 +-
 include/{ => linux/platform_data}/media/sii9234.h | 0
 include/{ => linux/platform_data}/media/soc_camera_platform.h | 0
 include/{ => linux/platform_data}/media/timb_radio.h  | 0
 include/{ => linux/platform_data}/media/timb_video.h  | 0
 32 files changed, 24 insertions(+), 25 deletions(-)
 rename include/{ => linux/platform_data}/media/gpio-ir-recv.h (99%)
 rename include/{ => linux/platform_data}/media/ir-rx51.h (100%)
 rename include/{ => linux/platform_data}/media/mmp-camera.h (100%)
 rename include/{ => linux/platform_data}/media/omap1_camera.h (100%)
 rename include/{ => linux/platform_data}/media/omap4iss.h (100%)
 rename include/{ => linux/platform_data}/media/s5p_hdmi.h (99%)
 rename include/{ => linux/platform_data}/media/si4713.h (96%)
 rename include/{ => linux/platform_data}/media/sii9234.h (100%)
 rename include/{ => linux/platform_data}/media/soc_camera_platform.h (100%)
 rename include/{ =>