[05:29] *** peter___ has left [09:25] <gnurou> svarbanov: the headers with which Chromium is built actually comes from the linux-headers package which should be part of the CrOS sysroot you mentioned. I don't believe you actually want to use this code at runtime (it won't work with an upstream kernel anyway) so there is no need to worry about the kernel version here [12:21] <hverkuil> pinchartl: ping [12:22] <pinchartl> hverkuil: pong [12:23] <hverkuil> I'm working on the drm bridge ops, as per your suggestion. [12:23] <hverkuil> I'm adding two new ops: connector_attach/detach. [12:24] <hverkuil> The question I have is whether connector_attach should return void or int. There is a lot to be said for returning void. [12:25] <hverkuil> If a CEC adapter fails to register, then that doesn't mean that the whole video output should fail as well. It works perfectly fine without CEC, you just miss one feature. [12:25] <hverkuil> It also simplifies error handling :-) [12:25] <hverkuil> Just want to see if you agree with that. [12:27] <pinchartl> good question [12:27] <pinchartl> I would prefer not aborting on non-fatal issues indeed [12:28] <pinchartl> I'd say void for now. if we later have to handle fatal issues in that operation, the signature can be changed [12:28] <hverkuil> OK, makes sense. [12:28] <hverkuil> Thank you. [12:30] <pinchartl> you're welcome