Re: [PATCH v2 3/3] [media] include/media: move platform_data to linux/platform_data/media
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
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
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/{ => linux/platform_data}/media/si4713.h (96%) > r
Re: [PATCH v2 3/3] [media] include/media: move platform_data to linux/platform_data/media
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