Re: [PATCH] INTEL DRM DRIVERS : No LVDS hardware on Intel D410PT and D425KT
On Sun, Oct 27, 2013 at 11:35:58AM -0700, Guenter Roeck wrote: > On 10/27/2013 10:33 AM, Greg KH wrote: > >On Sun, Oct 27, 2013 at 04:13:42PM +, Rob Pearce wrote: > >>From: Rob Pearce > >> > >>The Intel D410PT(LW) and D425KT Mini-ITX desktop boards both show up as > >>having LVDS but the hardware is not populated. This patch adds them to > >>the list of such systems. Patch is against 3.11.4 > >> > >>Signed-off-by: Rob Pearce > >>--- > >>Patch revised to match the D425KT exactly as the D425KTW does have LVDS. > >>According to Intel's documentation, the D410PTL and D410PLTW don't. > > > >Any reason you don't want this in the stable tree as well? > > > > Hi Greg, > > pardon my ignorance, but I thought this was supposed to be the maintainer's > call to make ? > Did I get this wrong ? Maintainer occasionally fumble it, so it's better when the patch submitter also thinks about this. I can always change it when I disagree ;-) -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] INTEL DRM DRIVERS : No LVDS hardware on Intel D410PT and D425KT
On 10/27/2013 10:33 AM, Greg KH wrote: On Sun, Oct 27, 2013 at 04:13:42PM +, Rob Pearce wrote: From: Rob Pearce The Intel D410PT(LW) and D425KT Mini-ITX desktop boards both show up as having LVDS but the hardware is not populated. This patch adds them to the list of such systems. Patch is against 3.11.4 Signed-off-by: Rob Pearce --- Patch revised to match the D425KT exactly as the D425KTW does have LVDS. According to Intel's documentation, the D410PTL and D410PLTW don't. Any reason you don't want this in the stable tree as well? Hi Greg, pardon my ignorance, but I thought this was supposed to be the maintainer's call to make ? Did I get this wrong ? Thanks, Guenter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] INTEL DRM DRIVERS : No LVDS hardware on Intel D410PT and D425KT
On 27/10/13 17:33, Greg KH wrote: > On Sun, Oct 27, 2013 at 04:13:42PM +, Rob Pearce wrote: >> From: Rob Pearce >> >> The Intel D410PT(LW) and D425KT Mini-ITX desktop boards both show up as >> having LVDS but the hardware is not populated. This patch adds them to >> the list of such systems. Patch is against 3.11.4 >> >> Signed-off-by: Rob Pearce >> --- >> Patch revised to match the D425KT exactly as the D425KTW does have LVDS. >> According to Intel's documentation, the D410PTL and D410PLTW don't. > > Any reason you don't want this in the stable tree as well? > No, should be in stable. Sorry, I'm obviously getting some etiquette wrong (this is the first patch I've submitted). Cheers, Rob -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] INTEL DRM DRIVERS : No LVDS hardware on Intel D410PT and D425KT
On Sun, Oct 27, 2013 at 10:33:02AM -0700, Greg KH wrote: > On Sun, Oct 27, 2013 at 04:13:42PM +, Rob Pearce wrote: > > From: Rob Pearce > > > > The Intel D410PT(LW) and D425KT Mini-ITX desktop boards both show up as > > having LVDS but the hardware is not populated. This patch adds them to > > the list of such systems. Patch is against 3.11.4 > > > > Signed-off-by: Rob Pearce > > --- > > Patch revised to match the D425KT exactly as the D425KTW does have LVDS. > > According to Intel's documentation, the D410PTL and D410PLTW don't. > > Any reason you don't want this in the stable tree as well? None. Picked up for -fixes, thanks for the patch. Also I prefer the patch change log in the commit message proper and less screaming in the summary ;-) All fixed while applying. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] INTEL DRM DRIVERS : No LVDS hardware on Intel D410PT and D425KT
On Sun, Oct 27, 2013 at 04:13:42PM +, Rob Pearce wrote: > From: Rob Pearce > > The Intel D410PT(LW) and D425KT Mini-ITX desktop boards both show up as > having LVDS but the hardware is not populated. This patch adds them to > the list of such systems. Patch is against 3.11.4 > > Signed-off-by: Rob Pearce > --- > Patch revised to match the D425KT exactly as the D425KTW does have LVDS. > According to Intel's documentation, the D410PTL and D410PLTW don't. Any reason you don't want this in the stable tree as well? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] INTEL DRM DRIVERS : No LVDS hardware on Intel D410PT and D425KT
Hi Daniel, On 27/10/13 13:51, Daniel Vetter wrote: >> +.matches = { >> > + DMI_MATCH(DMI_BOARD_VENDOR, "Intel"), >> > + DMI_MATCH(DMI_BOARD_NAME, "D425KT"), > At least this one here has a KTW variant with lvds connector. I think we > need a DMI_EXACT_MATCH. I haven't found out whether the D410PT board also > has such a cousin, so please digg in a bit for me. > Yes, you're right, sorry. I've had a dig and it looks like the D410PT variants don't have LVDS (the differences in that range are legacy I/O, PCIe and wireless). I'll re-submit with the 425 as an exact match. Regards, Rob -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] INTEL DRM DRIVERS : No LVDS hardware on Intel D410PT and D425KT
On Sun, Oct 27, 2013 at 01:35:30PM +, Rob Pearce wrote: > From: Rob Pearce > > These Intel D410PT and D425KT Mini-ITX desktop boards both show up as > having LVDS but the hardware is not populated. This patch adds them to > the list of such systems. Tested against 3.9.10 and 3.11.4 > > Signed-off-by: Rob Pearce > --- > diff -uprN -X linux-3.9.10/Documentation/dontdiff > linux-3.9.10/drivers/gpu/drm/i915/intel_lvds.c > linux-3.9.10-ovs/drivers/gpu/drm/i915/intel_lvds.c > --- linux-3.9.10/drivers/gpu/drm/i915/intel_lvds.c 2013-10-22 > 19:00:30.0 +0100 > +++ linux-3.9.10-ovs/drivers/gpu/drm/i915/intel_lvds.c 2013-10-22 > 18:58:56.0 +0100 > @@ -843,6 +843,22 @@ > }, > { > .callback = intel_no_lvds_dmi_callback, > + .ident = "Intel D410PT", > + .matches = { > + DMI_MATCH(DMI_BOARD_VENDOR, "Intel"), > + DMI_MATCH(DMI_BOARD_NAME, "D410PT"), > + }, > + }, > + { > + .callback = intel_no_lvds_dmi_callback, > + .ident = "Intel D425KT", > + .matches = { > + DMI_MATCH(DMI_BOARD_VENDOR, "Intel"), > + DMI_MATCH(DMI_BOARD_NAME, "D425KT"), At least this one here has a KTW variant with lvds connector. I think we need a DMI_EXACT_MATCH. I haven't found out whether the D410PT board also has such a cousin, so please digg in a bit for me. Thanks, Daniel > + }, > + }, > + { > + .callback = intel_no_lvds_dmi_callback, > .ident = "Supermicro X7SPA-H", > .matches = { > DMI_MATCH(DMI_SYS_VENDOR, "Supermicro"), > -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/