[dunno what the distribution was originally; adding intel-gfx] On Sun, 12 Apr 2015, Lukas Wunner <lukas@xxxxxxxxx> wrote: > All 17" models with i915 graphics had a resolution of 1920x1200, > necessitating dual channel LVDS. The 15" pre-retina models had > up to 1680x1050, hence used dual channel LVDS as well. I think this begs the question, why don't we assume dual link lvds when we know the native mode of the panel necessitates it? > Reference for MacBookPro9,1 (also tested by myself): > http://ubuntuforums.org/showthread.php?t=2224567 > > Reference for MacBookPro8,2 / 8,3: > https://wiki.archlinux.org/index.php/MacBookPro8,1/8,2/8,3_(2011) > > Reference for MacBookPro6,2: > https://bbs.archlinux.org/viewtopic.php?id=185770 I really wish the references were more conclusive on the dual channel parameter, or that we had Tested-by on the patch with the module parameter dropped. We don't have much chances for testing this, but if the patch is valid, it's definitely cc: stable material. As is, applying this gives me an uneasy feeling. :( BR, Jani. > > Signed-off-by: Lukas Wunner <lukas@xxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_lvds.c | 34 +++++++++++++++++++++++++++++++++- > 1 file changed, 33 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_lvds.c b/drivers/gpu/drm/i915/intel_lvds.c > index 071b96d..ee1000f 100644 > --- a/drivers/gpu/drm/i915/intel_lvds.c > +++ b/drivers/gpu/drm/i915/intel_lvds.c > @@ -812,12 +812,44 @@ static int intel_dual_link_lvds_callback(const struct dmi_system_id *id) > static const struct dmi_system_id intel_dual_link_lvds[] = { > { > .callback = intel_dual_link_lvds_callback, > - .ident = "Apple MacBook Pro (Core i5/i7 Series)", > + .ident = "Apple MacBook Pro 17\" (2010)", > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "Apple Inc."), > + DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro6,1"), > + }, > + }, > + { > + .callback = intel_dual_link_lvds_callback, > + .ident = "Apple MacBook Pro 15\" (2010)", > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "Apple Inc."), > + DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro6,2"), > + }, > + }, > + { > + .callback = intel_dual_link_lvds_callback, > + .ident = "Apple MacBook Pro 15\" (2011)", > .matches = { > DMI_MATCH(DMI_SYS_VENDOR, "Apple Inc."), > DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro8,2"), > }, > }, > + { > + .callback = intel_dual_link_lvds_callback, > + .ident = "Apple MacBook Pro 17\" (2011)", > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "Apple Inc."), > + DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro8,3"), > + }, > + }, > + { > + .callback = intel_dual_link_lvds_callback, > + .ident = "Apple MacBook Pro 15\" (2012)", > + .matches = { > + DMI_MATCH(DMI_SYS_VENDOR, "Apple Inc."), > + DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro9,1"), > + }, > + }, > { } /* terminating entry */ > }; > > -- > 1.8.5.2 (Apple Git-48) > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
