On Tue, Jun 25, 2019 at 03:10:32PM +0100, Russell King - ARM Linux admin wrote: > On Tue, Jun 25, 2019 at 04:07:55PM +0200, Daniel Vetter wrote: > > Otherwise I like this. Biggest problem I'm seeing here is rolling this out > > everywhere, this is a lot of work. And without widespread adoptions it's > > not terribly useful for userspace. > > There will be cases where it's not possible, because the I2C bus is > hidden behind a chip that doesn't give you direct access to the DDC > bus. Oh sure, plus lots of connectors where there's just not ddc bus at all. But if we only roll this out for a handful of drivers it's also not great, that's what I meant. Looking at $ git grep drm_do_get_edid there's only very few drivers where the ddc bus is hidden. There's a lot more where it's not, and I think a big series to tackle those would serve extremely well to make a case for this sysfs link. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch