Hi Marcio On Tue, 23 Apr 2013, Marcio Campos de Lima wrote: > Hi Guennadi > > What is the Linux version that has all of your patches already applied? > Where can I download it? You can pull the tmp-mt9p031-dontuse branch of git://linuxtv.org/gliakhovetski/v4l-dvb.git I'd be grateful if you let me know, when you've pulled it. I'd like to remove it again ASAP, because this is only a temporary branch created only for you to play with. Nobody else should use it, most commits in it will not go into the mainline. If you don't tell me anything I'll remove it anyway within the next couple of days. Thanks Guennadi > > Thanks > Marcio > > On 19/04/13 10:42, "Guennadi Liakhovetski" <g.liakhovetski@xxxxxx> wrote: > > >On Thu, 18 Apr 2013, Marcio Campos de Lima wrote: > > > >> Hi Guennadi > >> > >> Thanks a lot for your attention. > >> I think I cannot apply the patches. My Linux sources, downloaded from > >> www.at91.com, does not have the V4l2-async.h header and, I suppose, many > >> others headers. The MT9P031 sources have been modified and it is in a > >> different tree. Can you tell me where I can download an already patched > >> Kernel 3.6.9 which I can add theses functionality to the driver I am > >>using? > > > >Hm, no, sorry, today isn't your lucky day. I don't think I can help you > >with a 3.6 kernel. > > > >Thanks > >Guennadi > > > >> By the way, our MT9P031 sensor board has 10-bit format. I have fixed > >> already. > >> > >> Regards > >> MArcio > >> > >> On 18/04/13 19:44, "Guennadi Liakhovetski" <g.liakhovetski@xxxxxx> > >>wrote: > >> > >> >Hi Marcio > >> > > >> >On Thu, 18 Apr 2013, Marcio Campos de Lima wrote: > >> > > >> >> >Hi > >> >> > > >> >> >I am porting the MT9P031 sensor device driver for a custom designed > >> >>board > >> >> >based at the AT91SAM9M45-EK development board and Linux 3.6.9. > >> >> >The driver detects the sensor but does not create /dev/video1. > >> >> > > >> >> >Can anybody help me? > >> > > >> >Congratulations, today is your lucky day :-) I've just posted a > >> >patch-series > >> > > >> > >>>http://thread.gmane.org/gmane.linux.drivers.video-input-infrastructure/6 > >>>35 > >> >04 > >> > > >> >to do exactly what you need. Well, 99% of what you need :) With at91 > >> >you're using the atmel-isi soc-camera camera host driver. Shich isn't > >> >extended by this patch series to support the asynchronous subdevice > >> >registration, but it's rather easy to add, please, have a look at patch > >> >#15 for an example, based on the mx3_camera driver. Then there's a > >>slight > >> >problem of mt9p031 only exporting 12-bit formats. You can "fix" it > >> >internally by substituting 8-bit formats (you are using 8 bits, right?) > >> >instead. We'll need a proper solution at some point. The last patch in > >> >the > >> >series shows how to add support for the mt9p031 sensor to a board. > >> > > >> >Anyway, give it a try, feel free to ask. > >> > > >> >Thanks > >> >Guennadi > >> > > >> >> >Thanks > >> >> >Marcio > >> >> > >> >> This is the probe code fo the driver if this can help: > >> >> > >> >> /* > >> >> > >> > >>>>----------------------------------------------------------------------- > >>>>-- > >> >>-- > >> >> -- > >> >> * Driver initialization and probing > >> >> */ > >> >> > >> >> static int mt9p031_probe(struct i2c_client *client, > >> >> const struct i2c_device_id *did) > >> >> { > >> >> struct mt9p031_platform_data *pdata = client->dev.platform_data; > >> >> struct i2c_adapter *adapter = to_i2c_adapter(client->dev.parent); > >> >> struct mt9p031 *mt9p031; > >> >> unsigned int i; > >> >> int ret; > >> >> > >> >> if (pdata == NULL) { > >> >> dev_err(&client->dev, "No platform data\n"); > >> >> return -EINVAL; > >> >> } > >> >> > >> >> if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_WORD_DATA)) { > >> >> dev_warn(&client->dev, > >> >> "I2C-Adapter doesn't support I2C_FUNC_SMBUS_WORD\n"); > >> >> return -EIO; > >> >> } > >> >> > >> >> mt9p031 = kzalloc(sizeof(*mt9p031), GFP_KERNEL); > >> >> if (mt9p031 == NULL) > >> >> return -ENOMEM; > >> >> > >> >> mt9p031->pdata = pdata; > >> >> mt9p031->output_control = MT9P031_OUTPUT_CONTROL_DEF; > >> >> mt9p031->mode2 = MT9P031_READ_MODE_2_ROW_BLC; > >> >> > >> >> v4l2_ctrl_handler_init(&mt9p031->ctrls, ARRAY_SIZE(mt9p031_ctrls) + > >>4); > >> >> > >> >> v4l2_ctrl_new_std(&mt9p031->ctrls, &mt9p031_ctrl_ops, > >> >> V4L2_CID_EXPOSURE, MT9P031_SHUTTER_WIDTH_MIN, > >> >> MT9P031_SHUTTER_WIDTH_MAX, 1, > >> >> MT9P031_SHUTTER_WIDTH_DEF); > >> >> v4l2_ctrl_new_std(&mt9p031->ctrls, &mt9p031_ctrl_ops, > >> >> V4L2_CID_GAIN, MT9P031_GLOBAL_GAIN_MIN, > >> >> MT9P031_GLOBAL_GAIN_MAX, 1, MT9P031_GLOBAL_GAIN_DEF); > >> >> v4l2_ctrl_new_std(&mt9p031->ctrls, &mt9p031_ctrl_ops, > >> >> V4L2_CID_HFLIP, 0, 1, 1, 0); > >> >> v4l2_ctrl_new_std(&mt9p031->ctrls, &mt9p031_ctrl_ops, > >> >> V4L2_CID_VFLIP, 0, 1, 1, 0); > >> >> > >> >> for (i = 0; i < ARRAY_SIZE(mt9p031_ctrls); ++i) > >> >> v4l2_ctrl_new_custom(&mt9p031->ctrls, &mt9p031_ctrls[i], NULL); > >> >> > >> >> mt9p031->subdev.ctrl_handler = &mt9p031->ctrls; > >> >> > >> >> if (mt9p031->ctrls.error) > >> >> printk(KERN_INFO "%s: control initialization error %d\n", > >> >> __func__, mt9p031->ctrls.error); > >> >> > >> >> mutex_init(&mt9p031->power_lock); > >> >> v4l2_i2c_subdev_init(&mt9p031->subdev, client, &mt9p031_subdev_ops); > >> >> mt9p031->subdev.internal_ops = &mt9p031_subdev_internal_ops; > >> >> > >> >> mt9p031->pad.flags = MEDIA_PAD_FL_SOURCE; > >> >> ret = media_entity_init(&mt9p031->subdev.entity, 1, &mt9p031->pad, > >>0); > >> >> if (ret < 0) > >> >> goto done; > >> >> > >> >> mt9p031->subdev.flags |= V4L2_SUBDEV_FL_HAS_DEVNODE; > >> >> > >> >> mt9p031->crop.width = MT9P031_WINDOW_WIDTH_DEF; > >> >> mt9p031->crop.height = MT9P031_WINDOW_HEIGHT_DEF; > >> >> mt9p031->crop.left = MT9P031_COLUMN_START_DEF; > >> >> mt9p031->crop.top = MT9P031_ROW_START_DEF; > >> >> > >> >> if (mt9p031->pdata->version == MT9P031_MONOCHROME_VERSION) > >> >> mt9p031->format.code = V4L2_MBUS_FMT_Y12_1X12; > >> >> else > >> >> mt9p031->format.code = V4L2_MBUS_FMT_SGRBG12_1X12; > >> >> > >> >> mt9p031->format.width = MT9P031_WINDOW_WIDTH_DEF; > >> >> mt9p031->format.height = MT9P031_WINDOW_HEIGHT_DEF; > >> >> mt9p031->format.field = V4L2_FIELD_NONE; > >> >> mt9p031->format.colorspace = V4L2_COLORSPACE_SRGB; > >> >> isi_set_clk(); > >> >> mt9p031->pdata->ext_freq=21000000; > >> >> mt9p031->pdata->target_freq=48000000; > >> >> ret = mt9p031_pll_get_divs(mt9p031); > >> >> > >> >> done: > >> >> if (ret < 0) { > >> >> v4l2_ctrl_handler_free(&mt9p031->ctrls); > >> >> media_entity_cleanup(&mt9p031->subdev.entity); > >> >> kfree(mt9p031); > >> >> } > >> >> > >> >> return ret; > >> >> } > >> >> > >> >> > >> >> > >> >> -- > >> >> To unsubscribe from this list: send the line "unsubscribe > >>linux-media" > >> >>in > >> >> the body of a message to majordomo@xxxxxxxxxxxxxxx > >> >> More majordomo info at http://vger.kernel.org/majordomo-info.html > >> >> > >> > > >> >--- > >> >Guennadi Liakhovetski, Ph.D. > >> >Freelance Open-Source Software Developer > >> >http://www.open-technology.de/ > >> > >> > > > >--- > >Guennadi Liakhovetski, Ph.D. > >Freelance Open-Source Software Developer > >http://www.open-technology.de/ > > --- 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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
