RE: [PATCH-V1 2/2] AM3517: Add VPFE Capture driver support to board file

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

 



> -----Original Message-----
> From: Hiremath, Vaibhav
> Sent: Thursday, June 03, 2010 1:50 PM
> To: DebBarma, Tarun Kanti; linux-media@xxxxxxxxxxxxxxx
> Cc: mchehab@xxxxxxxxxx; Karicheri, Muralidharan; linux-
> omap@xxxxxxxxxxxxxxx
> Subject: RE: [PATCH-V1 2/2] AM3517: Add VPFE Capture driver support to
> board file
> 
> 
> > -----Original Message-----
> > From: DebBarma, Tarun Kanti
> > Sent: Thursday, June 03, 2010 1:44 PM
> > To: Hiremath, Vaibhav; linux-media@xxxxxxxxxxxxxxx
> > Cc: mchehab@xxxxxxxxxx; Karicheri, Muralidharan; linux-
> omap@xxxxxxxxxxxxxxx
> > Subject: RE: [PATCH-V1 2/2] AM3517: Add VPFE Capture driver support to
> board
> > file
> >
> >
> > > -----Original Message-----
> > > From: Hiremath, Vaibhav
> > > Sent: Thursday, June 03, 2010 1:41 PM
> > > To: DebBarma, Tarun Kanti; linux-media@xxxxxxxxxxxxxxx
> > > Cc: mchehab@xxxxxxxxxx; Karicheri, Muralidharan; linux-
> > > omap@xxxxxxxxxxxxxxx
> > > Subject: RE: [PATCH-V1 2/2] AM3517: Add VPFE Capture driver support to
> > > board file
> > >
> > >
> > > > -----Original Message-----
> > > > From: DebBarma, Tarun Kanti
> > > > Sent: Thursday, June 03, 2010 12:49 PM
> > > > To: Hiremath, Vaibhav; linux-media@xxxxxxxxxxxxxxx
> > > > Cc: mchehab@xxxxxxxxxx; Karicheri, Muralidharan; linux-
> > > omap@xxxxxxxxxxxxxxx
> > > > Subject: RE: [PATCH-V1 2/2] AM3517: Add VPFE Capture driver support
> to
> > > board
> > > > file
> > > >
> > > > Vaibhav,
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap-
> > > > > owner@xxxxxxxxxxxxxxx] On Behalf Of Hiremath, Vaibhav
> > > > > Sent: Thursday, June 03, 2010 12:12 PM
> > > > > To: linux-media@xxxxxxxxxxxxxxx
> > > > > Cc: mchehab@xxxxxxxxxx; Karicheri, Muralidharan; linux-
> > > > > omap@xxxxxxxxxxxxxxx; Hiremath, Vaibhav
> > > > > Subject: [PATCH-V1 2/2] AM3517: Add VPFE Capture driver support to
> > > board
> > > > > file
> > > > >
> > > > > From: Vaibhav Hiremath <hvaibhav@xxxxxx>
> > > > >
> > > > > Also created vpfe master/slave clock aliases, since naming
> > > > > convention is different in both Davinci and AM3517 devices.
> > > > >
> > > > > Signed-off-by: Vaibhav Hiremath <hvaibhav@xxxxxx>
> > > > > ---
> > > > >  arch/arm/mach-omap2/board-am3517evm.c |  161
> > > > > +++++++++++++++++++++++++++++++++
> > > > >  1 files changed, 161 insertions(+), 0 deletions(-)
> > > > >
> > > > > diff --git a/arch/arm/mach-omap2/board-am3517evm.c
> b/arch/arm/mach-
> > > > > omap2/board-am3517evm.c
> > > > > index c1c4389..f2ff751 100644
> > > > > --- a/arch/arm/mach-omap2/board-am3517evm.c
> > > > > +++ b/arch/arm/mach-omap2/board-am3517evm.c
> > > > > @@ -30,15 +30,168 @@
> > > > >
> > > > >  #include <plat/board.h>
> > > > >  #include <plat/common.h>
> > > > > +#include <plat/control.h>
> > > > >  #include <plat/usb.h>
> > > > >  #include <plat/display.h>
> > > > >
> > > > > +#include <media/tvp514x.h>
> > > > > +#include <media/davinci/vpfe_capture.h>
> > > > > +
> > > > >  #include "mux.h"
> > > > >
> > > > >  #define LCD_PANEL_PWR		176
> > > > >  #define LCD_PANEL_BKLIGHT_PWR	182
> > > > >  #define LCD_PANEL_PWM		181
> > > > >
> > > > > +/*
> > > > > + * VPFE - Video Decoder interface
> > > > > + */
> > > > > +#define TVP514X_STD_ALL		(V4L2_STD_NTSC | V4L2_STD_PAL)
> > > > > +
> > > > > +/* Inputs available at the TVP5146 */
> > > > > +static struct v4l2_input tvp5146_inputs[] = {
> > > > > +	{
> > > > > +		.index	= 0,
> > > > > +		.name	= "Composite",
> > > > > +		.type	= V4L2_INPUT_TYPE_CAMERA,
> > > > > +		.std	= TVP514X_STD_ALL,
> > > > > +	},
> > > > > +	{
> > > > > +		.index	= 1,
> > > > > +		.name	= "S-Video",
> > > > > +		.type	= V4L2_INPUT_TYPE_CAMERA,
> > > > > +		.std	= TVP514X_STD_ALL,
> > > > > +	},
> > > > > +};
> > > > > +
> > > > > +static struct tvp514x_platform_data tvp5146_pdata = {
> > > > > +	.clk_polarity	= 0,
> > > > > +	.hs_polarity	= 1,
> > > > > +	.vs_polarity	= 1
> > > > > +};
> > > > > +
> > > > > +static struct vpfe_route tvp5146_routes[] = {
> > > > > +	{
> > > > > +		.input	= INPUT_CVBS_VI1A,
> > > > > +		.output	= OUTPUT_10BIT_422_EMBEDDED_SYNC,
> > > > > +	},
> > > > > +	{
> > > > > +		.input	= INPUT_SVIDEO_VI2C_VI1C,
> > > > > +		.output	= OUTPUT_10BIT_422_EMBEDDED_SYNC,
> > > > > +	},
> > > > > +};
> > > > > +
> > > > > +static struct vpfe_subdev_info vpfe_sub_devs[] = {
> > > > > +	{
> > > > > +		.name		= "tvp5146",
> > > > > +		.grp_id		= 0,
> > > > > +		.num_inputs	= ARRAY_SIZE(tvp5146_inputs),
> > > > > +		.inputs		= tvp5146_inputs,
> > > > > +		.routes		= tvp5146_routes,
> > > > > +		.can_route	= 1,
> > > > > +		.ccdc_if_params	= {
> > > > > +			.if_type = VPFE_BT656,
> > > > > +			.hdpol	= VPFE_PINPOL_POSITIVE,
> > > > > +			.vdpol	= VPFE_PINPOL_POSITIVE,
> > > > > +		},
> > > > > +		.board_info	= {
> > > > > +			I2C_BOARD_INFO("tvp5146", 0x5C),
> > > > > +			.platform_data = &tvp5146_pdata,
> > > > > +		},
> > > > > +	},
> > > > > +};
> > > > > +
> > > > > +static void am3517_evm_clear_vpfe_intr(int vdint)
> > > > > +{
> > > > > +	unsigned int vpfe_int_clr;
> > > > > +
> > > > > +	vpfe_int_clr = omap_ctrl_readl(AM35XX_CONTROL_LVL_INTR_CLEAR);
> > > > > +
> > > > > +	switch (vdint) {
> > > > > +	/* VD0 interrrupt */
> > > > > +	case INT_35XX_CCDC_VD0_IRQ:
> > > > > +		vpfe_int_clr &= ~AM35XX_VPFE_CCDC_VD0_INT_CLR;
> > > > > +		vpfe_int_clr |= AM35XX_VPFE_CCDC_VD0_INT_CLR;
> > > > > +		break;
> > > > > +	/* VD1 interrrupt */
> > > > > +	case INT_35XX_CCDC_VD1_IRQ:
> > > > > +		vpfe_int_clr &= ~AM35XX_VPFE_CCDC_VD1_INT_CLR;
> > > > > +		vpfe_int_clr |= AM35XX_VPFE_CCDC_VD1_INT_CLR;
> > > > > +		break;
> > > > > +	/* VD2 interrrupt */
> > > > > +	case INT_35XX_CCDC_VD2_IRQ:
> > > > > +		vpfe_int_clr &= ~AM35XX_VPFE_CCDC_VD2_INT_CLR;
> > > > > +		vpfe_int_clr |= AM35XX_VPFE_CCDC_VD2_INT_CLR;
> > > > > +		break;
> > > > > +	/* Clear all interrrupts */
> > > > > +	default:
> > > > > +		vpfe_int_clr &= ~(AM35XX_VPFE_CCDC_VD0_INT_CLR |
> > > > > +				AM35XX_VPFE_CCDC_VD1_INT_CLR |
> > > > > +				AM35XX_VPFE_CCDC_VD2_INT_CLR);
> > > > > +		vpfe_int_clr |= (AM35XX_VPFE_CCDC_VD0_INT_CLR |
> > > > > +				AM35XX_VPFE_CCDC_VD1_INT_CLR |
> > > > > +				AM35XX_VPFE_CCDC_VD2_INT_CLR);
> > > > > +		break;
> > > > > +	}
> > > > > +	omap_ctrl_writel(vpfe_int_clr, AM35XX_CONTROL_LVL_INTR_CLEAR);
> > > > > +	vpfe_int_clr = omap_ctrl_readl(AM35XX_CONTROL_LVL_INTR_CLEAR);
> > > >
> > > > Is it necessary to assign to the local variable (vpfe_int_clr)? If
> not,
> > > we
> > > > can reduce the size of this routine by two assembly instructions:
> > > > One: copying the result to a register
> > > > Two: pushing the register value to stack
> > > >
> > > [Hiremath, Vaibhav] How are you going to achieve this? How are you
> going
> > > to define the switch case values here?
> >
> > [Tarun] I am only referring to the last statement, outside the switch()
> > statement.
> [Hiremath, Vaibhav] Ohhh Ok.
> It is required; actually the read operation is required to push/reflect
> the value written to register. In the past we have seen issues like write
> is not getting reflected immediately leading to spurious interrupts.

[Tarun Kanti DebBarma] 
Well, I understand & agree that the read is needed. What I am saying is whether the assignment to local variable is needed.

> 
> Thanks,
> Vaibhav
> > >
> > > Also currently this covers only VPFE Capture related interrupts but
> this
> > > function may required for other modules which are part of IPSS, like
> HECC,
> > > EMAC and USBOTG.
> > >
> > > Thanks,
> > > Vaibhav
> > >
> > > > -Tarun
> > > >
> > > >
> > > > > +}
> > > > > +
> > > > > +static struct vpfe_config vpfe_cfg = {
> > > > > +	.num_subdevs	= ARRAY_SIZE(vpfe_sub_devs),
> > > > > +	.i2c_adapter_id	= 3,
> > > > > +	.sub_devs	= vpfe_sub_devs,
> > > > > +	.clr_intr	= am3517_evm_clear_vpfe_intr,
> > > > > +	.card_name	= "AM3517 EVM",
> > > > > +	.ccdc		= "DM6446 CCDC",
> > > > > +};
> > > > > +
> > > > > +static struct resource vpfe_resources[] = {
> > > > > +	{
> > > > > +		.start	= INT_35XX_CCDC_VD0_IRQ,
> > > > > +		.end	= INT_35XX_CCDC_VD0_IRQ,
> > > > > +		.flags	= IORESOURCE_IRQ,
> > > > > +	},
> > > > > +	{
> > > > > +		.start	= INT_35XX_CCDC_VD1_IRQ,
> > > > > +		.end	= INT_35XX_CCDC_VD1_IRQ,
> > > > > +		.flags	= IORESOURCE_IRQ,
> > > > > +	},
> > > > > +};
> > > > > +
> > > > > +static u64 vpfe_capture_dma_mask = DMA_BIT_MASK(32);
> > > > > +static struct platform_device vpfe_capture_dev = {
> > > > > +	.name		= CAPTURE_DRV_NAME,
> > > > > +	.id		= -1,
> > > > > +	.num_resources	= ARRAY_SIZE(vpfe_resources),
> > > > > +	.resource	= vpfe_resources,
> > > > > +	.dev = {
> > > > > +		.dma_mask		= &vpfe_capture_dma_mask,
> > > > > +		.coherent_dma_mask	= DMA_BIT_MASK(32),
> > > > > +		.platform_data		= &vpfe_cfg,
> > > > > +	},
> > > > > +};
> > > > > +
> > > > > +static struct resource am3517_ccdc_resource[] = {
> > > > > +	/* CCDC Base address */
> > > > > +	{
> > > > > +		.start	= AM35XX_IPSS_VPFE_BASE,
> > > > > +		.end	= AM35XX_IPSS_VPFE_BASE + 0xffff,
> > > > > +		.flags	= IORESOURCE_MEM,
> > > > > +	},
> > > > > +};
> > > > > +
> > > > > +static struct platform_device am3517_ccdc_dev = {
> > > > > +	.name		= "dm644x_ccdc",
> > > > > +	.id		= -1,
> > > > > +	.num_resources	= ARRAY_SIZE(am3517_ccdc_resource),
> > > > > +	.resource	= am3517_ccdc_resource,
> > > > > +	.dev = {
> > > > > +		.dma_mask		= &vpfe_capture_dma_mask,
> > > > > +		.coherent_dma_mask	= DMA_BIT_MASK(32),
> > > > > +	},
> > > > > +};
> > > > > +
> > > > >  static struct i2c_board_info __initdata am3517evm_i2c_boardinfo[]
> = {
> > > > >  	{
> > > > >  		I2C_BOARD_INFO("s35390a", 0x30),
> > > > > @@ -46,6 +199,7 @@ static struct i2c_board_info __initdata
> > > > > am3517evm_i2c_boardinfo[] = {
> > > > >  	},
> > > > >  };
> > > > >
> > > > > +
> > > > >  /*
> > > > >   * RTC - S35390A
> > > > >   */
> > > > > @@ -261,6 +415,8 @@ static struct omap_board_config_kernel
> > > > > am3517_evm_config[] __initdata = {
> > > > >
> > > > >  static struct platform_device *am3517_evm_devices[] __initdata =
> {
> > > > >  	&am3517_evm_dss_device,
> > > > > +	&am3517_ccdc_dev,
> > > > > +	&vpfe_capture_dev,
> > > > >  };
> > > > >
> > > > >  static void __init am3517_evm_init_irq(void)
> > > > > @@ -313,6 +469,11 @@ static void __init am3517_evm_init(void)
> > > > >
> > > > >  	i2c_register_board_info(1, am3517evm_i2c_boardinfo,
> > > > >  				ARRAY_SIZE(am3517evm_i2c_boardinfo));
> > > > > +
> > > > > +	clk_add_alias("master", "dm644x_ccdc", "master",
> > > > > +			&vpfe_capture_dev.dev);
> > > > > +	clk_add_alias("slave", "dm644x_ccdc", "slave",
> > > > > +			&vpfe_capture_dev.dev);
> > > > >  }
> > > > >
> > > > >  static void __init am3517_evm_map_io(void)
> > > > > --
> > > > > 1.6.2.4
> > > > >
> > > > > --
> > > > > To unsubscribe from this list: send the line "unsubscribe linux-
> omap"
> > > in
> > > > > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > > > > More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux