<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_REOPENED " title="REOPENED - [snb] backlight issue on TOSHIBA PORTEGE R830 after suspend/resume" href="https://bugs.freedesktop.org/show_bug.cgi?id=82634#c18">Comment # 18</a> on <a class="bz_bug_link bz_status_REOPENED " title="REOPENED - [snb] backlight issue on TOSHIBA PORTEGE R830 after suspend/resume" href="https://bugs.freedesktop.org/show_bug.cgi?id=82634">bug 82634</a> from <a class="email" href="mailto:jani.nikula@intel.com" title="Jani Nikula <jani.nikula@intel.com>"> Jani Nikula</a> <pre>(In reply to Hans de Goede from <a href="show_bug.cgi?id=82634#c17">comment #17</a>) > Created <a href="attachment.cgi?id=116122" name="attach_116122" title="[PATCH] drm: i915: Add an enable_backlight_control module parameter">attachment 116122</a> <a href="attachment.cgi?id=116122&action=edit" title="[PATCH] drm: i915: Add an enable_backlight_control module parameter">[details]</a> <a href='page.cgi?id=splinter.html&bug=82634&attachment=116122'>[review]</a> [review] > [PATCH] drm: i915: Add an enable_backlight_control module parameter Hans, while I truly appreciate your efforts in fixing backlight issues, I really *really* would like to exhaust all other options before adding such module parameters. It should be made to Just Work(tm). All the cargo culting forum posts telling people to tweak their i915 module parameters are giving us headaches. We've even made some of our module parameters taint the kernel, and we won't look at bugs if people changed them from their platform specific defaults. Just wanted to let you know my opinion right from the start. ...but let's see if it helps. :)</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> <li>You are on the CC list for the bug.</li> <li>You are the assignee for the bug.</li> </ul> </body> </html>