Re: [PATCH 03/15] drm/ast: Reorganize widescreen test around hardware Gens

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jocelyn


Am 27.01.25 um 15:26 schrieb Jocelyn Falempe:
On 24/01/2025 08:57, Thomas Zimmermann wrote:
Testing for support of widescreen modes mixes up various hardware
Gens. First branch by hardware Gen, then do specific tests for each
Gen. By default, widesscreen support is disabled.

Later patches will add more specific tests for each Gen.

I think it introduces a bit too much repeated code.

Maybe adding a small helper like this would help?

I'll add a helper here and for the code in patch 6

Best regards
Thomas


static bool ast_vgacrd0_support_wuxga(struct ast_device *ast) {
    u8 vgacrd0;
    vgacrd0 = ast_get_index_reg(ast, AST_IO_VGACRI, 0xd0);
    return !(vgacrd0 & AST_IO_VGACRD0_VRAM_INIT_BY_BMC) ||
        (vgacrd0 & AST_IO_VGACRD0_IKVM_WIDESCREEN));
}


--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux