[03:36] <RAOF> Bah.
[03:36] <RAOF> mir_render_surface_get_buffer_stream is broken.
[03:54] <duflu> RAOF: Is there such thing as a hardware default gamma ramp, or would you just need to save and use the current ramps at startup?
[03:55] <RAOF> duflu: There's whatever's in the LUT after POST.
[03:55] <duflu> Yeah
[03:55] <RAOF> Which is going to be a sensible default.
[03:56] <duflu> RAOF: Unfortunately only the system compositor (first invocation) can get that. And a client can never get it
[03:56] <RAOF> Why won't a client get that?
[03:56] <RAOF> We read out the existing gamma curve at startup.
[03:56] <duflu> RAOF: I mean the client can't assume that someone else hasn't changed it already
[03:57] <RAOF> That's true. Why would a client care?
[03:57] <duflu> RAOF: To approximate 'xrandr --brightness' which is predictably just an ugly approximation
[03:58] <duflu> To keep it accurate you need to know the default full brightness ramp
[03:59] <RAOF> But we're going to provide a brightness knob anyway?
[03:59] <duflu> RAOF: Backlight control doesn't work on desktops, projectors, OLED... :)
[03:59] <RAOF> Incorrect!
[03:59] <duflu> There is no backlight device
[03:59] <duflu> That works
[04:00] <duflu> Blame intel
[04:00] <RAOF> Depends on the monitor.
[04:00] <duflu> Most non-laptop systems actually don't have a user controllable backlight device
[04:00] <duflu> And even some laptops don't (OLED)
[04:01] <RAOF> Both my monitors support brightness control.
[04:01] <duflu> RAOF: /sys/class/backlight works?
[04:01] <RAOF> X doesn't, but that's fine.
[04:01] <RAOF> duflu: No, but you can bang the I2C bus.
[04:02] <duflu> Yeah, I'm thinking about a portable solution
[04:02] <duflu> RAOF: Suggested tools for i2c controls?
[04:02] <RAOF> Indeed. A portable solution is us providing a brightness knob.
[04:03] <RAOF> ddccontrol is a reasonable start.
[04:04] <RAOF> Oh, hello! ddcci-dkms provides /sys/class/backlight.
[04:04] <RAOF> I wonder why that's not a part of drm?
[04:06] <RAOF> Anyway - the portable solution is for us to provide a brightness knob, backed by an actual backlight device if possible or by gamma ramps if not.
[04:08]  * RAOF likes that captureless lambdas desugar to C function pointers.
[04:09] <duflu> OK, this works... sudo ddccontrol -r 0x10 -w 100 dev:/dev/i2c-7
[04:09] <duflu> Not obvious or user friendly
[04:10] <RAOF> Oh, totally.
[04:11] <RAOF> Which, again, is why we provide a brightness knob on a MirOutput :)
[04:13] <duflu> Nice to have a solution that doesn't sacrifice colour precision
[04:13] <RAOF> Yes.
[04:14] <duflu> Oh, we can do fancy things too. Like ask the monitor what its orientation is
[04:14] <duflu> We should do that
[04:14] <RAOF> Plausibly? I haven't really fiddled with too much of that :)
[04:15] <duflu> My desktop monitor can do a few useful things...
[04:15] <duflu> Controls (valid/current/max) [Description - Value name]:
[04:15] <duflu> Control 0x02: +/1/65535 C [???]
[04:15] <duflu> Control 0x04: +/0/1 C [Restore Factory Defaults]
[04:15] <duflu> Control 0x05: +/0/1 C [Restore Brightness and Contrast]
[04:15] <duflu> Control 0x06: +/0/1   [???]
[04:15] <duflu> Control 0x08: +/0/1 C [Restore Factory Default Color]
[04:15] <duflu> Control 0x10: +/50/100 C [Brightness]
[04:15] <duflu> Control 0x12: +/50/100 C [Contrast]
[04:15] <duflu> Control 0x14: +/0/65535 C [???]
[04:15] <duflu> Control 0x16: +/100/100 C [Red maximum level]
[04:15] <duflu> Control 0x18: +/100/100 C [Green maximum level]
[04:15] <duflu> Control 0x1a: +/100/100 C [Blue maximum level]
[04:15] <duflu> Control 0x1e: +/0/1   [???]
[04:15] <duflu> Control 0x60: +/16/65535 C [Input Source Select]
[04:15] <duflu> Control 0x68: +/1/2   [???]
[04:15] <duflu> Control 0x6c: +/0/65535 C [Red minimum level]
[04:15] <duflu> Control 0x6e: +/0/65535 C [Green minimum level]
[04:15] <duflu> Control 0x70: +/0/65535 C [Blue minimum level]
[04:15] <duflu> Control 0x8a: +/50/65535   [???]
[04:15] <duflu> Control 0x8c: +/12/65535   [???]
[04:15] <duflu> Control 0xaa: +/1/65535 C [OSD Orientation - Landscape]
[04:15] <duflu> Control 0xac: +/8464/1 C [???]
[04:15] <duflu> Control 0xae: +/6000/65535 C [???]
[04:15] <duflu> Control 0xb0: +/0/1   [???]
[04:15] <duflu> Control 0xb6: +/3/65535 C [???]
[04:15] <duflu> Control 0xc0: +/0/65535   [???]
[04:15] <duflu> Control 0xc6: +/17868/65535 C [???]
[04:15] <duflu> Control 0xc8: +/269/147 C [???]
[04:16] <duflu> Control 0xc9: +/513/65535 C [???]
[04:16] <duflu> Control 0xca: +/0/2   [???]
[04:16] <duflu> Control 0xcc: +/2/11   [???]
[04:16] <duflu> Control 0xd6: +/1/65535 C [DPMS Control - On]
[04:16] <duflu> Control 0xdc: +/0/65535 C [???]
[04:16] <duflu> Control 0xdf: +/513/65535 C [???]
[04:16] <duflu> Control 0xe0: +/0/65535 C [???]
[04:16] <duflu> Control 0xe1: +/0/65535 C [Power control - Off]
[04:16] <duflu> Control 0xe2: +/0/65535 C [???]
[04:16] <duflu> Control 0xe3: +/0/65535 C [???]
[04:16] <duflu> Control 0xe4: +/0/65535 C [???]
[04:16] <duflu> Control 0xf0: +/0/65535 C [???]
[04:16] <duflu> Control 0xf1: +/3/65535 C [???]
[04:16] <duflu> Control 0xf2: +/0/65535 C [???]
[04:16] <duflu> Control 0xfd: +/102/65535 C [???]
[10:22] <alan_g> greyback: question answered? https://code.launchpad.net/~raof/mir/plumb-through-edid/+merge/312008
[10:23] <greyback> alan_g: yep
[12:17] <applemuncy_1> qtmir.surfaces: MirSurface[0x1f3b9e8,"unity8-dash"]::resize old (1920,1011), new (1080,1851)
[12:17] <applemuncy_1> WxH or HxW ?
[12:25] <alan_g> applemuncy_1: I've no special knowledge of qtmir, but WxH is normal
[12:29] <applemuncy_1> Thanks : )
[12:38] <greyback> yep, WxH
[18:17] <bschaefer> fritsch, hey, soo saw kodi just bump to 18, was there anything else i need to do for the mir PR?