Re: [PATCH] drm/amd/display: small cleanup in destruct()

2017-11-06 Thread Alex Deucher
On Mon, Nov 6, 2017 at 3:44 AM, Christian König wrote: > Am 06.11.2017 um 08:07 schrieb Dan Carpenter: >> >> Static analysis tools get annoyed that we don't indent this if >> statement. Actually, the if statement isn't required because kfree() >> can handle NULL

Re: [PATCH] drm/amd/display: small cleanup in destruct()

2017-11-06 Thread Christian König
Am 06.11.2017 um 08:07 schrieb Dan Carpenter: Static analysis tools get annoyed that we don't indent this if statement. Actually, the if statement isn't required because kfree() can handle NULL pointers just fine. The DCE110STRENC_FROM_STRENC() macro is a wrapper around container_of() but it's

[PATCH] drm/amd/display: small cleanup in destruct()

2017-11-05 Thread Dan Carpenter
Static analysis tools get annoyed that we don't indent this if statement. Actually, the if statement isn't required because kfree() can handle NULL pointers just fine. The DCE110STRENC_FROM_STRENC() macro is a wrapper around container_of() but it's basically a no-op or a cast. Anyway, it's not