Commit 83f04a7f authored by Lyude Paul's avatar Lyude Paul Committed by Greg Kroah-Hartman

drm/nouveau: Check backlight IDs are >= 0, not > 0

commit dc854914 upstream.

Remember, ida IDs start at 0, not 1!
Signed-off-by: default avatarLyude Paul <lyude@redhat.com>
Reviewed-by: default avatarKarol Herbst <kherbst@redhat.com>
Cc: stable@vger.kernel.org
Signed-off-by: default avatarBen Skeggs <bskeggs@redhat.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent c0845493
...@@ -116,7 +116,7 @@ nv40_backlight_init(struct drm_connector *connector) ...@@ -116,7 +116,7 @@ nv40_backlight_init(struct drm_connector *connector)
&nv40_bl_ops, &props); &nv40_bl_ops, &props);
if (IS_ERR(bd)) { if (IS_ERR(bd)) {
if (bl_connector.id > 0) if (bl_connector.id >= 0)
ida_simple_remove(&bl_ida, bl_connector.id); ida_simple_remove(&bl_ida, bl_connector.id);
return PTR_ERR(bd); return PTR_ERR(bd);
} }
...@@ -249,7 +249,7 @@ nv50_backlight_init(struct drm_connector *connector) ...@@ -249,7 +249,7 @@ nv50_backlight_init(struct drm_connector *connector)
nv_encoder, ops, &props); nv_encoder, ops, &props);
if (IS_ERR(bd)) { if (IS_ERR(bd)) {
if (bl_connector.id > 0) if (bl_connector.id >= 0)
ida_simple_remove(&bl_ida, bl_connector.id); ida_simple_remove(&bl_ida, bl_connector.id);
return PTR_ERR(bd); return PTR_ERR(bd);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment