On Tue, 21 Feb 2012, javier Martin wrote: > On 21 February 2012 09:39, Guennadi Liakhovetski <g.liakhovetski@xxxxxx> wrote: > > Hi Javier > > > > One more thing occurred to me: I don't see anywhere in your patch checking > > for supported pixel (fourcc) formats. I don't think the PRP can resize > > arbitrary formats? Most likely these would be limited to some YUV, and, > > possibly, some RGB formats? > > The PrP can resize every format which is supported as input by the eMMa. > > Currently, the driver supports 2 input formats: RGB565 and YUV422 > (YUYV) (see mx27_emma_prp_table[]). That's not how I understand it. The mx27_emma_prp_table[] array has 2 entries: the first one is indeed configured for RGB565, and the second one is converting input YUV422 to output YUV420. But the former one is not really that specific format, rather it is a generic configuration used as a pass-through mode for generic 16-bit formats. BTW, does that mean, that on i.MX27 the driver currently doesn't support 8-bit formats like Bayer? Thanks Guennadi > Since the commit of resizing registers is done in the stream_start > callback this makes sure that resizing won't be applied to unknown > formats. > > Regards. > -- > Javier Martin > Vista Silicon S.L. > CDTUC - FASE C - Oficina S-345 > Avda de los Castros s/n > 39005- Santander. Cantabria. Spain > +34 942 25 32 60 > www.vista-silicon.com --- 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
