[v5,2/2] Add support for mt9p031 (LI-5M03 module) in Beagleboard xM.

Message ID 1306835210-1345-2-git-send-email-javier.martin@vista-silicon.com (mailing list archive)
State Superseded, archived
Headers

Commit Message

Javier Martin May 31, 2011, 9:46 a.m. UTC
  Since isp clocks have not been exposed yet, this patch
includes a temporal solution for testing mt9p031 driver
in Beagleboard xM.

Signed-off-by: Javier Martin <javier.martin@vista-silicon.com>
---
 arch/arm/mach-omap2/Makefile                   |    1 +
 arch/arm/mach-omap2/board-omap3beagle-camera.c |   90 ++++++++++++++++++++++++
 arch/arm/mach-omap2/board-omap3beagle.c        |   55 ++++++++++++++
 3 files changed, 146 insertions(+), 0 deletions(-)
 create mode 100644 arch/arm/mach-omap2/board-omap3beagle-camera.c
  

Comments

Koen Kooi May 31, 2011, 1:23 p.m. UTC | #1
Op 31 mei 2011, om 11:46 heeft Javier Martin het volgende geschreven:

> diff --git a/arch/arm/mach-omap2/board-omap3beagle-camera.c b/arch/arm/mach-omap2/board-omap3beagle-camera.c
> new file mode 100644
> index 0000000..04365b2
> --- /dev/null
> +++ b/arch/arm/mach-omap2/board-omap3beagle-camera.c
> 
> +static int __init beagle_camera_init(void)
> +{
> +	reg_1v8 = regulator_get(NULL, "cam_1v8");
> +	if (IS_ERR(reg_1v8))
> +		pr_err("%s: cannot get cam_1v8 regulator\n", __func__);
> +	else
> +		regulator_enable(reg_1v8);
> +
> +	reg_2v8 = regulator_get(NULL, "cam_2v8");
> +	if (IS_ERR(reg_2v8))
> +		pr_err("%s: cannot get cam_2v8 regulator\n", __func__);
> +	else
> +		regulator_enable(reg_2v8);
> +
> +	omap_register_i2c_bus(2, 100, NULL, 0);
> +	gpio_request(MT9P031_RESET_GPIO, "cam_rst");
> +	gpio_direction_output(MT9P031_RESET_GPIO, 0);
> +	omap3_init_camera(&beagle_isp_platform_data);
> +	return 0;
> +}
> +late_initcall(beagle_camera_init);

There should probably a if (cpu_is_omap3630()) {} wrapped around that, so the camera doesn't get initted on a 3530 beagle.

regards,

Koen--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
  
Koen Kooi May 31, 2011, 1:34 p.m. UTC | #2
root@beagleboardxMC:~# yavta -f SGRBG8 -s 320x240 -n 4 --capture=10 --skip 3 -F `media-ctl -e "OMAP3 ISP CCDC output"`
Device /dev/video2 opened.
Device `OMAP3 ISP CCDC output' on `media' is a video capture device.
Video format set: SGRBG8 (47425247) 320x240 buffer size 76800
Video format: SGRBG8 (47425247) 320x240 buffer size 76800
4 buffers requested.
length: 76800 offset: 0
Buffer 0 mapped at address 0x402cf000.
length: 76800 offset: 77824
Buffer 1 mapped at address 0x402fe000.
length: 76800 offset: 155648
Buffer 2 mapped at address 0x40362000.
length: 76800 offset: 233472
Buffer 3 mapped at address 0x40416000.
0 (0) [-] 4294967295 76800 bytes 167.403289 1306829219.931121 0.002 fps
1 (1) [-] 4294967295 76800 bytes 167.633148 1306829220.160980 4.350 fps
2 (2) [-] 4294967295 76800 bytes 167.744506 1306829220.272308 8.980 fps
3 (3) [-] 4294967295 76800 bytes 167.855865 1306829220.383667 8.980 fps
4 (0) [-] 4294967295 76800 bytes 167.967193 1306829220.495025 8.982 fps
5 (1) [-] 4294967295 76800 bytes 168.078552 1306829220.606384 8.980 fps
6 (2) [-] 4294967295 76800 bytes 168.189910 1306829220.717742 8.980 fps
7 (3) [-] 4294967295 76800 bytes 168.301269 1306829220.829071 8.980 fps
8 (0) [-] 4294967295 76800 bytes 168.412597 1306829220.940429 8.982 fps
9 (1) [-] 4294967295 76800 bytes 168.523956 1306829221.051788 8.980 fps
Captured 10 frames in 1.254212 seconds (7.973134 fps, 612336.670356 B/s).
4 buffers released.

So that seems to be working! I haven't checked the frames yet, but is isn't throwing ISP errors anymore.

Op 31 mei 2011, om 11:46 heeft Javier Martin het volgende geschreven:

> Since isp clocks have not been exposed yet, this patch
> includes a temporal solution for testing mt9p031 driver
> in Beagleboard xM.
> 
> Signed-off-by: Javier Martin <javier.martin@vista-silicon.com>
> ---
> arch/arm/mach-omap2/Makefile                   |    1 +
> arch/arm/mach-omap2/board-omap3beagle-camera.c |   90 ++++++++++++++++++++++++
> arch/arm/mach-omap2/board-omap3beagle.c        |   55 ++++++++++++++
> 3 files changed, 146 insertions(+), 0 deletions(-)
> create mode 100644 arch/arm/mach-omap2/board-omap3beagle-camera.c
> 
> diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
> index 512b152..05cd983 100644
> --- a/arch/arm/mach-omap2/Makefile
> +++ b/arch/arm/mach-omap2/Makefile
> @@ -179,6 +179,7 @@ obj-$(CONFIG_MACH_OMAP_2430SDP)		+= board-2430sdp.o \
> 					   hsmmc.o
> obj-$(CONFIG_MACH_OMAP_APOLLON)		+= board-apollon.o
> obj-$(CONFIG_MACH_OMAP3_BEAGLE)		+= board-omap3beagle.o \
> +					   board-omap3beagle-camera.o \
> 					   hsmmc.o
> obj-$(CONFIG_MACH_DEVKIT8000)     	+= board-devkit8000.o \
>                                            hsmmc.o
> diff --git a/arch/arm/mach-omap2/board-omap3beagle-camera.c b/arch/arm/mach-omap2/board-omap3beagle-camera.c
> new file mode 100644
> index 0000000..04365b2
> --- /dev/null
> +++ b/arch/arm/mach-omap2/board-omap3beagle-camera.c
> @@ -0,0 +1,90 @@
> +#include <linux/gpio.h>
> +#include <linux/regulator/machine.h>
> +
> +#include <plat/i2c.h>
> +
> +#include <media/mt9p031.h>
> +
> +#include "devices.h"
> +#include "../../../drivers/media/video/omap3isp/isp.h"
> +
> +#define MT9P031_RESET_GPIO	98
> +#define MT9P031_XCLK		ISP_XCLK_A
> +
> +static struct regulator *reg_1v8, *reg_2v8;
> +
> +static int beagle_cam_set_xclk(struct v4l2_subdev *subdev, int hz)
> +{
> +	struct isp_device *isp = v4l2_dev_to_isp_device(subdev->v4l2_dev);
> +	int ret;
> +
> +	ret = isp->platform_cb.set_xclk(isp, hz, MT9P031_XCLK);
> +	return 0;
> +}
> +
> +static int beagle_cam_reset(struct v4l2_subdev *subdev, int active)
> +{
> +	/* Set RESET_BAR to !active */
> +	gpio_set_value(MT9P031_RESET_GPIO, !active);
> +
> +	return 0;
> +}
> +
> +static struct mt9p031_platform_data beagle_mt9p031_platform_data = {
> +	.set_xclk               = beagle_cam_set_xclk,
> +	.reset                  = beagle_cam_reset,
> +};
> +
> +static struct i2c_board_info mt9p031_camera_i2c_device = {
> +	I2C_BOARD_INFO("mt9p031", 0x48),
> +	.platform_data = &beagle_mt9p031_platform_data,
> +};
> +
> +static struct isp_subdev_i2c_board_info mt9p031_camera_subdevs[] = {
> +	{
> +		.board_info = &mt9p031_camera_i2c_device,
> +		.i2c_adapter_id = 2,
> +	},
> +	{ NULL, 0, },
> +};
> +
> +static struct isp_v4l2_subdevs_group beagle_camera_subdevs[] = {
> +	{
> +		.subdevs = mt9p031_camera_subdevs,
> +		.interface = ISP_INTERFACE_PARALLEL,
> +		.bus = {
> +				.parallel = {
> +					.data_lane_shift = 0,
> +					.clk_pol = 1,
> +					.bridge = ISPCTRL_PAR_BRIDGE_DISABLE,
> +				}
> +		},
> +	},
> +	{ },
> +};
> +
> +static struct isp_platform_data beagle_isp_platform_data = {
> +	.subdevs = beagle_camera_subdevs,
> +};
> +
> +static int __init beagle_camera_init(void)
> +{
> +	reg_1v8 = regulator_get(NULL, "cam_1v8");
> +	if (IS_ERR(reg_1v8))
> +		pr_err("%s: cannot get cam_1v8 regulator\n", __func__);
> +	else
> +		regulator_enable(reg_1v8);
> +
> +	reg_2v8 = regulator_get(NULL, "cam_2v8");
> +	if (IS_ERR(reg_2v8))
> +		pr_err("%s: cannot get cam_2v8 regulator\n", __func__);
> +	else
> +		regulator_enable(reg_2v8);
> +
> +	omap_register_i2c_bus(2, 100, NULL, 0);
> +	gpio_request(MT9P031_RESET_GPIO, "cam_rst");
> +	gpio_direction_output(MT9P031_RESET_GPIO, 0);
> +	omap3_init_camera(&beagle_isp_platform_data);
> +	return 0;
> +}
> +late_initcall(beagle_camera_init);
> diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
> index 33007fd..c18d21c 100644
> --- a/arch/arm/mach-omap2/board-omap3beagle.c
> +++ b/arch/arm/mach-omap2/board-omap3beagle.c
> @@ -24,12 +24,16 @@
> #include <linux/input.h>
> #include <linux/gpio_keys.h>
> #include <linux/opp.h>
> +#include <linux/i2c.h>
> +#include <linux/mm.h>
> +#include <linux/videodev2.h>
> 
> #include <linux/mtd/mtd.h>
> #include <linux/mtd/partitions.h>
> #include <linux/mtd/nand.h>
> #include <linux/mmc/host.h>
> 
> +#include <linux/gpio.h>
> #include <linux/regulator/machine.h>
> #include <linux/i2c/twl.h>
> 
> @@ -47,6 +51,7 @@
> #include <plat/nand.h>
> #include <plat/usb.h>
> #include <plat/omap_device.h>
> +#include <plat/i2c.h>
> 
> #include "mux.h"
> #include "hsmmc.h"
> @@ -273,6 +278,44 @@ static struct regulator_consumer_supply beagle_vsim_supply = {
> 
> static struct gpio_led gpio_leds[];
> 
> +static struct regulator_consumer_supply beagle_vaux3_supply = {
> +	.supply         = "cam_1v8",
> +};
> +
> +static struct regulator_consumer_supply beagle_vaux4_supply = {
> +	.supply         = "cam_2v8",
> +};
> +
> +/* VAUX3 for CAM_1V8 */
> +static struct regulator_init_data beagle_vaux3 = {
> +	.constraints = {
> +		.min_uV			= 1800000,
> +		.max_uV			= 1800000,
> +		.apply_uV		= true,
> +		.valid_modes_mask	= REGULATOR_MODE_NORMAL
> +					| REGULATOR_MODE_STANDBY,
> +		.valid_ops_mask		= REGULATOR_CHANGE_MODE
> +					| REGULATOR_CHANGE_STATUS,
> +	},
> +	.num_consumer_supplies		= 1,
> +	.consumer_supplies		= &beagle_vaux3_supply,
> +};
> +
> +/* VAUX4 for CAM_2V8 */
> +static struct regulator_init_data beagle_vaux4 = {
> +	.constraints = {
> +		.min_uV			= 1800000,
> +		.max_uV			= 1800000,
> +		.apply_uV		= true,
> +		.valid_modes_mask	= REGULATOR_MODE_NORMAL
> +					| REGULATOR_MODE_STANDBY,
> +		.valid_ops_mask		= REGULATOR_CHANGE_MODE
> +					| REGULATOR_CHANGE_STATUS,
> +	},
> +	.num_consumer_supplies  = 1,
> +	.consumer_supplies      = &beagle_vaux4_supply,
> +};
> +
> static int beagle_twl_gpio_setup(struct device *dev,
> 		unsigned gpio, unsigned ngpio)
> {
> @@ -309,6 +352,15 @@ static int beagle_twl_gpio_setup(struct device *dev,
> 			pr_err("%s: unable to configure EHCI_nOC\n", __func__);
> 	}
> 
> +	if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM) {
> +		/*
> +		 * Power on camera interface - only on pre-production, not
> +		 * needed on production boards
> +		 */
> +		gpio_request(gpio + 2, "CAM_EN");
> +		gpio_direction_output(gpio + 2, 1);
> +	}
> +
> 	/*
> 	 * TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
> 	 * high / others active low)
> @@ -451,6 +503,8 @@ static struct twl4030_platform_data beagle_twldata = {
> 	.vsim		= &beagle_vsim,
> 	.vdac		= &beagle_vdac,
> 	.vpll2		= &beagle_vpll2,
> +	.vaux3          = &beagle_vaux3,
> +	.vaux4          = &beagle_vaux4,
> };
> 
> static struct i2c_board_info __initdata beagle_i2c_boardinfo[] = {
> @@ -658,6 +712,7 @@ static void __init omap3_beagle_init(void)
> {
> 	omap3_mux_init(board_mux, OMAP_PACKAGE_CBB);
> 	omap3_beagle_init_rev();
> +
> 	omap3_beagle_i2c_init();
> 	platform_add_devices(omap3_beagle_devices,
> 			ARRAY_SIZE(omap3_beagle_devices));
> -- 
> 1.7.0.4
> 
> -- 
> You received this message because you are subscribed to the Google Groups "Beagle Board" group.
> To post to this group, send email to beagleboard@googlegroups.com.
> To unsubscribe from this group, send email to beagleboard+unsubscribe@googlegroups.com.
> For more options, visit this group at http://groups.google.com/group/beagleboard?hl=en.
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
  
Javier Martin May 31, 2011, 1:42 p.m. UTC | #3
On 31 May 2011 15:34, Koen Kooi <koen@beagleboard.org> wrote:
> root@beagleboardxMC:~# yavta -f SGRBG8 -s 320x240 -n 4 --capture=10 --skip 3 -F `media-ctl -e "OMAP3 ISP CCDC output"`
> Device /dev/video2 opened.
> Device `OMAP3 ISP CCDC output' on `media' is a video capture device.
> Video format set: SGRBG8 (47425247) 320x240 buffer size 76800
> Video format: SGRBG8 (47425247) 320x240 buffer size 76800
> 4 buffers requested.
> length: 76800 offset: 0
> Buffer 0 mapped at address 0x402cf000.
> length: 76800 offset: 77824
> Buffer 1 mapped at address 0x402fe000.
> length: 76800 offset: 155648
> Buffer 2 mapped at address 0x40362000.
> length: 76800 offset: 233472
> Buffer 3 mapped at address 0x40416000.
> 0 (0) [-] 4294967295 76800 bytes 167.403289 1306829219.931121 0.002 fps
> 1 (1) [-] 4294967295 76800 bytes 167.633148 1306829220.160980 4.350 fps
> 2 (2) [-] 4294967295 76800 bytes 167.744506 1306829220.272308 8.980 fps
> 3 (3) [-] 4294967295 76800 bytes 167.855865 1306829220.383667 8.980 fps
> 4 (0) [-] 4294967295 76800 bytes 167.967193 1306829220.495025 8.982 fps
> 5 (1) [-] 4294967295 76800 bytes 168.078552 1306829220.606384 8.980 fps
> 6 (2) [-] 4294967295 76800 bytes 168.189910 1306829220.717742 8.980 fps
> 7 (3) [-] 4294967295 76800 bytes 168.301269 1306829220.829071 8.980 fps
> 8 (0) [-] 4294967295 76800 bytes 168.412597 1306829220.940429 8.982 fps
> 9 (1) [-] 4294967295 76800 bytes 168.523956 1306829221.051788 8.980 fps
> Captured 10 frames in 1.254212 seconds (7.973134 fps, 612336.670356 B/s).
> 4 buffers released.
>
> So that seems to be working! I haven't checked the frames yet, but is isn't throwing ISP errors anymore.

Great!
Do you have a monochrome version of the same sensor?
  
Guennadi Liakhovetski May 31, 2011, 1:55 p.m. UTC | #4
On Tue, 31 May 2011, Koen Kooi wrote:

> Op 31 mei 2011, om 11:46 heeft Javier Martin het volgende geschreven:
> 
> > diff --git a/arch/arm/mach-omap2/board-omap3beagle-camera.c b/arch/arm/mach-omap2/board-omap3beagle-camera.c
> > new file mode 100644
> > index 0000000..04365b2
> > --- /dev/null
> > +++ b/arch/arm/mach-omap2/board-omap3beagle-camera.c
> > 
> > +static int __init beagle_camera_init(void)
> > +{
> > +	reg_1v8 = regulator_get(NULL, "cam_1v8");
> > +	if (IS_ERR(reg_1v8))
> > +		pr_err("%s: cannot get cam_1v8 regulator\n", __func__);
> > +	else
> > +		regulator_enable(reg_1v8);
> > +
> > +	reg_2v8 = regulator_get(NULL, "cam_2v8");
> > +	if (IS_ERR(reg_2v8))
> > +		pr_err("%s: cannot get cam_2v8 regulator\n", __func__);
> > +	else
> > +		regulator_enable(reg_2v8);
> > +
> > +	omap_register_i2c_bus(2, 100, NULL, 0);
> > +	gpio_request(MT9P031_RESET_GPIO, "cam_rst");
> > +	gpio_direction_output(MT9P031_RESET_GPIO, 0);
> > +	omap3_init_camera(&beagle_isp_platform_data);
> > +	return 0;
> > +}
> > +late_initcall(beagle_camera_init);
> 
> There should probably a if (cpu_is_omap3630()) {} wrapped around that, so the camera doesn't get initted on a 3530 beagle.

...speaking of which - if multiarch kernels are supported by OMAP3 you 
probably want to use something like

	if (!machine_is_omap3_beagle() || !cpu_is_omap3630())
		return;

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
  
Koen Kooi May 31, 2011, 2:01 p.m. UTC | #5
Op 31 mei 2011, om 15:42 heeft javier Martin het volgende geschreven:

> On 31 May 2011 15:34, Koen Kooi <koen@beagleboard.org> wrote:
>> root@beagleboardxMC:~# yavta -f SGRBG8 -s 320x240 -n 4 --capture=10 --skip 3 -F `media-ctl -e "OMAP3 ISP CCDC output"`
>> Device /dev/video2 opened.
>> Device `OMAP3 ISP CCDC output' on `media' is a video capture device.
>> Video format set: SGRBG8 (47425247) 320x240 buffer size 76800
>> Video format: SGRBG8 (47425247) 320x240 buffer size 76800
>> 4 buffers requested.
>> length: 76800 offset: 0
>> Buffer 0 mapped at address 0x402cf000.
>> length: 76800 offset: 77824
>> Buffer 1 mapped at address 0x402fe000.
>> length: 76800 offset: 155648
>> Buffer 2 mapped at address 0x40362000.
>> length: 76800 offset: 233472
>> Buffer 3 mapped at address 0x40416000.
>> 0 (0) [-] 4294967295 76800 bytes 167.403289 1306829219.931121 0.002 fps
>> 1 (1) [-] 4294967295 76800 bytes 167.633148 1306829220.160980 4.350 fps
>> 2 (2) [-] 4294967295 76800 bytes 167.744506 1306829220.272308 8.980 fps
>> 3 (3) [-] 4294967295 76800 bytes 167.855865 1306829220.383667 8.980 fps
>> 4 (0) [-] 4294967295 76800 bytes 167.967193 1306829220.495025 8.982 fps
>> 5 (1) [-] 4294967295 76800 bytes 168.078552 1306829220.606384 8.980 fps
>> 6 (2) [-] 4294967295 76800 bytes 168.189910 1306829220.717742 8.980 fps
>> 7 (3) [-] 4294967295 76800 bytes 168.301269 1306829220.829071 8.980 fps
>> 8 (0) [-] 4294967295 76800 bytes 168.412597 1306829220.940429 8.982 fps
>> 9 (1) [-] 4294967295 76800 bytes 168.523956 1306829221.051788 8.980 fps
>> Captured 10 frames in 1.254212 seconds (7.973134 fps, 612336.670356 B/s).
>> 4 buffers released.
>> 
>> So that seems to be working! I haven't checked the frames yet, but is isn't throwing ISP errors anymore.
> 
> Great!
> Do you have a monochrome version of the same sensor?

I think I only have the colour version, I got it with my leopard355 board way back. 

So what can I do with an unpatched mediactl and unpatched yavta? Is it already possible to point something like mplayer or gstreamer to a v4l2 node and see something? I lost the track of which patch goes where :)

regards,

Koen--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
  
Javier Martin May 31, 2011, 2:07 p.m. UTC | #6
On 31 May 2011 16:01, Koen Kooi <koen@beagleboard.org> wrote:
>
> Op 31 mei 2011, om 15:42 heeft javier Martin het volgende geschreven:
>
>> On 31 May 2011 15:34, Koen Kooi <koen@beagleboard.org> wrote:
>>> root@beagleboardxMC:~# yavta -f SGRBG8 -s 320x240 -n 4 --capture=10 --skip 3 -F `media-ctl -e "OMAP3 ISP CCDC output"`
>>> Device /dev/video2 opened.
>>> Device `OMAP3 ISP CCDC output' on `media' is a video capture device.
>>> Video format set: SGRBG8 (47425247) 320x240 buffer size 76800
>>> Video format: SGRBG8 (47425247) 320x240 buffer size 76800
>>> 4 buffers requested.
>>> length: 76800 offset: 0
>>> Buffer 0 mapped at address 0x402cf000.
>>> length: 76800 offset: 77824
>>> Buffer 1 mapped at address 0x402fe000.
>>> length: 76800 offset: 155648
>>> Buffer 2 mapped at address 0x40362000.
>>> length: 76800 offset: 233472
>>> Buffer 3 mapped at address 0x40416000.
>>> 0 (0) [-] 4294967295 76800 bytes 167.403289 1306829219.931121 0.002 fps
>>> 1 (1) [-] 4294967295 76800 bytes 167.633148 1306829220.160980 4.350 fps
>>> 2 (2) [-] 4294967295 76800 bytes 167.744506 1306829220.272308 8.980 fps
>>> 3 (3) [-] 4294967295 76800 bytes 167.855865 1306829220.383667 8.980 fps
>>> 4 (0) [-] 4294967295 76800 bytes 167.967193 1306829220.495025 8.982 fps
>>> 5 (1) [-] 4294967295 76800 bytes 168.078552 1306829220.606384 8.980 fps
>>> 6 (2) [-] 4294967295 76800 bytes 168.189910 1306829220.717742 8.980 fps
>>> 7 (3) [-] 4294967295 76800 bytes 168.301269 1306829220.829071 8.980 fps
>>> 8 (0) [-] 4294967295 76800 bytes 168.412597 1306829220.940429 8.982 fps
>>> 9 (1) [-] 4294967295 76800 bytes 168.523956 1306829221.051788 8.980 fps
>>> Captured 10 frames in 1.254212 seconds (7.973134 fps, 612336.670356 B/s).
>>> 4 buffers released.
>>>
>>> So that seems to be working! I haven't checked the frames yet, but is isn't throwing ISP errors anymore.
>>
>> Great!
>> Do you have a monochrome version of the same sensor?
>
> I think I only have the colour version, I got it with my leopard355 board way back.
>
> So what can I do with an unpatched mediactl and unpatched yavta? Is it already possible to point something like mplayer or gstreamer to a v4l2 node and see something? I lost the track of which patch goes where :)
>

I don't know, I haven't tried.
I'm still using old yavta + Guennadi's patch to enable stdoutput and
"nc" to view images in my PC with mplayer.
  
Guennadi Liakhovetski May 31, 2011, 2:09 p.m. UTC | #7
On Tue, 31 May 2011, Koen Kooi wrote:

> 
> Op 31 mei 2011, om 15:42 heeft javier Martin het volgende geschreven:
> 
> > On 31 May 2011 15:34, Koen Kooi <koen@beagleboard.org> wrote:
> >> root@beagleboardxMC:~# yavta -f SGRBG8 -s 320x240 -n 4 --capture=10 --skip 3 -F `media-ctl -e "OMAP3 ISP CCDC output"`
> >> Device /dev/video2 opened.
> >> Device `OMAP3 ISP CCDC output' on `media' is a video capture device.
> >> Video format set: SGRBG8 (47425247) 320x240 buffer size 76800
> >> Video format: SGRBG8 (47425247) 320x240 buffer size 76800
> >> 4 buffers requested.
> >> length: 76800 offset: 0
> >> Buffer 0 mapped at address 0x402cf000.
> >> length: 76800 offset: 77824
> >> Buffer 1 mapped at address 0x402fe000.
> >> length: 76800 offset: 155648
> >> Buffer 2 mapped at address 0x40362000.
> >> length: 76800 offset: 233472
> >> Buffer 3 mapped at address 0x40416000.
> >> 0 (0) [-] 4294967295 76800 bytes 167.403289 1306829219.931121 0.002 fps
> >> 1 (1) [-] 4294967295 76800 bytes 167.633148 1306829220.160980 4.350 fps
> >> 2 (2) [-] 4294967295 76800 bytes 167.744506 1306829220.272308 8.980 fps
> >> 3 (3) [-] 4294967295 76800 bytes 167.855865 1306829220.383667 8.980 fps
> >> 4 (0) [-] 4294967295 76800 bytes 167.967193 1306829220.495025 8.982 fps
> >> 5 (1) [-] 4294967295 76800 bytes 168.078552 1306829220.606384 8.980 fps
> >> 6 (2) [-] 4294967295 76800 bytes 168.189910 1306829220.717742 8.980 fps
> >> 7 (3) [-] 4294967295 76800 bytes 168.301269 1306829220.829071 8.980 fps
> >> 8 (0) [-] 4294967295 76800 bytes 168.412597 1306829220.940429 8.982 fps
> >> 9 (1) [-] 4294967295 76800 bytes 168.523956 1306829221.051788 8.980 fps
> >> Captured 10 frames in 1.254212 seconds (7.973134 fps, 612336.670356 B/s).
> >> 4 buffers released.
> >> 
> >> So that seems to be working! I haven't checked the frames yet, but is isn't throwing ISP errors anymore.
> > 
> > Great!
> > Do you have a monochrome version of the same sensor?
> 
> I think I only have the colour version, I got it with my leopard355 board way back. 
> 
> So what can I do with an unpatched mediactl and unpatched yavta? Is it 
> already possible to point something like mplayer or gstreamer to a v4l2 
> node and see something? I lost the track of which patch goes where :)

If you're a lucky owner of an LCD on your bb-xM;) you might be able to do 
something similar, to what's described here

http://download.open-technology.de/BeagleBoard_xM-MT9P031/BBxM-MT9P031.txt

but both yavta and mplayer on the board, while short-circuiting the 
network;) I.e., pipe yavta output directly to mplayer.

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
  
Laurent Pinchart May 31, 2011, 2:46 p.m. UTC | #8
On Tuesday 31 May 2011 15:55:04 Guennadi Liakhovetski wrote:
> On Tue, 31 May 2011, Koen Kooi wrote:
> > Op 31 mei 2011, om 11:46 heeft Javier Martin het volgende geschreven:
> > > diff --git a/arch/arm/mach-omap2/board-omap3beagle-camera.c
> > > b/arch/arm/mach-omap2/board-omap3beagle-camera.c new file mode 100644
> > > index 0000000..04365b2
> > > --- /dev/null
> > > +++ b/arch/arm/mach-omap2/board-omap3beagle-camera.c
> > > 
> > > +static int __init beagle_camera_init(void)
> > > +{
> > > +	reg_1v8 = regulator_get(NULL, "cam_1v8");
> > > +	if (IS_ERR(reg_1v8))
> > > +		pr_err("%s: cannot get cam_1v8 regulator\n", __func__);
> > > +	else
> > > +		regulator_enable(reg_1v8);
> > > +
> > > +	reg_2v8 = regulator_get(NULL, "cam_2v8");
> > > +	if (IS_ERR(reg_2v8))
> > > +		pr_err("%s: cannot get cam_2v8 regulator\n", __func__);
> > > +	else
> > > +		regulator_enable(reg_2v8);
> > > +
> > > +	omap_register_i2c_bus(2, 100, NULL, 0);
> > > +	gpio_request(MT9P031_RESET_GPIO, "cam_rst");
> > > +	gpio_direction_output(MT9P031_RESET_GPIO, 0);
> > > +	omap3_init_camera(&beagle_isp_platform_data);
> > > +	return 0;
> > > +}
> > > +late_initcall(beagle_camera_init);
> > 
> > There should probably a if (cpu_is_omap3630()) {} wrapped around that, so
> > the camera doesn't get initted on a 3530 beagle.
> 
> ...speaking of which - if multiarch kernels are supported by OMAP3 you
> probably want to use something like
> 
> 	if (!machine_is_omap3_beagle() || !cpu_is_omap3630())
> 		return;

Shouldn't you check the Beagleboard version instead? The OMAP3530 has an ISP, 
so there's nothing wrong with it per-se.
  
Guennadi Liakhovetski May 31, 2011, 2:53 p.m. UTC | #9
On Tue, 31 May 2011, Laurent Pinchart wrote:

> On Tuesday 31 May 2011 15:55:04 Guennadi Liakhovetski wrote:
> > On Tue, 31 May 2011, Koen Kooi wrote:
> > > Op 31 mei 2011, om 11:46 heeft Javier Martin het volgende geschreven:
> > > > diff --git a/arch/arm/mach-omap2/board-omap3beagle-camera.c
> > > > b/arch/arm/mach-omap2/board-omap3beagle-camera.c new file mode 100644
> > > > index 0000000..04365b2
> > > > --- /dev/null
> > > > +++ b/arch/arm/mach-omap2/board-omap3beagle-camera.c
> > > > 
> > > > +static int __init beagle_camera_init(void)
> > > > +{
> > > > +	reg_1v8 = regulator_get(NULL, "cam_1v8");
> > > > +	if (IS_ERR(reg_1v8))
> > > > +		pr_err("%s: cannot get cam_1v8 regulator\n", __func__);
> > > > +	else
> > > > +		regulator_enable(reg_1v8);
> > > > +
> > > > +	reg_2v8 = regulator_get(NULL, "cam_2v8");
> > > > +	if (IS_ERR(reg_2v8))
> > > > +		pr_err("%s: cannot get cam_2v8 regulator\n", __func__);
> > > > +	else
> > > > +		regulator_enable(reg_2v8);
> > > > +
> > > > +	omap_register_i2c_bus(2, 100, NULL, 0);
> > > > +	gpio_request(MT9P031_RESET_GPIO, "cam_rst");
> > > > +	gpio_direction_output(MT9P031_RESET_GPIO, 0);
> > > > +	omap3_init_camera(&beagle_isp_platform_data);
> > > > +	return 0;
> > > > +}
> > > > +late_initcall(beagle_camera_init);
> > > 
> > > There should probably a if (cpu_is_omap3630()) {} wrapped around that, so
> > > the camera doesn't get initted on a 3530 beagle.
> > 
> > ...speaking of which - if multiarch kernels are supported by OMAP3 you
> > probably want to use something like
> > 
> > 	if (!machine_is_omap3_beagle() || !cpu_is_omap3630())
> > 		return;
> 
> Shouldn't you check the Beagleboard version instead? The OMAP3530 has an ISP, 
> so there's nothing wrong with it per-se.

No idea whatsoever - in that part I'm just repeating, what the previous 
poster has said:)

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
  
Koen Kooi May 31, 2011, 3:12 p.m. UTC | #10
Op 31 mei 2011, om 16:46 heeft Laurent Pinchart het volgende geschreven:

> On Tuesday 31 May 2011 15:55:04 Guennadi Liakhovetski wrote:
>> On Tue, 31 May 2011, Koen Kooi wrote:
>>> Op 31 mei 2011, om 11:46 heeft Javier Martin het volgende geschreven:
>>>> diff --git a/arch/arm/mach-omap2/board-omap3beagle-camera.c
>>>> b/arch/arm/mach-omap2/board-omap3beagle-camera.c new file mode 100644
>>>> index 0000000..04365b2
>>>> --- /dev/null
>>>> +++ b/arch/arm/mach-omap2/board-omap3beagle-camera.c
>>>> 
>>>> +static int __init beagle_camera_init(void)
>>>> +{
>>>> +	reg_1v8 = regulator_get(NULL, "cam_1v8");
>>>> +	if (IS_ERR(reg_1v8))
>>>> +		pr_err("%s: cannot get cam_1v8 regulator\n", __func__);
>>>> +	else
>>>> +		regulator_enable(reg_1v8);
>>>> +
>>>> +	reg_2v8 = regulator_get(NULL, "cam_2v8");
>>>> +	if (IS_ERR(reg_2v8))
>>>> +		pr_err("%s: cannot get cam_2v8 regulator\n", __func__);
>>>> +	else
>>>> +		regulator_enable(reg_2v8);
>>>> +
>>>> +	omap_register_i2c_bus(2, 100, NULL, 0);
>>>> +	gpio_request(MT9P031_RESET_GPIO, "cam_rst");
>>>> +	gpio_direction_output(MT9P031_RESET_GPIO, 0);
>>>> +	omap3_init_camera(&beagle_isp_platform_data);
>>>> +	return 0;
>>>> +}
>>>> +late_initcall(beagle_camera_init);
>>> 
>>> There should probably a if (cpu_is_omap3630()) {} wrapped around that, so
>>> the camera doesn't get initted on a 3530 beagle.
>> 
>> ...speaking of which - if multiarch kernels are supported by OMAP3 you
>> probably want to use something like
>> 
>> 	if (!machine_is_omap3_beagle() || !cpu_is_omap3630())
>> 		return;
> 
> Shouldn't you check the Beagleboard version instead? The OMAP3530 has an ISP, 
> so there's nothing wrong with it per-se.

It has an ISP, but the pins aren't brought out, so you will never be able to use it. We could check the version, but that will look like:

if (omap3_beagle_get_rev() = OMAP3BEAGLE_BOARD_XM || omap3_beagle_get_rev() = OMAP3BEAGLE_BOARD_XMC || omap3_beagle_get_rev() = OMAP3BEAGLE_BOARD_XMD )

or check if you're not on     OMAP3BEAGLE_BOARD_AXBX, OMAP3BEAGLE_BOARD_C1_3 or OMAP3BEAGLE_BOARD_C4. I find the 3630 check a lot simpler :)

regards,

Koen



regards,

Koen--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
  
Koen Kooi May 31, 2011, 3:54 p.m. UTC | #11
Op 31 mei 2011, om 16:09 heeft Guennadi Liakhovetski het volgende geschreven:

> On Tue, 31 May 2011, Koen Kooi wrote:
> 
>> 
>> Op 31 mei 2011, om 15:42 heeft javier Martin het volgende geschreven:
>> 
>>> On 31 May 2011 15:34, Koen Kooi <koen@beagleboard.org> wrote:
>>>> root@beagleboardxMC:~# yavta -f SGRBG8 -s 320x240 -n 4 --capture=10 --skip 3 -F `media-ctl -e "OMAP3 ISP CCDC output"`
>>>> Device /dev/video2 opened.
>>>> Device `OMAP3 ISP CCDC output' on `media' is a video capture device.
>>>> Video format set: SGRBG8 (47425247) 320x240 buffer size 76800
>>>> Video format: SGRBG8 (47425247) 320x240 buffer size 76800
>>>> 4 buffers requested.
>>>> length: 76800 offset: 0
>>>> Buffer 0 mapped at address 0x402cf000.
>>>> length: 76800 offset: 77824
>>>> Buffer 1 mapped at address 0x402fe000.
>>>> length: 76800 offset: 155648
>>>> Buffer 2 mapped at address 0x40362000.
>>>> length: 76800 offset: 233472
>>>> Buffer 3 mapped at address 0x40416000.
>>>> 0 (0) [-] 4294967295 76800 bytes 167.403289 1306829219.931121 0.002 fps
>>>> 1 (1) [-] 4294967295 76800 bytes 167.633148 1306829220.160980 4.350 fps
>>>> 2 (2) [-] 4294967295 76800 bytes 167.744506 1306829220.272308 8.980 fps
>>>> 3 (3) [-] 4294967295 76800 bytes 167.855865 1306829220.383667 8.980 fps
>>>> 4 (0) [-] 4294967295 76800 bytes 167.967193 1306829220.495025 8.982 fps
>>>> 5 (1) [-] 4294967295 76800 bytes 168.078552 1306829220.606384 8.980 fps
>>>> 6 (2) [-] 4294967295 76800 bytes 168.189910 1306829220.717742 8.980 fps
>>>> 7 (3) [-] 4294967295 76800 bytes 168.301269 1306829220.829071 8.980 fps
>>>> 8 (0) [-] 4294967295 76800 bytes 168.412597 1306829220.940429 8.982 fps
>>>> 9 (1) [-] 4294967295 76800 bytes 168.523956 1306829221.051788 8.980 fps
>>>> Captured 10 frames in 1.254212 seconds (7.973134 fps, 612336.670356 B/s).
>>>> 4 buffers released.
>>>> 
>>>> So that seems to be working! I haven't checked the frames yet, but is isn't throwing ISP errors anymore.
>>> 
>>> Great!
>>> Do you have a monochrome version of the same sensor?
>> 
>> I think I only have the colour version, I got it with my leopard355 board way back. 
>> 
>> So what can I do with an unpatched mediactl and unpatched yavta? Is it 
>> already possible to point something like mplayer or gstreamer to a v4l2 
>> node and see something? I lost the track of which patch goes where :)
> 
> If you're a lucky owner of an LCD on your bb-xM;) you might be able to do 
> something similar, to what's described here
> 
> http://download.open-technology.de/BeagleBoard_xM-MT9P031/BBxM-MT9P031.txt
> 
> but both yavta and mplayer on the board, while short-circuiting the 
> network;) I.e., pipe yavta output directly to mplayer.

I tried that and I can say that shining a light into the sensor changes the picture on the screen, but that's about it.  Which part needs changing (subdev, isp, mt9p031, beagle-camera, etc) to enable 'standard' access? E.g. opening cheese in gnome.

regards,

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

Patch

diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
index 512b152..05cd983 100644
--- a/arch/arm/mach-omap2/Makefile
+++ b/arch/arm/mach-omap2/Makefile
@@ -179,6 +179,7 @@  obj-$(CONFIG_MACH_OMAP_2430SDP)		+= board-2430sdp.o \
 					   hsmmc.o
 obj-$(CONFIG_MACH_OMAP_APOLLON)		+= board-apollon.o
 obj-$(CONFIG_MACH_OMAP3_BEAGLE)		+= board-omap3beagle.o \
+					   board-omap3beagle-camera.o \
 					   hsmmc.o
 obj-$(CONFIG_MACH_DEVKIT8000)     	+= board-devkit8000.o \
                                            hsmmc.o
diff --git a/arch/arm/mach-omap2/board-omap3beagle-camera.c b/arch/arm/mach-omap2/board-omap3beagle-camera.c
new file mode 100644
index 0000000..04365b2
--- /dev/null
+++ b/arch/arm/mach-omap2/board-omap3beagle-camera.c
@@ -0,0 +1,90 @@ 
+#include <linux/gpio.h>
+#include <linux/regulator/machine.h>
+
+#include <plat/i2c.h>
+
+#include <media/mt9p031.h>
+
+#include "devices.h"
+#include "../../../drivers/media/video/omap3isp/isp.h"
+
+#define MT9P031_RESET_GPIO	98
+#define MT9P031_XCLK		ISP_XCLK_A
+
+static struct regulator *reg_1v8, *reg_2v8;
+
+static int beagle_cam_set_xclk(struct v4l2_subdev *subdev, int hz)
+{
+	struct isp_device *isp = v4l2_dev_to_isp_device(subdev->v4l2_dev);
+	int ret;
+
+	ret = isp->platform_cb.set_xclk(isp, hz, MT9P031_XCLK);
+	return 0;
+}
+
+static int beagle_cam_reset(struct v4l2_subdev *subdev, int active)
+{
+	/* Set RESET_BAR to !active */
+	gpio_set_value(MT9P031_RESET_GPIO, !active);
+
+	return 0;
+}
+
+static struct mt9p031_platform_data beagle_mt9p031_platform_data = {
+	.set_xclk               = beagle_cam_set_xclk,
+	.reset                  = beagle_cam_reset,
+};
+
+static struct i2c_board_info mt9p031_camera_i2c_device = {
+	I2C_BOARD_INFO("mt9p031", 0x48),
+	.platform_data = &beagle_mt9p031_platform_data,
+};
+
+static struct isp_subdev_i2c_board_info mt9p031_camera_subdevs[] = {
+	{
+		.board_info = &mt9p031_camera_i2c_device,
+		.i2c_adapter_id = 2,
+	},
+	{ NULL, 0, },
+};
+
+static struct isp_v4l2_subdevs_group beagle_camera_subdevs[] = {
+	{
+		.subdevs = mt9p031_camera_subdevs,
+		.interface = ISP_INTERFACE_PARALLEL,
+		.bus = {
+				.parallel = {
+					.data_lane_shift = 0,
+					.clk_pol = 1,
+					.bridge = ISPCTRL_PAR_BRIDGE_DISABLE,
+				}
+		},
+	},
+	{ },
+};
+
+static struct isp_platform_data beagle_isp_platform_data = {
+	.subdevs = beagle_camera_subdevs,
+};
+
+static int __init beagle_camera_init(void)
+{
+	reg_1v8 = regulator_get(NULL, "cam_1v8");
+	if (IS_ERR(reg_1v8))
+		pr_err("%s: cannot get cam_1v8 regulator\n", __func__);
+	else
+		regulator_enable(reg_1v8);
+
+	reg_2v8 = regulator_get(NULL, "cam_2v8");
+	if (IS_ERR(reg_2v8))
+		pr_err("%s: cannot get cam_2v8 regulator\n", __func__);
+	else
+		regulator_enable(reg_2v8);
+
+	omap_register_i2c_bus(2, 100, NULL, 0);
+	gpio_request(MT9P031_RESET_GPIO, "cam_rst");
+	gpio_direction_output(MT9P031_RESET_GPIO, 0);
+	omap3_init_camera(&beagle_isp_platform_data);
+	return 0;
+}
+late_initcall(beagle_camera_init);
diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
index 33007fd..c18d21c 100644
--- a/arch/arm/mach-omap2/board-omap3beagle.c
+++ b/arch/arm/mach-omap2/board-omap3beagle.c
@@ -24,12 +24,16 @@ 
 #include <linux/input.h>
 #include <linux/gpio_keys.h>
 #include <linux/opp.h>
+#include <linux/i2c.h>
+#include <linux/mm.h>
+#include <linux/videodev2.h>
 
 #include <linux/mtd/mtd.h>
 #include <linux/mtd/partitions.h>
 #include <linux/mtd/nand.h>
 #include <linux/mmc/host.h>
 
+#include <linux/gpio.h>
 #include <linux/regulator/machine.h>
 #include <linux/i2c/twl.h>
 
@@ -47,6 +51,7 @@ 
 #include <plat/nand.h>
 #include <plat/usb.h>
 #include <plat/omap_device.h>
+#include <plat/i2c.h>
 
 #include "mux.h"
 #include "hsmmc.h"
@@ -273,6 +278,44 @@  static struct regulator_consumer_supply beagle_vsim_supply = {
 
 static struct gpio_led gpio_leds[];
 
+static struct regulator_consumer_supply beagle_vaux3_supply = {
+	.supply         = "cam_1v8",
+};
+
+static struct regulator_consumer_supply beagle_vaux4_supply = {
+	.supply         = "cam_2v8",
+};
+
+/* VAUX3 for CAM_1V8 */
+static struct regulator_init_data beagle_vaux3 = {
+	.constraints = {
+		.min_uV			= 1800000,
+		.max_uV			= 1800000,
+		.apply_uV		= true,
+		.valid_modes_mask	= REGULATOR_MODE_NORMAL
+					| REGULATOR_MODE_STANDBY,
+		.valid_ops_mask		= REGULATOR_CHANGE_MODE
+					| REGULATOR_CHANGE_STATUS,
+	},
+	.num_consumer_supplies		= 1,
+	.consumer_supplies		= &beagle_vaux3_supply,
+};
+
+/* VAUX4 for CAM_2V8 */
+static struct regulator_init_data beagle_vaux4 = {
+	.constraints = {
+		.min_uV			= 1800000,
+		.max_uV			= 1800000,
+		.apply_uV		= true,
+		.valid_modes_mask	= REGULATOR_MODE_NORMAL
+					| REGULATOR_MODE_STANDBY,
+		.valid_ops_mask		= REGULATOR_CHANGE_MODE
+					| REGULATOR_CHANGE_STATUS,
+	},
+	.num_consumer_supplies  = 1,
+	.consumer_supplies      = &beagle_vaux4_supply,
+};
+
 static int beagle_twl_gpio_setup(struct device *dev,
 		unsigned gpio, unsigned ngpio)
 {
@@ -309,6 +352,15 @@  static int beagle_twl_gpio_setup(struct device *dev,
 			pr_err("%s: unable to configure EHCI_nOC\n", __func__);
 	}
 
+	if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM) {
+		/*
+		 * Power on camera interface - only on pre-production, not
+		 * needed on production boards
+		 */
+		gpio_request(gpio + 2, "CAM_EN");
+		gpio_direction_output(gpio + 2, 1);
+	}
+
 	/*
 	 * TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
 	 * high / others active low)
@@ -451,6 +503,8 @@  static struct twl4030_platform_data beagle_twldata = {
 	.vsim		= &beagle_vsim,
 	.vdac		= &beagle_vdac,
 	.vpll2		= &beagle_vpll2,
+	.vaux3          = &beagle_vaux3,
+	.vaux4          = &beagle_vaux4,
 };
 
 static struct i2c_board_info __initdata beagle_i2c_boardinfo[] = {
@@ -658,6 +712,7 @@  static void __init omap3_beagle_init(void)
 {
 	omap3_mux_init(board_mux, OMAP_PACKAGE_CBB);
 	omap3_beagle_init_rev();
+
 	omap3_beagle_i2c_init();
 	platform_add_devices(omap3_beagle_devices,
 			ARRAY_SIZE(omap3_beagle_devices));