| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| *** | hackvana has left | [02:38] |
| ...................................................................... (idle for 5h48mn) | ||
| hverkuil | pinchartl: can you set the vsp1 series in patchwork to 'accepted'? That would clean up the patchwork list. | [08:26] |
| ................................... (idle for 2h51mn) | ||
| pinchartl | hverkuil: done
mchehab: https://bugzilla.kernel.org/show_bug.cgi?id=111291 how would you like that to be fixed ? | [11:17] |
| mchehab | pinchartl: as we've discussed before: adding proper entity types
an "unknonwn" entity is not useful, as userspace apps cannot handle them in a generic way | [11:27] |
| pinchartl | mchehab: yes, but the thing is here, all we know about the UVC extension units is that they do "something"
there's no information provided by the device regarding what they do | [11:37] |
| mchehab | then, what's the sense of exposing such extension as an entity? | [11:38] |
| pinchartl | the can be controlled by userspace
they expose controls the kernel doesn't know what those controls do the hardware reports a list of controls and how to access them but that's it, it doesn't tell what those controls are for | [11:39] |
| mchehab | and how userspace knows? | [11:41] |
| pinchartl | the extension units have a GUID. userspace needs to know about the device
it requires device-specific code in userspace that's how the UVC protocol was designed | [11:42] |
| mchehab | well, we can expose those crap as MEDIA_ENT_F_UVC_EXTENSION
and document that userspace should use GUID + USB ID in order to add device-specific handlers for this specific entity function | [11:43] |
| pinchartl | fine with me
I'll do that | [11:49] |
| mchehab | ok
in the case of the UVC processing entity... do we have any clue about what they do? [ 2.533004] uvcvideo 3-10:1.0: Entity type for entity Processing 2 was not initialized! mchehab is assuming that this one should be MEDIA_ENT_F_CAM_SENSOR: [ 2.533006] uvcvideo 3-10:1.0: Entity type for entity Camera 1 was not initialized! | [11:49] |
| pinchartl | various types of image-processing. brightness, contrast, hue, saturation, sharpness, gamma, digital zoom, white balance (manual/auto), backlight compensation, gain, power line frequency
the camera terminal handles exposure (auto/manual), focus (auto/manual), iris, zoom, pan, tilt, roll not all those controls need to be implemented, it's device-specific | [11:52] |
| mchehab | ok. So, IMO, they should have a different function than the UVC_EXTENSION, as we have a better idea about what they do than what an extension does | [11:56] |
| pinchartl | yes, but it's still quite broad | [11:59] |
| mchehab | yes... crap spec ;)
mchehab wanders how other OSes handle those device-specific UVC GUIDs as UVC cameras are supposed to work as-is without needing to install vendor-specific drivers | [11:59] |
| pinchartl | the processing unit and camera terminal are well-defined, so they're mapped to known V4L2 controls
the only latitude that the device vendors have is what deciding which controls they want to implement for extension units, configuring them isn't mandatory, but can lead to better operation they're sometimes used for firmware update too that requires device-specific userspace software | [12:01] |
| .... (idle for 19mn) | ||
| mchehab | ok. | [12:21] |
| ........................... (idle for 2h10mn) | ||
| hverkuil | OK, I've gone through most of my pending patches. Monday I want to take a look at the r-car rewrite from Niklas/Ulrich.
If there is anything else I need to look at, let me know. | [14:31] |
| ......... (idle for 40mn) | ||
| ezequielg | hverkuil: hey, have you seen the tw686x improvements set? hopefully it should address everyone's use cases. | [15:12] |
| hverkuil | ezequielg: I've seen it but not reviewed them yet. | [15:13] |
| ezequielg | hverkuil: and btw, how did your tests on tw686x went? | [15:13] |
| hverkuil | Didn't have time, sorry.
It's on my todo list. | [15:13] |
| ezequielg | hverkuil: right. no problem.
i tried to purchase one of those boards, but could not find one that shipped here :( anyway, i know you are busy, so take your time, no worries on my side :) | [15:13] |
| *** | slowfool has quit IRC (*.net *.split) | [15:25] |
| .......... (idle for 47mn) | ||
| larsc | hverkuil: that vpfe patch you reverted looks really fun | [16:12] |
| ...................................... (idle for 3h7mn) | ||
| *** | awalls1 has left | [19:19] |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |