Re: [PATCH] drm: Combine identical if/elif code blocks

From: Thorsten Blum
Date: Mon Jun 03 2024 - 16:19:28 EST


On 16. May 2024, at 03:24, Luc Ma <onion0709@xxxxxxxxx> wrote:
> On Wed, 15 May 2024 at 17:31, Thorsten Blum <thorsten.blum@xxxxxxxxxx> wrote:
>> On 15. May 2024, at 11:22, Thorsten Blum <thorsten.blum@xxxxxxxxxx> wrote:
>>> On 15. May 2024, at 09:43, Luc Ma <onion0709@xxxxxxxxx> wrote:
>>>> On Tue, 14 May 2024 at 19:37, Thorsten Blum <thorsten.blum@xxxxxxxxxx> wrote:
>>>>>
>>>>> Merge the identical if/elif code blocks and remove the following two
>>>>> warnings reported by make includecheck:
>>>>>
>>>>> asm/ioctl.h is included more than once
>>>>> linux/types.h is included more than once
>>>>>
>>>>> Signed-off-by: Thorsten Blum <thorsten.blum@xxxxxxxxxx>
>>>>> ---
>>>>> include/uapi/drm/drm.h | 8 +-------
>>>>> 1 file changed, 1 insertion(+), 7 deletions(-)
>>>>>
>>>>> diff --git a/include/uapi/drm/drm.h b/include/uapi/drm/drm.h
>>>>> index 16122819edfe..315af7b19c97 100644
>>>>> --- a/include/uapi/drm/drm.h
>>>>> +++ b/include/uapi/drm/drm.h
>>>>> @@ -35,13 +35,7 @@
>>>>> #ifndef _DRM_H_
>>>>> #define _DRM_H_
>>>>>
>>>>> -#if defined(__KERNEL__)
>>>>> -
>>>>> -#include <linux/types.h>
>>>>> -#include <asm/ioctl.h>
>>>>> -typedef unsigned int drm_handle_t;
>>>>> -
>>>>> -#elif defined(__linux__)
>>>>
>>>> I think it is intentionally like that. Please see
>>>> https://patchwork.freedesktop.org/patch/78747/
>>>
>>> Thank you for the link.
>>>
>>> Does anyone know if the reason for the change from 2016 ("make
>>> headers_install can't handle fancy conditions, ...") is still valid?
>
> I guess it is. If you can try ./scripts/unifdef as below[1], you might
> find out the thing.
>
> [1]https://elixir.bootlin.com/linux/v6.9/source/scripts/headers_install.sh#L41

I just tested it with make headers_install and it worked fine.

Thorsten