On 21 February 2015 at 03:41, Alex Deucher <alexdeucher@xxxxxxxxx> wrote: > On Thu, Feb 19, 2015 at 7:43 PM, Alex Deucher <alexdeucher@xxxxxxxxx> wrote: >> On Thu, Feb 19, 2015 at 6:21 PM, Dave Airlie <airlied@xxxxxxxxx> wrote: >>> From: Dave Airlie <airlied@xxxxxxxxxx> >>> >>> The atombios tables have an unfortunate restriction on only >>> being able to write 12 bytes, MST really wants 16-bytes here, >>> and since the hw can do it, we should just write directly to it. >>> >>> This uses a module option to allow for it now, and maybe >>> we should provide the old code as a fallback for a while. >>> >>> Signed-off-by: Dave Airlie <airlied@xxxxxxxxxx> >>> --- >>> >>> Just putting this out there for people to know I'm doing it, >>> most likely I'd want to merge on by default for MST systems >>> at least. >>> >>> TODO: testing, irq support? >> >> This code will only work on DCE5+. Things moved around on older >> asics. That said, DCE5 is the first asic that supports MST, so we can >> stick with atom on the older ones. I'm fine with enabling it for DCE5 >> and newer by default. > > > I took a quick pass at cleaning this up. See attached. Oh this look much nicer! Thanks, I've tested it on a couple of monitors, probably should throw a few more things at it, We can look into irq support later I guess, as this should unblock my mst work. Dave. _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
