Skip to contentSkip to conversation

drivers:gpu:drm Remove unneeded struct of type psb_intel_i2c_chan in the header file psb_drv.h

PATCH In review Nicholas Krause 2 participants 3 messages Nov 30, 2014

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 drivers:gpu:drm Remove unneeded struct of type psb_intel_i2c_chan in the header file psb_drv.h

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.

---
 drivers/gpu/drm/gma500/psb_drv.h | 1 -
 1 file changed, 1 deletion(-)
drivers/gpu/drm/gma500/psb_drv.h +0 -1
index 55ebe2b..fc74518 100644
@@ -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;
Signature · 6 lines
2.1.0

_______________________________________________
dri-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel
maintainer
#1 lore raw
Quoted 5 lines · Nicholas Krause

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.

The lvds_i2c_bus is in use and can't be removed. Please at least compile test your patches before submitting them.

Cheers
Patrik

Quoted 17 lines
---
 drivers/gpu/drm/gma500/psb_drv.h | 1 -
 1 file changed, 1 deletion(-)
drivers/gpu/drm/gma500/psb_drv.h +0 -2
index 55ebe2b..fc74518 100644
@@ -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;
--
Quoted 2 lines

2.1.0

_______________________________________________
dri-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel

author
#2 lore raw

Sorry about that Patrik. I will resend a v2 with just the FIXME removed. Regards Nick

Quoted 24 lines · Patrik Jakobsson
Quoted 5 lines · Nicholas Krause

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.

The lvds_i2c_bus is in use and can't be removed. Please at least compile test your patches before submitting them.

Cheers
Patrik

Quoted 11 lines
---
 drivers/gpu/drm/gma500/psb_drv.h | 1 -
 1 file changed, 1 deletion(-)
drivers/gpu/drm/gma500/psb_drv.h +0 -0
index 55ebe2b..fc74518 100644
@@ -522,7 +522,6 @@ struct drm_psb_private {
        struct drm_display_mode *sdvo_lvds_vbt_mode;
 
Quoted 8 lines
Quoted 8 lines
drivers/gpu/drm/gma500/psb_drv.h +0 -2
        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
[email protected]
http://lists.freedesktop.org/mailman/listinfo/dri-devel