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/63504 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 majord...@vger.kernel.org > 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/ -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
