21:35 Lyude: oh hey - looks like I wasn't wrong about type C hotplugging after all
21:36 imirkin: that it's a pain?
21:36 Lyude: currently reviewing some patches for out of band HPD events from type C ports, apparently the HPD pins are no longer guaranteed to be on the GPU
21:36 Lyude: imirkin: no that there were extra typec events we weren't handling :P
21:36 imirkin: that's just an implementation detail of said pain...
21:36 Lyude: i suppose
21:37 karolherbst: Lyude: wait what?
21:38 Lyude: karolherbst: yeah - I'm reviewing a bunch of patches from hans for supporting this universally across drivers
21:38 karolherbst: uhhh.. annoying
21:38 Lyude: i mean we're about to have it supported soon :), I'm just glad for that
21:38 karolherbst: cool
21:51 Lyude: looks like we'll need to hook up a callback or two for nouveau but that's it
21:52 Lyude: or, oh. no, it needs more then that :\
21:52 Lyude: at least we know these are a thing that exist now though
21:53 Lyude: this will likely need some kind of info from nvidia though, skeggsb_ if you want to come up with something for that person from lenovo to look into ^ :)
22:48 karolherbst: so I'd really like to get https://gitlab.freedesktop.org/mesa/drm/-/merge_requests/188 in, should I just merge or does anybody wants to review it before I'll just do that?
22:49 karolherbst: I wouldn't mind that much if that wouldn't add a new API call
22:49 imirkin: mmm
22:50 imirkin: karolherbst: i don't think this matters, but
22:50 imirkin: nouveau_pushbuf_new starts out with something similar
22:50 imirkin: where it also sets req.nr_push = 0;
22:51 imirkin: this isn't strictly required, since the thing should get zero-initialized
22:51 imirkin: but i'd encourage you to do the same thing there too
22:51 imirkin: and add a similar comment that this is a no-op submission
22:51 karolherbst: mhh yeah, makes sense
22:52 karolherbst: done
23:12 imirkin: karolherbst: also i dunno about sticking "bool" into public API's
23:13 karolherbst: yeah.... not sure either, but since C99 those are actually well defined.. soo
23:13 imirkin: not all is C99 in the world
23:13 imirkin: that's my point
23:13 karolherbst: not even the nouveau ddx?
23:14 imirkin: i assume it is
23:14 imirkin: but drm_nouveau is meant to be more general
23:14 karolherbst: yeah...
23:15 karolherbst: amdgpu seems to use bool
23:15 karolherbst: inside amdgpu/amdgpu.h
23:15 imirkin: ok
23:15 imirkin: fine i guess
23:16 karolherbst: I mean.. if something still defaults to c89 they kind of deserve the pain or so