Re: [PULL] generic image bounds setting and alignment function

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

 



On Tue, 16 Jun 2009, Mauro Carvalho Chehab wrote:
> Em Tue, 16 Jun 2009 17:57:20 +0200 (CEST)
> Guennadi Liakhovetski <g.liakhovetski@xxxxxx> escreveu:
> > On Sat, 30 May 2009, Trent Piepho wrote:
> > +	if (walign + halign < salign) {
> > +		/* Max walign where there is still a valid width */
> > +		unsigned int wmaxa = __fls(wmax ^ (wmin - 1));
> >
> > I cannot follow correctness of the above, sorry. Take a simple example:
> > wmax=0x7f, wmin=7, wmaxa = __fls(0x7f ^ 6) = __fls(0x79) = 0. And the
> > comment says it's the "maximum walign where there is still a valid width."
> > What am I missing?
> >
> > +
> > +		/* up the smaller alignment until we have enough */
> > +		do {
> > +			if (walign <= halign && walign < wmaxa) {
> >
>
> As I'm still cooking the patches, I prefer to postpone the align ones until we
> are comfortable with them.
>
> Trent,
>
> Could you please take a look on the above comments

There is no bug with the wmaxa code, but there is a different bug
elsewhere.

Please pull from http://linuxtv.org/hg/~tap/fix

for the following changeset:

01/01: v4l2: Fix flaw in alignment code
http://linuxtv.org/hg/~tap/fix?cmd=changeset;node=4ef7fb102b6c
--
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