linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] Media Controller "taint" fixes
@ 2019-01-15  1:24 Brad Love
  2019-01-15  1:24 ` [PATCH 1/2] cx231xx-video: Set media controller taint for analog Brad Love
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Brad Love @ 2019-01-15  1:24 UTC (permalink / raw)
  To: linux-media, mchehab; +Cc: Brad Love

Hauppauge USBLive2 was reported broken. A change in media controller
logic appears to be the culprit.

Fixes: 9d6d20e652 ("v4l2-mc: switch it to use the new approach to setup pipelines")

Without "taint" set for signal type, devices
with analog capture fail during probe:

[    5.821715] cx231xx 3-2:1.1: v4l2 driver version 0.0.3
[    5.955721] cx231xx 3-2:1.1: Registered video device video0 [v4l2]
[    5.955797] cx231xx 3-2:1.1: Registered VBI device vbi0
[    5.955802] cx231xx 3-2:1.1: video EndPoint Addr 0x84, Alternate settings: 5
[    5.955805] cx231xx 3-2:1.1: VBI EndPoint Addr 0x85, Alternate settings: 2
[    5.955807] cx231xx 3-2:1.1: sliced CC EndPoint Addr 0x86, Alternate settings: 2
[    5.955834] cx231xx 3-2:1.1: V4L2 device vbi0 deregistered
[    5.955889] cx231xx 3-2:1.1: V4L2 device video0 deregistered
[    5.959131] cx231xx: probe of 3-2:1.1 failed with error -22
[    5.959190] usbcore: registered new interface driver cx231xx


This series sets the taint as follows:
- source pads from the bridge to PAD_SIGNAL_ANALOG
- sink pads on the decoder to PAD_SIGNAL_ANALOG
- source pads on the decoder to PAD_SIGNAL_DV



Brad Love (2):
  cx231xx-video: Set media controller taint for analog outputs
  cx25840-core: Set media controller taint for pads

 drivers/media/i2c/cx25840/cx25840-core.c  | 6 ++++++
 drivers/media/usb/cx231xx/cx231xx-video.c | 1 +
 2 files changed, 7 insertions(+)

-- 
2.7.4


^ permalink raw reply	[flat|nested] 6+ messages in thread
* [PATCH 1/2] cx231xx-video: Set media controller taint for analog
  2019-01-15  1:24 [PATCH 0/2] Media Controller "taint" fixes Brad Love
@ 2019-01-15  1:24 ` Brad Love
  2019-01-15  1:24 ` [PATCH 2/2] cx25840-core: Set media controller taint for pads Brad Love
  2019-02-18 16:08 ` [PATCH 0/2] Media Controller "taint" fixes Hans Verkuil
  2 siblings, 0 replies; 6+ messages in thread
From: Brad Love @ 2019-01-15  1:24 UTC (permalink / raw)
  To: linux-media, mchehab; +Cc: Brad Love

Fixes: 9d6d20e652 ("v4l2-mc: switch it to use the new approach to setup pipelines")

Without "taint" set for signal type, devices
with analog capture fail during probe:

[    5.821715] cx231xx 3-2:1.1: v4l2 driver version 0.0.3
[    5.955721] cx231xx 3-2:1.1: Registered video device video0 [v4l2]
[    5.955797] cx231xx 3-2:1.1: Registered VBI device vbi0
[    5.955802] cx231xx 3-2:1.1: video EndPoint Addr 0x84, Alternate settings: 5
[    5.955805] cx231xx 3-2:1.1: VBI EndPoint Addr 0x85, Alternate settings: 2
[    5.955807] cx231xx 3-2:1.1: sliced CC EndPoint Addr 0x86, Alternate settings: 2
[    5.955834] cx231xx 3-2:1.1: V4L2 device vbi0 deregistered
[    5.955889] cx231xx 3-2:1.1: V4L2 device video0 deregistered
[    5.959131] cx231xx: probe of 3-2:1.1 failed with error -22
[    5.959190] usbcore: registered new interface driver cx231xx


This sets the taint for the source pad on the bridge to PAD_SIGNAL_ANALOG


Signed-off-by: Brad Love <brad@nextdimension.cc>
---
 drivers/media/usb/cx231xx/cx231xx-video.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/media/usb/cx231xx/cx231xx-video.c b/drivers/media/usb/cx231xx/cx231xx-video.c
index d5e51a5..12f2c00 100644
--- a/drivers/media/usb/cx231xx/cx231xx-video.c
+++ b/drivers/media/usb/cx231xx/cx231xx-video.c
@@ -1120,6 +1120,7 @@ void cx231xx_v4l2_create_entities(struct cx231xx *dev)
 		ent->name = iname[INPUT(i)->type];
 		ent->flags = MEDIA_ENT_FL_CONNECTOR;
 		dev->input_pad[i].flags = MEDIA_PAD_FL_SOURCE;
+		dev->input_pad[i].sig_type = PAD_SIGNAL_ANALOG;
 
 		switch (INPUT(i)->type) {
 		case CX231XX_VMUX_COMPOSITE1:
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH 2/2] cx25840-core: Set media controller taint for pads
  2019-01-15  1:24 [PATCH 0/2] Media Controller "taint" fixes Brad Love
  2019-01-15  1:24 ` [PATCH 1/2] cx231xx-video: Set media controller taint for analog Brad Love
@ 2019-01-15  1:24 ` Brad Love
  2019-04-05 11:24   ` Sean Young
  2019-02-18 16:08 ` [PATCH 0/2] Media Controller "taint" fixes Hans Verkuil
  2 siblings, 1 reply; 6+ messages in thread
From: Brad Love @ 2019-01-15  1:24 UTC (permalink / raw)
  To: linux-media, mchehab; +Cc: Brad Love

Fixes: 9d6d20e652 ("v4l2-mc: switch it to use the new approach to setup pipelines")

Without "taint" set for signal type, devices
with analog capture fail during probe:

[    5.821715] cx231xx 3-2:1.1: v4l2 driver version 0.0.3
[    5.955721] cx231xx 3-2:1.1: Registered video device video0 [v4l2]
[    5.955797] cx231xx 3-2:1.1: Registered VBI device vbi0
[    5.955802] cx231xx 3-2:1.1: video EndPoint Addr 0x84, Alternate settings: 5
[    5.955805] cx231xx 3-2:1.1: VBI EndPoint Addr 0x85, Alternate settings: 2
[    5.955807] cx231xx 3-2:1.1: sliced CC EndPoint Addr 0x86, Alternate settings: 2
[    5.955834] cx231xx 3-2:1.1: V4L2 device vbi0 deregistered
[    5.955889] cx231xx 3-2:1.1: V4L2 device video0 deregistered
[    5.959131] cx231xx: probe of 3-2:1.1 failed with error -22
[    5.959190] usbcore: registered new interface driver cx231xx


This sets the taint as follows:
- sink pads to PAD_SIGNAL_ANALOG
- source pads to PAD_SIGNAL_DV


Signed-off-by: Brad Love <brad@nextdimension.cc>
---
 drivers/media/i2c/cx25840/cx25840-core.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/media/i2c/cx25840/cx25840-core.c b/drivers/media/i2c/cx25840/cx25840-core.c
index b23d8e4..0c94be5 100644
--- a/drivers/media/i2c/cx25840/cx25840-core.c
+++ b/drivers/media/i2c/cx25840/cx25840-core.c
@@ -5225,8 +5225,12 @@ static int cx25840_probe(struct i2c_client *client,
 	 * those extra inputs. So, let's add it only when needed.
 	 */
 	state->pads[CX25840_PAD_INPUT].flags = MEDIA_PAD_FL_SINK;
+	state->pads[CX25840_PAD_INPUT].sig_type = PAD_SIGNAL_ANALOG;
 	state->pads[CX25840_PAD_VID_OUT].flags = MEDIA_PAD_FL_SOURCE;
+	state->pads[CX25840_PAD_VID_OUT].sig_type = PAD_SIGNAL_DV;
 	state->pads[CX25840_PAD_VBI_OUT].flags = MEDIA_PAD_FL_SOURCE;
+	state->pads[CX25840_PAD_VBI_OUT].sig_type = PAD_SIGNAL_DV;
+
 	sd->entity.function = MEDIA_ENT_F_ATV_DECODER;
 
 	ret = media_entity_pads_init(&sd->entity, ARRAY_SIZE(state->pads),
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] Media Controller "taint" fixes
  2019-01-15  1:24 [PATCH 0/2] Media Controller "taint" fixes Brad Love
  2019-01-15  1:24 ` [PATCH 1/2] cx231xx-video: Set media controller taint for analog Brad Love
  2019-01-15  1:24 ` [PATCH 2/2] cx25840-core: Set media controller taint for pads Brad Love
@ 2019-02-18 16:08 ` Hans Verkuil
  2019-02-18 17:58   ` Brad Love
  2 siblings, 1 reply; 6+ messages in thread
From: Hans Verkuil @ 2019-02-18 16:08 UTC (permalink / raw)
  To: Brad Love, linux-media, mchehab

Hi Brad,

On 1/15/19 2:24 AM, Brad Love wrote:
> Hauppauge USBLive2 was reported broken. A change in media controller
> logic appears to be the culprit.
> 
> Fixes: 9d6d20e652 ("v4l2-mc: switch it to use the new approach to setup pipelines")
> 
> Without "taint" set for signal type, devices
> with analog capture fail during probe:
> 
> [    5.821715] cx231xx 3-2:1.1: v4l2 driver version 0.0.3
> [    5.955721] cx231xx 3-2:1.1: Registered video device video0 [v4l2]
> [    5.955797] cx231xx 3-2:1.1: Registered VBI device vbi0
> [    5.955802] cx231xx 3-2:1.1: video EndPoint Addr 0x84, Alternate settings: 5
> [    5.955805] cx231xx 3-2:1.1: VBI EndPoint Addr 0x85, Alternate settings: 2
> [    5.955807] cx231xx 3-2:1.1: sliced CC EndPoint Addr 0x86, Alternate settings: 2
> [    5.955834] cx231xx 3-2:1.1: V4L2 device vbi0 deregistered
> [    5.955889] cx231xx 3-2:1.1: V4L2 device video0 deregistered
> [    5.959131] cx231xx: probe of 3-2:1.1 failed with error -22
> [    5.959190] usbcore: registered new interface driver cx231xx
> 
> 
> This series sets the taint as follows:
> - source pads from the bridge to PAD_SIGNAL_ANALOG
> - sink pads on the decoder to PAD_SIGNAL_ANALOG
> - source pads on the decoder to PAD_SIGNAL_DV

Mauro asked me to look at this, but it is still failing for me:

[ 2046.476092] usb usb3: New USB device found, idVendor=1d6b, idProduct=0002, bcdDevice= 5.00
[ 2046.476098] usb usb3: New USB device strings: Mfr=3, Product=2, SerialNumber=1
[ 2046.476102] usb usb3: Product: xHCI Host Controller
[ 2046.476107] usb usb3: Manufacturer: Linux 5.0.0-rc1-zen xhci-hcd
[ 2046.476111] usb usb3: SerialNumber: 0000:39:00.0
[ 2046.476677] hub 3-0:1.0: USB hub found
[ 2046.476898] hub 3-0:1.0: 2 ports detected
[ 2046.478160] xhci_hcd 0000:39:00.0: xHCI Host Controller
[ 2046.478677] xhci_hcd 0000:39:00.0: new USB bus registered, assigned bus number 4
[ 2046.478690] xhci_hcd 0000:39:00.0: Host supports USB 3.1 Enhanced SuperSpeed
[ 2046.478838] usb usb4: New USB device found, idVendor=1d6b, idProduct=0003, bcdDevice= 5.00
[ 2046.478843] usb usb4: New USB device strings: Mfr=3, Product=2, SerialNumber=1
[ 2046.478847] usb usb4: Product: xHCI Host Controller
[ 2046.478851] usb usb4: Manufacturer: Linux 5.0.0-rc1-zen xhci-hcd
[ 2046.478855] usb usb4: SerialNumber: 0000:39:00.0
[ 2046.479180] hub 4-0:1.0: USB hub found
[ 2046.479206] hub 4-0:1.0: 2 ports detected
[ 2046.802013] usb 3-2: new high-speed USB device number 2 using xhci_hcd
[ 2046.934170] usb 3-2: New USB device found, idVendor=2040, idProduct=c200, bcdDevice=40.01
[ 2046.934188] usb 3-2: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[ 2046.934197] usb 3-2: Product: Hauppauge Device
[ 2046.934206] usb 3-2: Manufacturer: Hauppauge
[ 2046.934214] usb 3-2: SerialNumber: 0013567005
[ 2046.942224] cx231xx 3-2:1.1: New device Hauppauge Hauppauge Device @ 480 Mbps (2040:c200) with 6 interfaces
[ 2046.942626] cx231xx 3-2:1.1: can't change interface 3 alt no. to 3: Max. Pkt size = 0
[ 2046.942631] cx231xx 3-2:1.1: Identified as Hauppauge USB Live 2 (card=9)
[ 2046.944251] i2c i2c-10: Added multiplexed i2c bus 12
[ 2046.944382] i2c i2c-10: Added multiplexed i2c bus 13
[ 2047.054566] cx25840 9-0044: cx23102 A/V decoder found @ 0x88 (cx231xx #0-0)
[ 2049.997665] cx25840 9-0044: loaded v4l-cx231xx-avcore-01.fw firmware (16382 bytes)
[ 2050.091897] cx231xx 3-2:1.1: v4l2 driver version 0.0.3
[ 2050.307929] cx231xx 3-2:1.1: Registered video device video0 [v4l2]
[ 2050.308349] cx231xx 3-2:1.1: Registered VBI device vbi0
[ 2050.314083] cx231xx 3-2:1.1: audio EndPoint Addr 0x83, Alternate settings: 3
[ 2050.314131] cx231xx 3-2:1.1: video EndPoint Addr 0x84, Alternate settings: 5
[ 2050.314135] cx231xx 3-2:1.1: VBI EndPoint Addr 0x85, Alternate settings: 2
[ 2050.314138] cx231xx 3-2:1.1: sliced CC EndPoint Addr 0x86, Alternate settings: 2
[ 2050.314148] usb 3-2: couldn't get decoder output pad for V4L I/O
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

[ 2050.314151] cx231xx 3-2:1.1: V4L2 device vbi0 deregistered
[ 2050.314449] cx231xx 3-2:1.1: V4L2 device video0 deregistered
[ 2050.316448] cx231xx: probe of 3-2:1.1 failed with error -22

Can you take another look?

See also:

https://lore.kernel.org/linux-media/1550027010.2460608.1656864112.3A25F771@webmail.messagingengine.com/

And also:

https://patchwork.kernel.org/patch/10763655/

I'm really confused what the status is and what has and hasn't been tested/reviewed.

Regards,

	Hans

> 
> 
> 
> Brad Love (2):
>   cx231xx-video: Set media controller taint for analog outputs
>   cx25840-core: Set media controller taint for pads
> 
>  drivers/media/i2c/cx25840/cx25840-core.c  | 6 ++++++
>  drivers/media/usb/cx231xx/cx231xx-video.c | 1 +
>  2 files changed, 7 insertions(+)
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] Media Controller "taint" fixes
  2019-02-18 16:08 ` [PATCH 0/2] Media Controller "taint" fixes Hans Verkuil
@ 2019-02-18 17:58   ` Brad Love
  0 siblings, 0 replies; 6+ messages in thread
From: Brad Love @ 2019-02-18 17:58 UTC (permalink / raw)
  To: Hans Verkuil, Brad Love, linux-media, mchehab

Hi Hans,


On 18/02/2019 10.08, Hans Verkuil wrote:
> Hi Brad,
>
> On 1/15/19 2:24 AM, Brad Love wrote:
>> Hauppauge USBLive2 was reported broken. A change in media controller
>> logic appears to be the culprit.
>>
>> Fixes: 9d6d20e652 ("v4l2-mc: switch it to use the new approach to setup pipelines")
>>
>> Without "taint" set for signal type, devices
>> with analog capture fail during probe:
>>
>> [    5.821715] cx231xx 3-2:1.1: v4l2 driver version 0.0.3
>> [    5.955721] cx231xx 3-2:1.1: Registered video device video0 [v4l2]
>> [    5.955797] cx231xx 3-2:1.1: Registered VBI device vbi0
>> [    5.955802] cx231xx 3-2:1.1: video EndPoint Addr 0x84, Alternate settings: 5
>> [    5.955805] cx231xx 3-2:1.1: VBI EndPoint Addr 0x85, Alternate settings: 2
>> [    5.955807] cx231xx 3-2:1.1: sliced CC EndPoint Addr 0x86, Alternate settings: 2
>> [    5.955834] cx231xx 3-2:1.1: V4L2 device vbi0 deregistered
>> [    5.955889] cx231xx 3-2:1.1: V4L2 device video0 deregistered
>> [    5.959131] cx231xx: probe of 3-2:1.1 failed with error -22
>> [    5.959190] usbcore: registered new interface driver cx231xx
>>
>>
>> This series sets the taint as follows:
>> - source pads from the bridge to PAD_SIGNAL_ANALOG
>> - sink pads on the decoder to PAD_SIGNAL_ANALOG
>> - source pads on the decoder to PAD_SIGNAL_DV
> Mauro asked me to look at this, but it is still failing for me:
>
> [ 2046.476092] usb usb3: New USB device found, idVendor=1d6b, idProduct=0002, bcdDevice= 5.00
> [ 2046.476098] usb usb3: New USB device strings: Mfr=3, Product=2, SerialNumber=1
> [ 2046.476102] usb usb3: Product: xHCI Host Controller
> [ 2046.476107] usb usb3: Manufacturer: Linux 5.0.0-rc1-zen xhci-hcd
> [ 2046.476111] usb usb3: SerialNumber: 0000:39:00.0
> [ 2046.476677] hub 3-0:1.0: USB hub found
> [ 2046.476898] hub 3-0:1.0: 2 ports detected
> [ 2046.478160] xhci_hcd 0000:39:00.0: xHCI Host Controller
> [ 2046.478677] xhci_hcd 0000:39:00.0: new USB bus registered, assigned bus number 4
> [ 2046.478690] xhci_hcd 0000:39:00.0: Host supports USB 3.1 Enhanced SuperSpeed
> [ 2046.478838] usb usb4: New USB device found, idVendor=1d6b, idProduct=0003, bcdDevice= 5.00
> [ 2046.478843] usb usb4: New USB device strings: Mfr=3, Product=2, SerialNumber=1
> [ 2046.478847] usb usb4: Product: xHCI Host Controller
> [ 2046.478851] usb usb4: Manufacturer: Linux 5.0.0-rc1-zen xhci-hcd
> [ 2046.478855] usb usb4: SerialNumber: 0000:39:00.0
> [ 2046.479180] hub 4-0:1.0: USB hub found
> [ 2046.479206] hub 4-0:1.0: 2 ports detected
> [ 2046.802013] usb 3-2: new high-speed USB device number 2 using xhci_hcd
> [ 2046.934170] usb 3-2: New USB device found, idVendor=2040, idProduct=c200, bcdDevice=40.01
> [ 2046.934188] usb 3-2: New USB device strings: Mfr=1, Product=2, SerialNumber=3
> [ 2046.934197] usb 3-2: Product: Hauppauge Device
> [ 2046.934206] usb 3-2: Manufacturer: Hauppauge
> [ 2046.934214] usb 3-2: SerialNumber: 0013567005
> [ 2046.942224] cx231xx 3-2:1.1: New device Hauppauge Hauppauge Device @ 480 Mbps (2040:c200) with 6 interfaces
> [ 2046.942626] cx231xx 3-2:1.1: can't change interface 3 alt no. to 3: Max. Pkt size = 0
> [ 2046.942631] cx231xx 3-2:1.1: Identified as Hauppauge USB Live 2 (card=9)
> [ 2046.944251] i2c i2c-10: Added multiplexed i2c bus 12
> [ 2046.944382] i2c i2c-10: Added multiplexed i2c bus 13
> [ 2047.054566] cx25840 9-0044: cx23102 A/V decoder found @ 0x88 (cx231xx #0-0)
> [ 2049.997665] cx25840 9-0044: loaded v4l-cx231xx-avcore-01.fw firmware (16382 bytes)
> [ 2050.091897] cx231xx 3-2:1.1: v4l2 driver version 0.0.3
> [ 2050.307929] cx231xx 3-2:1.1: Registered video device video0 [v4l2]
> [ 2050.308349] cx231xx 3-2:1.1: Registered VBI device vbi0
> [ 2050.314083] cx231xx 3-2:1.1: audio EndPoint Addr 0x83, Alternate settings: 3
> [ 2050.314131] cx231xx 3-2:1.1: video EndPoint Addr 0x84, Alternate settings: 5
> [ 2050.314135] cx231xx 3-2:1.1: VBI EndPoint Addr 0x85, Alternate settings: 2
> [ 2050.314138] cx231xx 3-2:1.1: sliced CC EndPoint Addr 0x86, Alternate settings: 2
> [ 2050.314148] usb 3-2: couldn't get decoder output pad for V4L I/O
> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>
> [ 2050.314151] cx231xx 3-2:1.1: V4L2 device vbi0 deregistered
> [ 2050.314449] cx231xx 3-2:1.1: V4L2 device video0 deregistered
> [ 2050.316448] cx231xx: probe of 3-2:1.1 failed with error -22
>
> Can you take another look?
>
> See also:
>
> https://lore.kernel.org/linux-media/1550027010.2460608.1656864112.3A25F771@webmail.messagingengine.com/
>
> And also:
>
> https://patchwork.kernel.org/patch/10763655/
>
> I'm really confused what the status is and what has and hasn't been tested/reviewed.
>
> Regards,
>
> 	Hans


I fetched tip and rebased with both patches from this series applied.
When A usblive2 is connected everything works.

Note, patch 1/2 just explicitly sets sig_type, but is not required, as
found out later during testing. Patch 2/2 by itself fixes this issue
entirely. The message from 1/2 should have been in 2/2, since that is
the fix. Patch 1/2 does not have any adverse affects in operation.



[   10.432033] usb 3-2: new high-speed USB device number 3 using xhci_hcd
[   10.582784] usb 3-2: New USB device found, idVendor=2040,
idProduct=c200, bcdDevice=40.01
[   10.582787] usb 3-2: New USB device strings: Mfr=1, Product=2,
SerialNumber=3
[   10.582788] usb 3-2: Product: Hauppauge Device
[   10.582789] usb 3-2: Manufacturer: Hauppauge
[   10.582790] usb 3-2: SerialNumber: 0013871590
[   10.710747] media: Linux media interface: v0.10
[   10.715890] videodev: Linux video capture interface: v2.00
[   10.731810] cx231xx 3-2:1.1: New device Hauppauge Hauppauge Device @
480 Mbps (2040:c200) with 6 interfaces
[   10.731902] cx231xx 3-2:1.1: can't change interface 3 alt no. to 3:
Max. Pkt size = 0
[   10.731904] cx231xx 3-2:1.1: Identified as Hauppauge USB Live 2 (card=9)
[   10.733128] i2c i2c-17: Added multiplexed i2c bus 19
[   10.733221] i2c i2c-17: Added multiplexed i2c bus 20
[   10.870542] cx25840 16-0044: cx23102 A/V decoder found @ 0x88
(cx231xx #0-0)
[   12.995801] cx25840 16-0044: loaded v4l-cx231xx-avcore-01.fw firmware
(16382 bytes)
[   13.027555] cx231xx 3-2:1.1: v4l2 driver version 0.0.3
[   13.125674] cx231xx 3-2:1.1: Registered video device video0 [v4l2]
[   13.125736] cx231xx 3-2:1.1: Registered VBI device vbi0
[   13.125741] cx231xx 3-2:1.1: video EndPoint Addr 0x84, Alternate
settings: 5
[   13.125744] cx231xx 3-2:1.1: VBI EndPoint Addr 0x85, Alternate
settings: 2
[   13.125747] cx231xx 3-2:1.1: sliced CC EndPoint Addr 0x86, Alternate
settings: 2
[   13.125920] usbcore: registered new interface driver cx231xx
[   13.136241] cx231xx 3-2:1.1: audio EndPoint Addr 0x83, Alternate
settings: 3
[   13.136245] cx231xx 3-2:1.1: Cx231xx Audio Extension initialized




>>
>>
>> Brad Love (2):
>>   cx231xx-video: Set media controller taint for analog outputs
>>   cx25840-core: Set media controller taint for pads
>>
>>  drivers/media/i2c/cx25840/cx25840-core.c  | 6 ++++++
>>  drivers/media/usb/cx231xx/cx231xx-video.c | 1 +
>>  2 files changed, 7 insertions(+)
>>


^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] cx25840-core: Set media controller taint for pads
  2019-01-15  1:24 ` [PATCH 2/2] cx25840-core: Set media controller taint for pads Brad Love
@ 2019-04-05 11:24   ` Sean Young
  0 siblings, 0 replies; 6+ messages in thread
From: Sean Young @ 2019-04-05 11:24 UTC (permalink / raw)
  To: Brad Love; +Cc: linux-media, mchehab

On Mon, Jan 14, 2019 at 07:24:08PM -0600, Brad Love wrote:
> Fixes: 9d6d20e652 ("v4l2-mc: switch it to use the new approach to setup pipelines")
> 
> Without "taint" set for signal type, devices
> with analog capture fail during probe:
> 
> [    5.821715] cx231xx 3-2:1.1: v4l2 driver version 0.0.3
> [    5.955721] cx231xx 3-2:1.1: Registered video device video0 [v4l2]
> [    5.955797] cx231xx 3-2:1.1: Registered VBI device vbi0
> [    5.955802] cx231xx 3-2:1.1: video EndPoint Addr 0x84, Alternate settings: 5
> [    5.955805] cx231xx 3-2:1.1: VBI EndPoint Addr 0x85, Alternate settings: 2
> [    5.955807] cx231xx 3-2:1.1: sliced CC EndPoint Addr 0x86, Alternate settings: 2
> [    5.955834] cx231xx 3-2:1.1: V4L2 device vbi0 deregistered
> [    5.955889] cx231xx 3-2:1.1: V4L2 device video0 deregistered
> [    5.959131] cx231xx: probe of 3-2:1.1 failed with error -22
> [    5.959190] usbcore: registered new interface driver cx231xx
> 
> 
> This sets the taint as follows:
> - sink pads to PAD_SIGNAL_ANALOG
> - source pads to PAD_SIGNAL_DV

This patch no longer applies, looks like similar fix was done in
commit 46c039d06b6ecabb94bd16c3a999b28dc83b79ce.

Sean

> 
> 
> Signed-off-by: Brad Love <brad@nextdimension.cc>
> ---
>  drivers/media/i2c/cx25840/cx25840-core.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/media/i2c/cx25840/cx25840-core.c b/drivers/media/i2c/cx25840/cx25840-core.c
> index b23d8e4..0c94be5 100644
> --- a/drivers/media/i2c/cx25840/cx25840-core.c
> +++ b/drivers/media/i2c/cx25840/cx25840-core.c
> @@ -5225,8 +5225,12 @@ static int cx25840_probe(struct i2c_client *client,
>  	 * those extra inputs. So, let's add it only when needed.
>  	 */
>  	state->pads[CX25840_PAD_INPUT].flags = MEDIA_PAD_FL_SINK;
> +	state->pads[CX25840_PAD_INPUT].sig_type = PAD_SIGNAL_ANALOG;
>  	state->pads[CX25840_PAD_VID_OUT].flags = MEDIA_PAD_FL_SOURCE;
> +	state->pads[CX25840_PAD_VID_OUT].sig_type = PAD_SIGNAL_DV;
>  	state->pads[CX25840_PAD_VBI_OUT].flags = MEDIA_PAD_FL_SOURCE;
> +	state->pads[CX25840_PAD_VBI_OUT].sig_type = PAD_SIGNAL_DV;
> +
>  	sd->entity.function = MEDIA_ENT_F_ATV_DECODER;
>  
>  	ret = media_entity_pads_init(&sd->entity, ARRAY_SIZE(state->pads),
> -- 
> 2.7.4

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-04-05 11:24 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-15  1:24 [PATCH 0/2] Media Controller "taint" fixes Brad Love
2019-01-15  1:24 ` [PATCH 1/2] cx231xx-video: Set media controller taint for analog Brad Love
2019-01-15  1:24 ` [PATCH 2/2] cx25840-core: Set media controller taint for pads Brad Love
2019-04-05 11:24   ` Sean Young
2019-02-18 16:08 ` [PATCH 0/2] Media Controller "taint" fixes Hans Verkuil
2019-02-18 17:58   ` Brad Love

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).