From mboxrd@z Thu Jan 1 00:00:00 1970 From: g.liakhovetski@gmx.de (Guennadi Liakhovetski) Date: Fri, 4 Sep 2009 21:05:38 +0200 (CEST) Subject: [PATCH] Add RGB555X and RGB565X formats to pxa-camera In-Reply-To: <4A76DF29.1050008@compulab.co.il> References: <200908031031.00676.marek.vasut@gmail.com> <4A76CB7C.10401@gmail.com> <4A76DF29.1050008@compulab.co.il> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, 3 Aug 2009, Mike Rapoport wrote: > > 2. Mike, while reviewing this patch I came across code in > > pxa_camera_setup_cicr(), introduced by your earlier patch: > > > > case V4L2_PIX_FMT_RGB555: > > cicr1 |= CICR1_RGB_BPP_VAL(1) | CICR1_RGBT_CONV_VAL(2) | > > CICR1_TBIT | CICR1_COLOR_SP_VAL(1); > > break; > > > > Why are you enabling the RGB to RGBT conversion here unconditionally? > > Generally, what are the advantages of configuring CICR1 for a specific RGB > > format compared to using just a raw capture? Do I understand it right, > > that ATM we are not using any of those features? > > As far as I remember I've tried to overlay the captured imagery using pxa > overlay1. Most probably it's left here after those tries. Mike, could you, please, verify that those bits are indeed unneeded and provide patch to remove them? Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/