- 29 Sep, 2017 33 commits
-
-
Dave Airlie authored
turns out to be a win to inline this. Signed-off-by:
Dave Airlie <airlied@redhat.com> Reviewed-by:
Harry Wentland <harry.wentland@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Dave Airlie authored
This just makes it easier to find. Signed-off-by:
-
Dave Airlie authored
These aren't currently used outside this file. Signed-off-by:
-
Dave Airlie authored
This is worth 300 bytes, and one less source file. Signed-off-by:
-
Dave Airlie authored
Signed-off-by:
-
Dave Airlie authored
This only gets us 100 bytes, but may as well be consistent. Signed-off-by:
-
Dave Airlie authored
This uses two things that might be Linux specific, __builtin_constant_p (might be gcc) and BUILD_BUG_ON. (maybe other dm's can have something similiar). This saves 4k in the bw calcs code. Signed-off-by:
-
Dave Airlie authored
This results in a ~4.5k code size reduction. Signed-off-by:
-
Dave Airlie authored
Signed-off-by:
-
Dave Airlie authored
I've no idea if these are used on other platforms, but they aren't used outside this file here, so make them static. Drops 300 bytes. Signed-off-by:
-
Dave Airlie authored
This decreases code size by a few hundred bytes. Signed-off-by:
-
Dave Airlie authored
This drops ~400 bytes here. Signed-off-by:
-
Dave Airlie authored
This just removes more unused return/errors paths. Signed-off-by:
-
Dave Airlie authored
Signed-off-by:
-
Dave Airlie authored
This never returned anything else. Signed-off-by:
-
Dave Airlie authored
This removes any remaining pointless return codepaths from the DCE code. Signed-off-by:
-
Dave Airlie authored
This is similiar to previous patches, don't return when we don't need to, also do error checking before allocating memory, makes it simpler to cleanup after. Signed-off-by:
-
Dave Airlie authored
There was lots of return true, and error checking that was never used in these paths. Just remove it all. v2: I missed one return true. Signed-off-by:
-
Dave Airlie authored
There is no need to check for these pointers being valid at this level. Check earlier if required. Signed-off-by:
-
Dave Airlie authored
The return value was unused. Signed-off-by:
-
Dave Airlie authored
This function never returned false under any sane circumstances. Signed-off-by:
-
Dave Airlie authored
Signed-off-by:
-
Dave Airlie authored
This never returns anything but true. Signed-off-by:
-
Dave Airlie authored
This doesn't return anything except true. Signed-off-by:
-
Dave Airlie authored
This only ever returned true. Signed-off-by:
-
Dave Airlie authored
The checks weren't useful here really. Signed-off-by:
-
Dave Airlie authored
This can't fail as is. Signed-off-by:
-
Dave Airlie authored
Signed-off-by:
-
Dave Airlie authored
Signed-off-by:
-
Dave Airlie authored
Signed-off-by:
-
Dave Airlie authored
Signed-off-by:
-
Dave Airlie authored
Signed-off-by:
-
Dave Airlie authored
This isn't referenced anywhere, and if it was it should be const. Signed-off-by:
-
- 28 Sep, 2017 7 commits
-
-
Harry Wentland authored
It's not in a good shape and currently completely unused. Signed-off-by:
-
Dave Airlie authored
These aren't used in the tree anywhere, and there is a TODO. Signed-off-by:
-
Dave Airlie authored
This adds ~50k to the driver text segment, and 10k to data segment. text data bss dec hex filename 2385556 39681 1045 2426282 2505aa drivers/gpu/drm/amd/amdgpu/amdgpu.o text data bss dec hex filename 2336593 28857 1045 2366495 241c1f drivers/gpu/drm/amd/amdgpu/amdgpu.o Signed-off-by:
-
Dave Airlie authored
This code isn't used, and this function is huge, reimport later if going to be used. Signed-off-by:
-
Dave Airlie authored
This extracts the bios parser object id handling into a common file. Signed-off-by:
-
Dave Airlie authored
This is unused code. Signed-off-by:
-
Dave Airlie authored
This reduces code size. Signed-off-by:
-
