Sorry about that Patrik. I will resend a v2 with just the FIXME removed. Regards Nick On 2014-11-30 08:02 AM, Patrik Jakobsson wrote: > On Sun, Nov 30, 2014 at 3:24 AM, Nicholas Krause <xerofoify@xxxxxxxxx> wrote: >> Removes unneeeded struct *lvds_i2c_bus of type, psb_intel_i2c_chan as this struct >> is no needed due to never being used in the header file, psb_drv.h and therefore >> should be removed from this header file. >> >> Signed-off-by: Nicholas Krause <xerofoify@xxxxxxxxx> > > The lvds_i2c_bus is in use and can't be removed. Please at least compile test > your patches before submitting them. > > Cheers > Patrik > >> --- >> drivers/gpu/drm/gma500/psb_drv.h | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/gma500/psb_drv.h b/drivers/gpu/drm/gma500/psb_drv.h >> index 55ebe2b..fc74518 100644 >> --- a/drivers/gpu/drm/gma500/psb_drv.h >> +++ b/drivers/gpu/drm/gma500/psb_drv.h >> @@ -522,7 +522,6 @@ struct drm_psb_private { >> struct drm_display_mode *sdvo_lvds_vbt_mode; >> >> struct bdb_lvds_backlight *lvds_bl; /* LVDS backlight info from VBT */ >> - struct psb_intel_i2c_chan *lvds_i2c_bus; /* FIXME: Remove this? */ >> >> /* Feature bits from the VBIOS */ >> unsigned int int_tv_support:1; >> -- >> 2.1.0 >> _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel