public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Andrew Morton <akpm@osdl.org>
Cc: Stelian Pop <stelian@popies.net>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Linus Torvalds <torvalds@osdl.org>,
	v4l-dvb-maintainer@linuxtv.org
Subject: Re: [PATCH] Fix __ucmpdi2 in v4l2_norm_to_name()
Date: Sun, 07 Jan 2007 09:44:40 -0200	[thread overview]
Message-ID: <1168170280.27419.69.camel@praia> (raw)
In-Reply-To: <20070104151837.1a878a20.akpm@osdl.org>

Em Qui, 2007-01-04 às 15:18 -0800, Andrew Morton escreveu:
> On Thu, 04 Jan 2007 20:59:08 -0200
> Mauro Carvalho Chehab <mchehab@infradead.org> wrote:
> 
> > > The largest value we use here is 0x02000000.  Perhaps v4l2_std_id shouldn't
> > > be 64-bit?
> > Too late to change it to 32 bits. It is at V4L2 userspace API since
> > kernel 2.6.0.
> 
> You could perhaps make it 32-bit internally, and still 64-bit on the
> kernel<->userspace boundary.   64-bit quantities are expensive..
Hmm... there are some discussions currently on v4l ML about the need to
add some standards to support some digital streams, like those used on
webcams. Depending on the result of those discussions, we can need to
use more bits. So, I think it is not worth right now to replace video
std on every place it occurs.

I'm to just do the fix at v4l2-common.c.
> 
> > We can, however use this approach as a workaround, with
> > the proper documentation. I'll handle it after I return from vacations
> > next week.
Ok, I've wrote such patch. I should send today or tomorrow to Linus,
together with other patches.
> 
> Thanks.
Cheers, 
Mauro.


  reply	other threads:[~2007-01-07 11:45 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-04 11:10 [PATCH] Fix __ucmpdi2 in v4l2_norm_to_name() Stelian Pop
2007-01-04 12:09 ` [v4l-dvb-maintainer] " Trent Piepho
2007-01-04 12:53   ` Stelian Pop
2007-01-04 22:48 ` Andrew Morton
2007-01-04 22:59   ` Mauro Carvalho Chehab
2007-01-04 23:18     ` Andrew Morton
2007-01-07 11:44       ` Mauro Carvalho Chehab [this message]
2007-01-15  9:22         ` Stelian Pop
2007-01-04 23:19     ` Stelian Pop
2007-01-05 14:20       ` Segher Boessenkool

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1168170280.27419.69.camel@praia \
    --to=mchehab@infradead.org \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=stelian@popies.net \
    --cc=torvalds@osdl.org \
    --cc=v4l-dvb-maintainer@linuxtv.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox