mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* Re: macbook webcam no longer works on .35-rc
       [not found] <1277932269.11050.1.camel@jlt3.sipsolutions.net>
@ 2010-07-02  7:26 ` Johannes Berg
  2010-07-05  7:28   ` Laurent Pinchart
  2010-07-05  7:31   ` Laurent Pinchart
  0 siblings, 2 replies; 7+ messages in thread
From: Johannes Berg @ 2010-07-02  7:26 UTC (permalink / raw)
  To: Laurent Pinchart; +Cc: linux-media, Rafael J. Wysocki, LKML, Linus Torvalds

On Wed, 2010-06-30 at 23:11 +0200, Johannes Berg wrote:
> I'm pretty sure this was a regression in .34, but haven't checked right
> now, can bisect when I find time but wanted to inquire first if somebody
> had ideas. All I get is:
> 
> [57372.078968] uvcvideo: Failed to query (130) UVC control 5 (unit 3) :
> -32 (exp. 1).

Didn't have to bisect it. The problem is caused by

commit 59529081e092506edb81a42d914e2d0522f65ca7
Author: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date:   Sat Jan 23 06:30:20 2010 -0300

    V4L/DVB: uvcvideo: Cache control min, max, res and def query results

which reverts cleanly, but then the driver doesn't compile and one also
needs to revert

commit cf7a50eeb6f462a0b7d1619fcb27a727a2981769
Author: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date:   Sun Apr 25 16:27:14 2010 -0300

    V4L/DVB: uvcvideo: Prevent division by 0 when control step value is 0

and

commit e54532e591cd5b9ce77dbc8d9786ae9f600f101a
Author: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date:   Sat Jan 23 07:07:53 2010 -0300

    V4L/DVB: uvcvideo: Clamp control values to the minimum and maximum values


in that order.

These are commits introduced in the .34/.35 cycles. Please fix.

Can we revert things that are regressions pre-dating the current kernel
version?

johannes


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: macbook webcam no longer works on .35-rc
  2010-07-02  7:26 ` macbook webcam no longer works on .35-rc Johannes Berg
@ 2010-07-05  7:28   ` Laurent Pinchart
  2010-07-05  8:15     ` Johannes Berg
  2010-07-05  7:31   ` Laurent Pinchart
  1 sibling, 1 reply; 7+ messages in thread
From: Laurent Pinchart @ 2010-07-05  7:28 UTC (permalink / raw)
  To: Johannes Berg; +Cc: linux-media, Rafael J. Wysocki, LKML, Linus Torvalds

Hi Johannes,

On Friday 02 July 2010 09:26:15 Johannes Berg wrote:
> On Wed, 2010-06-30 at 23:11 +0200, Johannes Berg wrote:
> > I'm pretty sure this was a regression in .34, but haven't checked right
> > now, can bisect when I find time but wanted to inquire first if somebody
> > had ideas. All I get is:
> > 
> > [57372.078968] uvcvideo: Failed to query (130) UVC control 5 (unit 3) :
> > -32 (exp. 1).

Does it prevent your camera from working, or does it "just" print annoying 
messages to the kernel log ?

-- 
Regards,

Laurent Pinchart

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: macbook webcam no longer works on .35-rc
  2010-07-02  7:26 ` macbook webcam no longer works on .35-rc Johannes Berg
  2010-07-05  7:28   ` Laurent Pinchart
@ 2010-07-05  7:31   ` Laurent Pinchart
  1 sibling, 0 replies; 7+ messages in thread
From: Laurent Pinchart @ 2010-07-05  7:31 UTC (permalink / raw)
  To: Johannes Berg; +Cc: linux-media, Rafael J. Wysocki, LKML, Linus Torvalds

Hi Johannes,

On Friday 02 July 2010 09:26:15 Johannes Berg wrote:
> On Wed, 2010-06-30 at 23:11 +0200, Johannes Berg wrote:
> > I'm pretty sure this was a regression in .34, but haven't checked right
> > now, can bisect when I find time but wanted to inquire first if somebody
> > had ideas. All I get is:
> > 
> > [57372.078968] uvcvideo: Failed to query (130) UVC control 5 (unit 3) :
> > -32 (exp. 1).

Is that the only error messages printed by the driver to the kernel log, or do 
you get other similar ones ? If you get others, please report them.

Could you please send me the output of lsusb -v for your camera ?

-- 
Regards,

Laurent Pinchart

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: macbook webcam no longer works on .35-rc
  2010-07-05  7:28   ` Laurent Pinchart
@ 2010-07-05  8:15     ` Johannes Berg
  2010-07-05  8:23       ` Laurent Pinchart
  0 siblings, 1 reply; 7+ messages in thread
From: Johannes Berg @ 2010-07-05  8:15 UTC (permalink / raw)
  To: Laurent Pinchart; +Cc: linux-media, Rafael J. Wysocki, LKML, Linus Torvalds

On Mon, 2010-07-05 at 09:28 +0200, Laurent Pinchart wrote:
> Hi Johannes,
> 
> On Friday 02 July 2010 09:26:15 Johannes Berg wrote:
> > On Wed, 2010-06-30 at 23:11 +0200, Johannes Berg wrote:
> > > I'm pretty sure this was a regression in .34, but haven't checked right
> > > now, can bisect when I find time but wanted to inquire first if somebody
> > > had ideas. All I get is:
> > > 
> > > [57372.078968] uvcvideo: Failed to query (130) UVC control 5 (unit 3) :
> > > -32 (exp. 1).
> 
> Does it prevent your camera from working, or does it "just" print annoying 
> messages to the kernel log ?

It prevents it from working, and it's the only error message since the
tool/driver/libv4l/whatever just gives up after getting -EIO.

lsusb -v is:


Bus 001 Device 002: ID 05ac:8507 Apple, Inc. 
Device Descriptor:
  bLength                18
  bDescriptorType         1
  bcdUSB               2.00
  bDeviceClass          239 Miscellaneous Device
  bDeviceSubClass         2 ?
  bDeviceProtocol         1 Interface Association
  bMaxPacketSize0        64
  idVendor           0x05ac Apple, Inc.
  idProduct          0x8507 
  bcdDevice            4.18
  iManufacturer           1 Apple Inc.
  iProduct                2 Built-in iSight
  iSerial                 3 8J8B41B6T40U3A00
  bNumConfigurations      1
  Configuration Descriptor:
    bLength                 9
    bDescriptorType         2
    wTotalLength          642
    bNumInterfaces          3
    bConfigurationValue     1
    iConfiguration          0 
    bmAttributes         0x80
      (Bus Powered)
    MaxPower              500mA
    Interface Association:
      bLength                 8
      bDescriptorType        11
      bFirstInterface         0
      bInterfaceCount         2
      bFunctionClass         14 Video
      bFunctionSubClass       3 Video Interface Collection
      bFunctionProtocol       0 
      iFunction               2 Built-in iSight
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        0
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass        14 Video
      bInterfaceSubClass      1 Video Control
      bInterfaceProtocol      0 
      iInterface              2 Built-in iSight
      VideoControl Interface Descriptor:
        bLength                13
        bDescriptorType        36
        bDescriptorSubtype      1 (HEADER)
        bcdUVC               1.00
        wTotalLength           51
        dwClockFrequency        4.000000MHz
        bInCollection           1
        baInterfaceNr( 0)       1
      VideoControl Interface Descriptor:
        bLength                18
        bDescriptorType        36
        bDescriptorSubtype      2 (INPUT_TERMINAL)
        bTerminalID             1
        wTerminalType      0x0201 Camera Sensor
        bAssocTerminal          0
        iTerminal               0 
        wObjectiveFocalLengthMin      0
        wObjectiveFocalLengthMax      0
        wOcularFocalLength            0
        bControlSize                  3
        bmControls           0x0000000a
          Auto-Exposure Mode
          Exposure Time (Absolute)
      VideoControl Interface Descriptor:
        bLength                 9
        bDescriptorType        36
        bDescriptorSubtype      3 (OUTPUT_TERMINAL)
        bTerminalID             2
        wTerminalType      0x0101 USB Streaming
        bAssocTerminal          0
        bSourceID               3
        iTerminal               0 
      VideoControl Interface Descriptor:
        bLength                11
        bDescriptorType        36
        bDescriptorSubtype      5 (PROCESSING_UNIT)
      Warning: Descriptor too short
        bUnitID                 3
        bSourceID               1
        wMaxMultiplier          0
        bControlSize            2
        bmControls     0x0000157f
          Brightness
          Contrast
          Hue
          Saturation
          Sharpness
          Gamma
          White Balance Temperature
          Backlight Compensation
          Power Line Frequency
          White Balance Temperature, Auto
        iProcessing             0 
        bmVideoStandards     0x 0
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x81  EP 1 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0008  1x 8 bytes
        bInterval               6
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       0
      bNumEndpoints           0
      bInterfaceClass        14 Video
      bInterfaceSubClass      2 Video Streaming
      bInterfaceProtocol      0 
      iInterface              0 
      VideoStreaming Interface Descriptor:
        bLength                            14
        bDescriptorType                    36
        bDescriptorSubtype                  1 (INPUT_HEADER)
        bNumFormats                         1
        wTotalLength                      423
        bEndPointAddress                  130
        bmInfo                              0
        bTerminalLink                       2
        bStillCaptureMethod                 2
        bTriggerSupport                     0
        bTriggerUsage                       0
        bControlSize                        1
        bmaControls( 0)                    27
      VideoStreaming Interface Descriptor:
        bLength                            27
        bDescriptorType                    36
        bDescriptorSubtype                  4 (FORMAT_UNCOMPRESSED)
        bFormatIndex                        1
        bNumFrameDescriptors                5
        guidFormat                            {59555932-0000-1000-8000-00aa00389b71}
        bBitsPerPixel                      16
        bDefaultFrameIndex                  5
        bAspectRatioX                       0
        bAspectRatioY                       0
        bmInterlaceFlags                 0x00
          Interlaced stream or variable: No
          Fields per frame: 1 fields
          Field 1 first: No
          Field pattern: Field 1 only
          bCopyProtect                      0
      VideoStreaming Interface Descriptor:
        bLength                            70
        bDescriptorType                    36
        bDescriptorSubtype                  5 (FRAME_UNCOMPRESSED)
        bFrameIndex                         1
        bmCapabilities                   0x00
          Still image unsupported
        wWidth                            160
        wHeight                           120
        dwMinBitRate                 24576000
        dwMaxBitRate                147456000
        dwMaxVideoFrameBufferSize       38400
        dwDefaultFrameInterval         333333
        bFrameIntervalType                 11
        dwFrameInterval( 0)            333333
        dwFrameInterval( 1)            363636
        dwFrameInterval( 2)            400000
        dwFrameInterval( 3)            444444
        dwFrameInterval( 4)            500000
        dwFrameInterval( 5)            571428
        dwFrameInterval( 6)            666667
        dwFrameInterval( 7)            800000
        dwFrameInterval( 8)           1000000
        dwFrameInterval( 9)           1333333
        dwFrameInterval(10)           2000000
      VideoStreaming Interface Descriptor:
        bLength                            70
        bDescriptorType                    36
        bDescriptorSubtype                  5 (FRAME_UNCOMPRESSED)
        bFrameIndex                         2
        bmCapabilities                   0x00
          Still image unsupported
        wWidth                            176
        wHeight                           144
        dwMinBitRate                 24576000
        dwMaxBitRate                147456000
        dwMaxVideoFrameBufferSize       50688
        dwDefaultFrameInterval         333333
        bFrameIntervalType                 11
        dwFrameInterval( 0)            333333
        dwFrameInterval( 1)            363636
        dwFrameInterval( 2)            400000
        dwFrameInterval( 3)            444444
        dwFrameInterval( 4)            500000
        dwFrameInterval( 5)            571428
        dwFrameInterval( 6)            666667
        dwFrameInterval( 7)            800000
        dwFrameInterval( 8)           1000000
        dwFrameInterval( 9)           1333333
        dwFrameInterval(10)           2000000
      VideoStreaming Interface Descriptor:
        bLength                            70
        bDescriptorType                    36
        bDescriptorSubtype                  5 (FRAME_UNCOMPRESSED)
        bFrameIndex                         3
        bmCapabilities                   0x00
          Still image unsupported
        wWidth                            320
        wHeight                           240
        dwMinBitRate                 24576000
        dwMaxBitRate                147456000
        dwMaxVideoFrameBufferSize      153600
        dwDefaultFrameInterval         333333
        bFrameIntervalType                 11
        dwFrameInterval( 0)            333333
        dwFrameInterval( 1)            363636
        dwFrameInterval( 2)            400000
        dwFrameInterval( 3)            444444
        dwFrameInterval( 4)            500000
        dwFrameInterval( 5)            571428
        dwFrameInterval( 6)            666667
        dwFrameInterval( 7)            800000
        dwFrameInterval( 8)           1000000
        dwFrameInterval( 9)           1333333
        dwFrameInterval(10)           2000000
      VideoStreaming Interface Descriptor:
        bLength                            70
        bDescriptorType                    36
        bDescriptorSubtype                  5 (FRAME_UNCOMPRESSED)
        bFrameIndex                         4
        bmCapabilities                   0x00
          Still image unsupported
        wWidth                            352
        wHeight                           288
        dwMinBitRate                 24576000
        dwMaxBitRate                147456000
        dwMaxVideoFrameBufferSize      202752
        dwDefaultFrameInterval         333333
        bFrameIntervalType                 11
        dwFrameInterval( 0)            333333
        dwFrameInterval( 1)            363636
        dwFrameInterval( 2)            400000
        dwFrameInterval( 3)            444444
        dwFrameInterval( 4)            500000
        dwFrameInterval( 5)            571428
        dwFrameInterval( 6)            666667
        dwFrameInterval( 7)            800000
        dwFrameInterval( 8)           1000000
        dwFrameInterval( 9)           1333333
        dwFrameInterval(10)           2000000
      VideoStreaming Interface Descriptor:
        bLength                            70
        bDescriptorType                    36
        bDescriptorSubtype                  5 (FRAME_UNCOMPRESSED)
        bFrameIndex                         5
        bmCapabilities                   0x00
          Still image unsupported
        wWidth                            640
        wHeight                           480
        dwMinBitRate                 24576000
        dwMaxBitRate                147456000
        dwMaxVideoFrameBufferSize      614400
        dwDefaultFrameInterval         333333
        bFrameIntervalType                 11
        dwFrameInterval( 0)            333333
        dwFrameInterval( 1)            363636
        dwFrameInterval( 2)            400000
        dwFrameInterval( 3)            444444
        dwFrameInterval( 4)            500000
        dwFrameInterval( 5)            571428
        dwFrameInterval( 6)            666667
        dwFrameInterval( 7)            800000
        dwFrameInterval( 8)           1000000
        dwFrameInterval( 9)           1333333
        dwFrameInterval(10)           2000000
      VideoStreaming Interface Descriptor:
        bLength                            26
        bDescriptorType                    36
        bDescriptorSubtype                  3 (STILL_IMAGE_FRAME)
        bEndpointAddress                    0
        bNumImageSizePatterns               5
        wWidth( 0)                        160
        wHeight( 0)                       120
        wWidth( 1)                        176
        wHeight( 1)                       144
        wWidth( 2)                        320
        wHeight( 2)                       240
        wWidth( 3)                        352
        wHeight( 3)                       288
        wWidth( 4)                        640
        wHeight( 4)                       480
        bNumCompressionPatterns             5
      VideoStreaming Interface Descriptor:
        bLength                             6
        bDescriptorType                    36
        bDescriptorSubtype                 13 (COLORFORMAT)
        bColorPrimaries                     1 (BT.709,sRGB)
        bTransferCharacteristics            1 (BT.709)
        bMatrixCoefficients                 1 (BT.709)
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       1
      bNumEndpoints           1
      bInterfaceClass        14 Video
      bInterfaceSubClass      2 Video Streaming
      bInterfaceProtocol      0 
      iInterface              0 
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x82  EP 2 IN
        bmAttributes            5
          Transfer Type            Isochronous
          Synch Type               Asynchronous
          Usage Type               Data
        wMaxPacketSize     0x0080  1x 128 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       2
      bNumEndpoints           1
      bInterfaceClass        14 Video
      bInterfaceSubClass      2 Video Streaming
      bInterfaceProtocol      0 
      iInterface              0 
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x82  EP 2 IN
        bmAttributes            5
          Transfer Type            Isochronous
          Synch Type               Asynchronous
          Usage Type               Data
        wMaxPacketSize     0x0200  1x 512 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       3
      bNumEndpoints           1
      bInterfaceClass        14 Video
      bInterfaceSubClass      2 Video Streaming
      bInterfaceProtocol      0 
      iInterface              0 
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x82  EP 2 IN
        bmAttributes            5
          Transfer Type            Isochronous
          Synch Type               Asynchronous
          Usage Type               Data
        wMaxPacketSize     0x0400  1x 1024 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       4
      bNumEndpoints           1
      bInterfaceClass        14 Video
      bInterfaceSubClass      2 Video Streaming
      bInterfaceProtocol      0 
      iInterface              0 
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x82  EP 2 IN
        bmAttributes            5
          Transfer Type            Isochronous
          Synch Type               Asynchronous
          Usage Type               Data
        wMaxPacketSize     0x0b00  2x 768 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       5
      bNumEndpoints           1
      bInterfaceClass        14 Video
      bInterfaceSubClass      2 Video Streaming
      bInterfaceProtocol      0 
      iInterface              0 
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x82  EP 2 IN
        bmAttributes            5
          Transfer Type            Isochronous
          Synch Type               Asynchronous
          Usage Type               Data
        wMaxPacketSize     0x0c00  2x 1024 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       6
      bNumEndpoints           1
      bInterfaceClass        14 Video
      bInterfaceSubClass      2 Video Streaming
      bInterfaceProtocol      0 
      iInterface              0 
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x82  EP 2 IN
        bmAttributes            5
          Transfer Type            Isochronous
          Synch Type               Asynchronous
          Usage Type               Data
        wMaxPacketSize     0x1380  3x 896 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       7
      bNumEndpoints           1
      bInterfaceClass        14 Video
      bInterfaceSubClass      2 Video Streaming
      bInterfaceProtocol      0 
      iInterface              0 
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x82  EP 2 IN
        bmAttributes            5
          Transfer Type            Isochronous
          Synch Type               Asynchronous
          Usage Type               Data
        wMaxPacketSize     0x1400  3x 1024 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        2
      bAlternateSetting       0
      bNumEndpoints           0
      bInterfaceClass       255 Vendor Specific Class
      bInterfaceSubClass      0 
      bInterfaceProtocol      0 
      iInterface              0 
Device Qualifier (for other device speed):
  bLength                10
  bDescriptorType         6
  bcdUSB               2.00
  bDeviceClass          239 Miscellaneous Device
  bDeviceSubClass         2 ?
  bDeviceProtocol         1 Interface Association
  bMaxPacketSize0        64
  bNumConfigurations      1
Device Status:     0x0000
  (Bus Powered)

johannes

PS: I'll be on vacation until Saturday starting right now.



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: macbook webcam no longer works on .35-rc
  2010-07-05  8:15     ` Johannes Berg
@ 2010-07-05  8:23       ` Laurent Pinchart
  2010-07-12 12:36         ` Johannes Berg
  0 siblings, 1 reply; 7+ messages in thread
From: Laurent Pinchart @ 2010-07-05  8:23 UTC (permalink / raw)
  To: Johannes Berg; +Cc: linux-media, Rafael J. Wysocki, LKML, Linus Torvalds

Hi Johannes,

On Monday 05 July 2010 10:15:53 Johannes Berg wrote:
> On Mon, 2010-07-05 at 09:28 +0200, Laurent Pinchart wrote:
> > Hi Johannes,
> > 
> > On Friday 02 July 2010 09:26:15 Johannes Berg wrote:
> > > On Wed, 2010-06-30 at 23:11 +0200, Johannes Berg wrote:
> > > > I'm pretty sure this was a regression in .34, but haven't checked
> > > > right now, can bisect when I find time but wanted to inquire first
> > > > if somebody had ideas. All I get is:
> > > > 
> > > > [57372.078968] uvcvideo: Failed to query (130) UVC control 5 (unit 3)
> > > > : -32 (exp. 1).
> > 
> > Does it prevent your camera from working, or does it "just" print
> > annoying messages to the kernel log ?
> 
> It prevents it from working, and it's the only error message since the
> tool/driver/libv4l/whatever just gives up after getting -EIO.
> 
> lsusb -v is:

