On Wed, 02 Feb 2022, Ville Syrjälä <ville.syrj...@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

Reply via email to