Re: [PATCH RFC] [media] add Aptina mt9m114 HD digital image sensor driver

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

 



Hi Scott,

On Monday 01 April 2013 17:33:02 Scott Jiang wrote:
> Hi Laurent,
> 
> >> >> +struct mt9m114_reg {
> >> >> +     u16 reg;
> >> >> +     u32 val;
> >> >> +     int width;
> >> >> +};
> >> >> +
> >> >> +enum {
> >> >> +     MT9M114_QVGA,
> >> >> +     MT9M114_VGA,
> >> >> +     MT9M114_WVGA,
> >> >> +     MT9M114_720P,
> >> >> +};
> >> > 
> >> > This is the part I don't like. Instead of hardcoding 4 different
> >> > resolutions and using large register address/value tables, you should
> >> > compute the register values from the image size requested by the user.
> >> 
> >> In fact we get this table with the Aptina development tool. So we only
> >> support fixed resolutions. If we compute each register value, it only
> >> makes the code more complex.
> > 
> > But it also makes the code more useful, as the user won't be limited to
> > the 4 resolutions above.
> 
> The problem is Aptina datasheet doesn't tell us how to calculate these
> values. We only have some register presets.

Have you tried requesting the information from Aptina ?

-- 
Regards,

Laurent Pinchart

--
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




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux