- 06 Oct, 2017 9 commits
-
-
Dave Airlie authored
These were misaligned. found by the cocci ifcol script. 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 dereference acrtc after freeing it. Found by the kfree cocci script. Signed-off-by:
-
Dave Airlie authored
Don't bother checking for it. Found with the cocci ifnullfree.cocci script. Signed-off-by:
-
Dave Airlie authored
This just aligns a few things with kernel style. Signed-off-by:
-
Dave Airlie authored
This isn't used or required. Signed-off-by:
-
Dave Airlie authored
Doesn't appear to be used. Signed-off-by:
-
Dave Airlie authored
This should produce the same result. Signed-off-by:
-
Dave Airlie authored
We allocate this struct zeroed, so don't need to memset in the constructor. Signed-off-by:
-
Dave Airlie authored
This is worth 400 bytes. Signed-off-by:
-
- 29 Sep, 2017 31 commits
-
-
Dave Airlie authored
turns out to be a win to inline this. Signed-off-by:
-
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:
-
