On Wed, 18 Jan 2023, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> wrote:
> On Wed, Jan 18, 2023 at 06:06:24PM +0100, Nirmoy Das wrote:
>> Removed unused i915 var.
>>
>> Fixes: a273e95721e9 ("drm/i915: Allow switching away via vga-switcheroo if uninitialized")
>> Signed-off-by: Nirmoy Das <nirmoy.das@xxxxxxxxx>
>
> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
>
> and pushed to drm-misc-fixes.
>
> Thanks for the patch.
I guess this was a quick fix to avoid warnings about it.
But really, for i915 the way to go would be to have i915 local var, and
pass that around instead of struct drm_device *dev.
BR,
Jani.
>
>> ---
>> drivers/gpu/drm/i915/i915_driver.c | 2 --
>> 1 file changed, 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/i915_driver.c b/drivers/gpu/drm/i915/i915_driver.c
>> index 33e231b120c1..6469c7c1e154 100644
>> --- a/drivers/gpu/drm/i915/i915_driver.c
>> +++ b/drivers/gpu/drm/i915/i915_driver.c
>> @@ -1071,8 +1071,6 @@ static int i915_driver_open(struct drm_device *dev, struct drm_file *file)
>> */
>> static void i915_driver_lastclose(struct drm_device *dev)
>> {
>> - struct drm_i915_private *i915 = to_i915(dev);
>> -
>> intel_fbdev_restore_mode(dev);
>>
>> vga_switcheroo_process_delayed_switch();
>> --
>> 2.39.0
>>
--
Jani Nikula, Intel Open Source Graphics Center
