[Intel-gfx] [PATCH 3/4] drm/i915/vga: switch to use VGA definitions from video/vga.h

Jani Nikula jani.nikula at intel.com
Wed Feb 2 11:25:49 UTC 2022


On Wed, 02 Feb 2022, Ville Syrjälä <ville.syrjala at linux.intel.com> wrote:
>> I dropped the ball here a bit. If I respin the same patches, but with
>> the above line changed to one of these, is it okay? Which do you prefer?
>> 
>> 1)	outb(VGA_SEQ_CLOCK_MODE, VGA_SEQ_I); /* SR01 */
>> 
>> 2)	#define SR01 VGA_SEQ_CLOCK_MODE
>> 	outb(SR01, VGA_SEQ_I);
>> 
>> 3)	outb(0x01, VGA_SEQ_I);
>
> 3 seems like the best option of these.

Thanks, v2 on the list.

BR,
Jani.

-- 
Jani Nikula, Intel Open Source Graphics Center


More information about the Intel-gfx mailing list