Re: [PULL] http://www.kernellabs.com/hg/~stoth/saa7164-dev

2009-09-19 Thread Steven Toth
>> > drivers/media/video/saa7164/saa7164-buffer.c: In function 
>> > ‘saa7164_buffer_alloc’:
>> > drivers/media/video/saa7164/saa7164-buffer.c:110: warning: cast to pointer 
>> > from integer of different size
>> > drivers/media/video/saa7164/saa7164-buffer.c:112: warning: cast to pointer 
>> > from integer of different size
>>
>> Last I checked prior to merge the only 64bit'ism was the warning for the 
>> call to
>> the kernel software demux. Let me look into this today.
>
> Fixed. The error where with i386.

Thanks Mauro,

- Steve

-- 
Steven Toth - Kernel Labs
http://www.kernellabs.com
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PULL] http://www.kernellabs.com/hg/~stoth/saa7164-dev

2009-09-18 Thread Mauro Carvalho Chehab
Em Fri, 18 Sep 2009 11:03:13 -0400
Steven Toth  escreveu:

> On 9/18/09 12:24 AM, Mauro Carvalho Chehab wrote:
> > Em Thu, 17 Sep 2009 20:05:14 -0400
> > Steven Toth  escreveu:
> >
> >>
> >> Mauro,
> >>
> >> Please pull from http://www.kernellabs.com/hg/~stoth/saa7164-dev
> >>
> >>  -  SAA7164: Remove the SAA7164 bus id, no longer required.
> >>  -  SAA7164: Remove the i2c client_attach/detach support, no longer 
> >> required.
> >>  -  SAA7164: Removed bus registration messages from driver startup
> >>
> >>drivers/media/video/saa7164/saa7164-i2c.c |   62 --
> >>include/linux/i2c-id.h|1
> >>2 files changed, 1 insertion(+), 62 deletions(-)
> >>
> >> These fix the i2c attach/detach compilation error and the id assignment 
> >> we'd
> >> previously discussed, as well as reducing slightly the driver verbosity 
> >> during
> >> i2c bus registration.
> >
> > Ok, now it compiles, but  there are still two warnings against upstream, 
> > with x86_64:
> >
> > drivers/media/video/saa7164/saa7164-buffer.c: In function 
> > ‘saa7164_buffer_alloc’:
> > drivers/media/video/saa7164/saa7164-buffer.c:110: warning: cast to pointer 
> > from integer of different size
> > drivers/media/video/saa7164/saa7164-buffer.c:112: warning: cast to pointer 
> > from integer of different size
> 
> Last I checked prior to merge the only 64bit'ism was the warning for the call 
> to 
> the kernel software demux. Let me look into this today.

Fixed. The error where with i386.
> 
> - Steve
> 




Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PULL] http://www.kernellabs.com/hg/~stoth/saa7164-dev

2009-09-18 Thread Steven Toth

On 9/18/09 4:27 AM, Hot Wheelz wrote:

Hi Mauro,

What about saa7162

How far from being complete is that?


For some reason this email went specifically to me, not Mauro (who I've cc'd 
in).

The saa7164 tree does not support he saa7162, it was never designed to do so.

Regards,

--
Steven Toth - Kernel Labs
http://www.kernellabs.com
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PULL] http://www.kernellabs.com/hg/~stoth/saa7164-dev

2009-09-18 Thread Steven Toth

On 9/18/09 12:24 AM, Mauro Carvalho Chehab wrote:

Em Thu, 17 Sep 2009 20:05:14 -0400
Steven Toth  escreveu:



Mauro,

Please pull from http://www.kernellabs.com/hg/~stoth/saa7164-dev

 -  SAA7164: Remove the SAA7164 bus id, no longer required.
 -  SAA7164: Remove the i2c client_attach/detach support, no longer 
required.
 -  SAA7164: Removed bus registration messages from driver startup

   drivers/media/video/saa7164/saa7164-i2c.c |   62 --
   include/linux/i2c-id.h|1
   2 files changed, 1 insertion(+), 62 deletions(-)

These fix the i2c attach/detach compilation error and the id assignment we'd
previously discussed, as well as reducing slightly the driver verbosity during
i2c bus registration.


Ok, now it compiles, but  there are still two warnings against upstream, with 
x86_64:

drivers/media/video/saa7164/saa7164-buffer.c: In function 
‘saa7164_buffer_alloc’:
drivers/media/video/saa7164/saa7164-buffer.c:110: warning: cast to pointer from 
integer of different size
drivers/media/video/saa7164/saa7164-buffer.c:112: warning: cast to pointer from 
integer of different size


Last I checked prior to merge the only 64bit'ism was the warning for the call to 
the kernel software demux. Let me look into this today.


- Steve

--
Steven Toth - Kernel Labs
http://www.kernellabs.com
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PULL] http://www.kernellabs.com/hg/~stoth/saa7164-dev

2009-09-17 Thread Mauro Carvalho Chehab
Em Thu, 17 Sep 2009 20:05:14 -0400
Steven Toth  escreveu:

> 
> Mauro,
> 
> Please pull from http://www.kernellabs.com/hg/~stoth/saa7164-dev
> 
> -  SAA7164: Remove the SAA7164 bus id, no longer required.
> -  SAA7164: Remove the i2c client_attach/detach support, no longer 
> required.
> -  SAA7164: Removed bus registration messages from driver startup
> 
>   drivers/media/video/saa7164/saa7164-i2c.c |   62 --
>   include/linux/i2c-id.h|1
>   2 files changed, 1 insertion(+), 62 deletions(-)
> 
> These fix the i2c attach/detach compilation error and the id assignment we'd 
> previously discussed, as well as reducing slightly the driver verbosity 
> during 
> i2c bus registration.

Ok, now it compiles, but  there are still two warnings against upstream, with 
x86_64:

drivers/media/video/saa7164/saa7164-buffer.c: In function 
‘saa7164_buffer_alloc’:
drivers/media/video/saa7164/saa7164-buffer.c:110: warning: cast to pointer from 
integer of different size
drivers/media/video/saa7164/saa7164-buffer.c:112: warning: cast to pointer from 
integer of different size

> 
> Thanks,
> 




Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PULL] http://www.kernellabs.com/hg/~stoth/saa7164-dev

2009-09-17 Thread Steven Toth


Mauro,

Please pull from http://www.kernellabs.com/hg/~stoth/saa7164-dev

   -  SAA7164: Remove the SAA7164 bus id, no longer required.
   -  SAA7164: Remove the i2c client_attach/detach support, no longer required.
   -  SAA7164: Removed bus registration messages from driver startup

 drivers/media/video/saa7164/saa7164-i2c.c |   62 --
 include/linux/i2c-id.h|1
 2 files changed, 1 insertion(+), 62 deletions(-)

These fix the i2c attach/detach compilation error and the id assignment we'd 
previously discussed, as well as reducing slightly the driver verbosity during 
i2c bus registration.


Thanks,

--
Steven Toth - Kernel Labs
http://www.kernellabs.com
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html