[snip]

Thanks.

> PS: I'll be on vacation until Saturday starting right now.

Could you please test the following patch when you will have time ?

diff --git a/drivers/media/video/uvc/uvc_ctrl.c b/drivers/media/video/uvc/uvc_ctrl.c
index 9af4d47..a350fad 100644
--- a/drivers/media/video/uvc/uvc_ctrl.c
+++ b/drivers/media/video/uvc/uvc_ctrl.c
@@ -122,8 +122,8 @@ static struct uvc_control_info uvc_ctrls[] = {
 		.selector	= UVC_PU_POWER_LINE_FREQUENCY_CONTROL,
 		.index		= 10,
 		.size		= 1,
-		.flags		= UVC_CONTROL_SET_CUR | UVC_CONTROL_GET_RANGE
-				| UVC_CONTROL_RESTORE,
+		.flags		= UVC_CONTROL_SET_CUR | UVC_CONTROL_GET_CUR
+				| UVC_CONTROL_GET_DEF | UVC_CONTROL_RESTORE,
 	},
 	{
 		.entity		= UVC_GUID_UVC_PROCESSING,


-- 
Regards,

Laurent Pinchart

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: macbook webcam no longer works on .35-rc
  2010-07-05  8:23       ` Laurent Pinchart
@ 2010-07-12 12:36         ` Johannes Berg
  2010-07-12 15:34           ` Laurent Pinchart
  0 siblings, 1 reply; 7+ messages in thread
From: Johannes Berg @ 2010-07-12 12:36 UTC (permalink / raw)
  To: Laurent Pinchart; +Cc: linux-media, Rafael J. Wysocki, LKML, Linus Torvalds

On Mon, 2010-07-05 at 10:23 +0200, Laurent Pinchart wrote:

> Could you please test the following patch when you will have time ?

That fixes it, thank you.

johannes

> diff --git a/drivers/media/video/uvc/uvc_ctrl.c b/drivers/media/video/uvc/uvc_ctrl.c
> index 9af4d47..a350fad 100644
> --- a/drivers/media/video/uvc/uvc_ctrl.c
> +++ b/drivers/media/video/uvc/uvc_ctrl.c
> @@ -122,8 +122,8 @@ static struct uvc_control_info uvc_ctrls[] = {
>  		.selector	= UVC_PU_POWER_LINE_FREQUENCY_CONTROL,
>  		.index		= 10,
>  		.size		= 1,
> -		.flags		= UVC_CONTROL_SET_CUR | UVC_CONTROL_GET_RANGE
> -				| UVC_CONTROL_RESTORE,
> +		.flags		= UVC_CONTROL_SET_CUR | UVC_CONTROL_GET_CUR
> +				| UVC_CONTROL_GET_DEF | UVC_CONTROL_RESTORE,
>  	},
>  	{
>  		.entity		= UVC_GUID_UVC_PROCESSING,
> 
> 



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: macbook webcam no longer works on .35-rc
  2010-07-12 12:36         ` Johannes Berg
@ 2010-07-12 15:34           ` Laurent Pinchart
  0 siblings, 0 replies; 7+ messages in thread
From: Laurent Pinchart @ 2010-07-12 15:34 UTC (permalink / raw)
  To: Johannes Berg; +Cc: linux-media, Rafael J. Wysocki, LKML, Linus Torvalds

Hi Johannes,

On Monday 12 July 2010 14:36:26 Johannes Berg wrote:
> On Mon, 2010-07-05 at 10:23 +0200, Laurent Pinchart wrote:
> > Could you please test the following patch when you will have time ?
> 
> That fixes it, thank you.

The fix has been applied to 
http://git.kernel.org/?p=linux/kernel/git/mchehab/linux-next.git;a=shortlog 
and should end up in the next 2.6.35-rc.

-- 
Regards,

Laurent Pinchart

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2010-07-12 15:34 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1277932269.11050.1.camel@jlt3.sipsolutions.net>
2010-07-02  7:26 ` macbook webcam no longer works on .35-rc Johannes Berg
2010-07-05  7:28   ` Laurent Pinchart
2010-07-05  8:15     ` Johannes Berg
2010-07-05  8:23       ` Laurent Pinchart
2010-07-12 12:36         ` Johannes Berg
2010-07-12 15:34           ` Laurent Pinchart
2010-07-05  7:31   ` Laurent Pinchart

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®