Age | Commit message (Collapse) | Author |
|
We're relying on 1.18 now, so drop compat with older ones.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-ast/-/merge_requests/11>
|
|
Our minimal version is known to have it, so no check needed anymore.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-ast/-/merge_requests/11>
|
|
Already effectively required by use of XNFcallocarray() introduced in
xorg/xserver@b96dc999 - xserver-1.18.0, released in Nov. 2015.
Allows dropping remnants of code for XAA and pre-pciaccess X servers
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-ast/-/merge_requests/9>
|
|
xnfcalloc is just an alias for XNFcallocarray() that doesn't seem to serve
any practical purpose, so it can go away once all drivers stopped using it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-ast/-/merge_requests/8>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Replace "... AST2500PreCatchCRT), 60, 2, 35 }"
with "... AST2500PreCatchCRT), 60, 2, 0x35 }"
|
|
Based on Linux DRM https://patchwork.kernel.org/patch/9578793/
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
ast_vgatool.c:3400:1: warning: ‘static’ is not at beginning of declaration [-Wold-style-declaration]
void static vGetDefaultSettings(ScrnInfoPtr pScrn)
^~~~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
At top level:
ast_mode.c:389:23: warning: ‘DAC_EGA’ defined but not used [-Wunused-variable]
static VBIOS_DAC_INFO DAC_EGA[] = {
^~~~~~~
ast_mode.c:370:23: warning: ‘DAC_TEXT’ defined but not used [-Wunused-variable]
static VBIOS_DAC_INFO DAC_TEXT[] = {
^~~~~~~~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Clears 67 out of 84 warnings of the form:
In file included from ast_cursor.c:53:0:
ast_cursor.c: In function ‘ASTLoadCursorImage’:
ast_vgatool.h:149:15: warning: variable ‘temp’ set but not used [-Wunused-but-set-variable]
ULONG temp; \
^
ast_cursor.c:293:9: note: in expansion of macro ‘WriteAST1180SOC’
WriteAST1180SOC(AST1180_GFX_BASE+AST1180_HWC1_PATTERNADDR, ulPatternAddr);
^~~~~~~~~~~~~~~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Found by using:
codespell --builtin clear,rare,usage,informal,code,names
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
|
|
|
|
|
|
|
|
Two other places in this file already use uint32_t when passing
it to PCI_READ_LONG.
Signed-off-by: Thomas Klausner <wiz@NetBSD.org>
|
|
|
|
|
|
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
This makes ast_vgatools.c much more readable.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
xf86UnclaimPciSlot() received a GDevPtr as 2nd argument with
ABI_VIDEODRV_VERSION 13.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
Chontel -> Chrontel
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
Move inclusion of local headers from ast.h to the individual
files. Include only those headers which are needed.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
- Made sure exported functions have 'AST' in their name.
This avoids name space conflicts with the server or other drivers.
- Placed declaration of exported functions in a header file.
This ensures that changes to the function type or arguments are followed
thru everywhere.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
- Made sure exported functions have 'AST' in their name.
This avoids name space conflicts with the server or other drivers.
- Placed declaration of exported functions in a header file.
This ensures that changes to the function type or arguments are followed
thru everywhere.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
- Make local functions static
This sometimes helps the compiler to optimize.
- Made sure exported functions have 'AST' in their name.
This avoids name space conflicts with the server or other drivers.
- Placed declaration of exported functions in a header file.
This ensures that changes to the function type or arguments are followed
thru everywhere.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
- Make sure exported functions have 'AST' in their name.
This avoids name space conflicts with the server or other drivers.
- Place declaration of exported functions in a header file.
This ensures that changes to the function type or arguments are followed
thru everywhere.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
- Make local functions static
This sometimes helps the compiler to optimize.
- Make sure exported functions have 'AST' in their name.
This avoids name space conflicts with the server or other drivers.
- Place declaration of exported functions in a header file.
This ensures that changes to the function type or arguments are followed
thru everywhere.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
- Make local functions static
This sometimes helps the compiler to optimize.
- Make sure exported functions have 'AST' in their name.
This avoids name space conflicts with the server or other drivers.
- Place declaration of exported functions in a header file.
This ensures that changes to the function type or arguments are followed
thru everywhere.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
SetI2CReg(), GetI2CReg()
wait_fw_ready(), read_data() and clear_cmd() are used nowhere
currently.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
An archaeological survey revealed that this ABI change took place
in 2006 already.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
For newer ABI versions IOADDRESS can be replaced by int
which is big enough to hold an PIO offset.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
xf86PciInfo.h is not needed in the ASpeed driver at all.
Signed-off-by: Egbert Eich <eich@suse.de>
Acked-by: Y.C. Chen <yc_chen@aspeedtech.com>
|
|
|
|
|
|
Seems some code came from cirrus driver by copy & paste. Fixed
this in error message about device already in use by KMS module.
Signed-off-by: Stefan Dirsch <sndirsch@suse.de>
Reviewed-by: Dave Airlie <airlied@redhat.com>
|
|
if we find a kernel driver the code leave the pci slot claimed which
stops us falling back. unclaim the pci slot. patch inspired by
correspondant patch for cirrus driver by Dave Airlie.
Signed-off-by: Stefan Dirsch <sndirsch@suse.de>
Reviewed-by: Dave Airlie <airlied@redhat.com>
|
|
This reverts commit 7f24775a468066601a79beaffb4efa5a36c5ed59.
|
|
This reverts commit 2a2ecb551781d89e8e2a3123bb7d5b20ee807953.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|