Re: LinuxTv doesn't build anymore after upgrading Ubuntu to 3.13.0-88

2016-06-13 Thread Andreas Matthies


Am 13.06.2016 um 21:01 schrieb Hans Verkuil:

On 06/13/2016 08:52 PM, Andreas Matthies wrote:

But now I get
...
CC [M]  /home/andreas/Downloads/media_build/v4l/uvc_v4l2.o
CC [M]  /home/andreas/Downloads/media_build/v4l/uvc_video.o
/home/andreas/Downloads/media_build/v4l/uvc_video.c: In function
'uvc_endpoint_max_bpi':
/home/andreas/Downloads/media_build/v4l/uvc_video.c:1473:7: error:
'USB_SPEED_SUPER_PLUS' undeclared (first use in this function)
case USB_SPEED_SUPER_PLUS:
 ^

When building for 4.6? I know this fails for older kernels but it should be fine
for the 4.6 kernel.

I'll make a patch fixing this some time this week though.


No, I'm running and (hopefully) building for 3.11:
andreas@andreas-xubuntu:~$ uname -a
Linux andreas-xubuntu 3.13.0-88-generic #135-Ubuntu SMP Wed Jun 8 
21:10:42 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux


I have worked around that bug just by deleting the line using 
USB_SPEED_SUPER_PLUS for now but a clean patch for that would be better.


. Andreas


--
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: LinuxTv doesn't build anymore after upgrading Ubuntu to 3.13.0-88

2016-06-13 Thread Hans Verkuil
On 06/13/2016 08:52 PM, Andreas Matthies wrote:
> But now I get
> ...
>CC [M]  /home/andreas/Downloads/media_build/v4l/uvc_v4l2.o
>CC [M]  /home/andreas/Downloads/media_build/v4l/uvc_video.o
> /home/andreas/Downloads/media_build/v4l/uvc_video.c: In function 
> 'uvc_endpoint_max_bpi':
> /home/andreas/Downloads/media_build/v4l/uvc_video.c:1473:7: error: 
> 'USB_SPEED_SUPER_PLUS' undeclared (first use in this function)
>case USB_SPEED_SUPER_PLUS:
> ^

When building for 4.6? I know this fails for older kernels but it should be fine
for the 4.6 kernel.

I'll make a patch fixing this some time this week though.

Regards,

Hans

> /home/andreas/Downloads/media_build/v4l/uvc_video.c:1473:7: note: each 
> undeclared identifier is reported only once for each function it appears in
> make[3]: *** [/home/andreas/Downloads/media_build/v4l/uvc_video.o] Fehler 1
> make[2]: *** [_module_/home/andreas/Downloads/media_build/v4l] Error 2
> ...
> 
> Am 13.06.2016 um 19:55 schrieb Hans Verkuil:
>> On 06/13/2016 07:14 PM, Andreas Matthies wrote:
>>> Hi.
>>>
>>> Seems that there's a problem in v4.6_i2c_mux.patch. After Ubuntu was
>>> upgraded to 3.13.0-88 I tried to rebuild the tv drivers and get
>>>
>>> make[2]: Entering directory `/home/andreas/Downloads/media_build/linux'
>>> Applying patches for kernel 3.13.0-88-generic
>>> patch -s -f -N -p1 -i ../backports/api_version.patch
>>> patch -s -f -N -p1 -i ../backports/pr_fmt.patch
>>> patch -s -f -N -p1 -i ../backports/debug.patch
>>> patch -s -f -N -p1 -i ../backports/drx39xxj.patch
>>> patch -s -f -N -p1 -i ../backports/v4.6_i2c_mux.patch
>>> 2 out of 23 hunks FAILED
>>> make[2]: *** [apply_patches] Error 1
>> Fixed. Thanks for reporting this.
>>
>> Regards,
>>
>>  Hans
> 
> --
> 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
> 
--
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: LinuxTv doesn't build anymore after upgrading Ubuntu to 3.13.0-88

2016-06-13 Thread Hans Verkuil
On 06/13/2016 08:52 PM, Andreas Matthies wrote:
> But now I get
> ...
>CC [M]  /home/andreas/Downloads/media_build/v4l/uvc_v4l2.o
>CC [M]  /home/andreas/Downloads/media_build/v4l/uvc_video.o
> /home/andreas/Downloads/media_build/v4l/uvc_video.c: In function 
> 'uvc_endpoint_max_bpi':
> /home/andreas/Downloads/media_build/v4l/uvc_video.c:1473:7: error: 
> 'USB_SPEED_SUPER_PLUS' undeclared (first use in this function)
>case USB_SPEED_SUPER_PLUS:
> ^
> /home/andreas/Downloads/media_build/v4l/uvc_video.c:1473:7: note: each 
> undeclared identifier is reported only once for each function it appears in
> make[3]: *** [/home/andreas/Downloads/media_build/v4l/uvc_video.o] Fehler 1
> make[2]: *** [_module_/home/andreas/Downloads/media_build/v4l] Error 2
> ...
> 
> Am 13.06.2016 um 19:55 schrieb Hans Verkuil:
>> On 06/13/2016 07:14 PM, Andreas Matthies wrote:
>>> Hi.
>>>
>>> Seems that there's a problem in v4.6_i2c_mux.patch. After Ubuntu was
>>> upgraded to 3.13.0-88 I tried to rebuild the tv drivers and get
>>>
>>> make[2]: Entering directory `/home/andreas/Downloads/media_build/linux'
>>> Applying patches for kernel 3.13.0-88-generic
>>> patch -s -f -N -p1 -i ../backports/api_version.patch
>>> patch -s -f -N -p1 -i ../backports/pr_fmt.patch
>>> patch -s -f -N -p1 -i ../backports/debug.patch
>>> patch -s -f -N -p1 -i ../backports/drx39xxj.patch
>>> patch -s -f -N -p1 -i ../backports/v4.6_i2c_mux.patch
>>> 2 out of 23 hunks FAILED
>>> make[2]: *** [apply_patches] Error 1
>> Fixed. Thanks for reporting this.
>>
>> Regards,
>>
>>  Hans
> 
> --
> 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
> 
--
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: LinuxTv doesn't build anymore after upgrading Ubuntu to 3.13.0-88

2016-06-13 Thread Andreas Matthies

But now I get
...
  CC [M]  /home/andreas/Downloads/media_build/v4l/uvc_v4l2.o
  CC [M]  /home/andreas/Downloads/media_build/v4l/uvc_video.o
/home/andreas/Downloads/media_build/v4l/uvc_video.c: In function 
'uvc_endpoint_max_bpi':
/home/andreas/Downloads/media_build/v4l/uvc_video.c:1473:7: error: 
'USB_SPEED_SUPER_PLUS' undeclared (first use in this function)

  case USB_SPEED_SUPER_PLUS:
   ^
/home/andreas/Downloads/media_build/v4l/uvc_video.c:1473:7: note: each 
undeclared identifier is reported only once for each function it appears in

make[3]: *** [/home/andreas/Downloads/media_build/v4l/uvc_video.o] Fehler 1
make[2]: *** [_module_/home/andreas/Downloads/media_build/v4l] Error 2
...

Am 13.06.2016 um 19:55 schrieb Hans Verkuil:

On 06/13/2016 07:14 PM, Andreas Matthies wrote:

Hi.

Seems that there's a problem in v4.6_i2c_mux.patch. After Ubuntu was
upgraded to 3.13.0-88 I tried to rebuild the tv drivers and get

make[2]: Entering directory `/home/andreas/Downloads/media_build/linux'
Applying patches for kernel 3.13.0-88-generic
patch -s -f -N -p1 -i ../backports/api_version.patch
patch -s -f -N -p1 -i ../backports/pr_fmt.patch
patch -s -f -N -p1 -i ../backports/debug.patch
patch -s -f -N -p1 -i ../backports/drx39xxj.patch
patch -s -f -N -p1 -i ../backports/v4.6_i2c_mux.patch
2 out of 23 hunks FAILED
make[2]: *** [apply_patches] Error 1

Fixed. Thanks for reporting this.

Regards,

Hans


--
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: LinuxTv doesn't build anymore after upgrading Ubuntu to 3.13.0-88

2016-06-13 Thread Andreas Matthies



Am 13.06.2016 um 19:55 schrieb Hans Verkuil:

On 06/13/2016 07:14 PM, Andreas Matthies wrote:

Hi.

Seems that there's a problem in v4.6_i2c_mux.patch. After Ubuntu was
upgraded to 3.13.0-88 I tried to rebuild the tv drivers and get

make[2]: Entering directory `/home/andreas/Downloads/media_build/linux'
Applying patches for kernel 3.13.0-88-generic
patch -s -f -N -p1 -i ../backports/api_version.patch
patch -s -f -N -p1 -i ../backports/pr_fmt.patch
patch -s -f -N -p1 -i ../backports/debug.patch
patch -s -f -N -p1 -i ../backports/drx39xxj.patch
patch -s -f -N -p1 -i ../backports/v4.6_i2c_mux.patch
2 out of 23 hunks FAILED
make[2]: *** [apply_patches] Error 1

Fixed. Thanks for reporting this.


That was fast! Thanks.
--
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: LinuxTv doesn't build anymore after upgrading Ubuntu to 3.13.0-88

2016-06-13 Thread Hans Verkuil
On 06/13/2016 07:14 PM, Andreas Matthies wrote:
> Hi.
> 
> Seems that there's a problem in v4.6_i2c_mux.patch. After Ubuntu was 
> upgraded to 3.13.0-88 I tried to rebuild the tv drivers and get
> 
> make[2]: Entering directory `/home/andreas/Downloads/media_build/linux'
> Applying patches for kernel 3.13.0-88-generic
> patch -s -f -N -p1 -i ../backports/api_version.patch
> patch -s -f -N -p1 -i ../backports/pr_fmt.patch
> patch -s -f -N -p1 -i ../backports/debug.patch
> patch -s -f -N -p1 -i ../backports/drx39xxj.patch
> patch -s -f -N -p1 -i ../backports/v4.6_i2c_mux.patch
> 2 out of 23 hunks FAILED
> make[2]: *** [apply_patches] Error 1

Fixed. Thanks for reporting this.

Regards,

Hans
--
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


LinuxTv doesn't build anymore after upgrading Ubuntu to 3.13.0-88

2016-06-13 Thread Andreas Matthies

Hi.

Seems that there's a problem in v4.6_i2c_mux.patch. After Ubuntu was 
upgraded to 3.13.0-88 I tried to rebuild the tv drivers and get


make[2]: Entering directory `/home/andreas/Downloads/media_build/linux'
Applying patches for kernel 3.13.0-88-generic
patch -s -f -N -p1 -i ../backports/api_version.patch
patch -s -f -N -p1 -i ../backports/pr_fmt.patch
patch -s -f -N -p1 -i ../backports/debug.patch
patch -s -f -N -p1 -i ../backports/drx39xxj.patch
patch -s -f -N -p1 -i ../backports/v4.6_i2c_mux.patch
2 out of 23 hunks FAILED
make[2]: *** [apply_patches] Error 1

Here's the reject file rtl2832.c.rej:
--- drivers/media/dvb-frontends/rtl2832.c
+++ drivers/media/dvb-frontends/rtl2832.c
@@ -1124,7 +1280,7 @@
 else
 u8tmp = 0x00;

-ret = regmap_update_bits(dev->regmap, 0x061, 0xc0, u8tmp);
+ret = rtl2832_update_bits(client, 0x061, 0xc0, u8tmp);
 if (ret)
 goto err;

@@ -1159,14 +1315,14 @@
 buf[1] = (dev->filters >>  8) & 0xff;
 buf[2] = (dev->filters >> 16) & 0xff;
 buf[3] = (dev->filters >> 24) & 0xff;
-ret = regmap_bulk_write(dev->regmap, 0x062, buf, 4);
+ret = rtl2832_bulk_write(client, 0x062, buf, 4);
 if (ret)
 goto err;

 /* add PID */
 buf[0] = (pid >> 8) & 0xff;
 buf[1] = (pid >> 0) & 0xff;
-ret = regmap_bulk_write(dev->regmap, 0x066 + 2 * index, buf, 2);
+ret = rtl2832_bulk_write(client, 0x066 + 2 * index, buf, 2);
 if (ret)
 goto err;

And here's what the source file contains for the first reject:
...
else
u8tmp = 0x00;

if (dev->slave_ts)
ret = regmap_update_bits(dev->regmap, 0x021, 0xc0, u8tmp);
else
ret = regmap_update_bits(dev->regmap, 0x061, 0xc0, u8tmp);
if (ret)
goto err;
...

Hope you can make the drivers compile again soon.

. Andreas



--
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