[PATCH 8/8] drm/client: s/unsigned int i/int i/

Thomas Zimmermann tzimmermann at suse.de
Mon Oct 7 07:43:47 UTC 2024


Hi

Am 03.10.24 um 13:33 schrieb Ville Syrjala:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
> Replace the 'unsigned int i' footguns with plain old signed
> int. Avoids accidents if/when someone decides they need
> to iterate backwards.

Why are signed types preferable here?

Best regards
Thomas

>
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> ---
>   drivers/gpu/drm/drm_client_modeset.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_client_modeset.c b/drivers/gpu/drm/drm_client_modeset.c
> index ccf5c9b5537b..875d517fa8f2 100644
> --- a/drivers/gpu/drm/drm_client_modeset.c
> +++ b/drivers/gpu/drm/drm_client_modeset.c
> @@ -39,7 +39,7 @@ int drm_client_modeset_create(struct drm_client_dev *client)
>   	unsigned int max_connector_count = 1;
>   	struct drm_mode_set *modeset;
>   	struct drm_crtc *crtc;
> -	unsigned int i = 0;
> +	int i = 0;
>   
>   	/* Add terminating zero entry to enable index less iteration */
>   	client->modesets = kcalloc(num_crtc + 1, sizeof(*client->modesets), GFP_KERNEL);
> @@ -75,7 +75,7 @@ static void drm_client_modeset_release(struct drm_client_dev *client)
>   	struct drm_mode_set *modeset;
>   
>   	drm_client_for_each_modeset(modeset, client) {
> -		unsigned int i;
> +		int i;
>   
>   		drm_mode_destroy(client->dev, modeset->mode);
>   		modeset->mode = NULL;
> @@ -925,7 +925,7 @@ bool drm_client_rotation(struct drm_mode_set *modeset, unsigned int *rotation)
>   	struct drm_plane *plane = modeset->crtc->primary;
>   	struct drm_cmdline_mode *cmdline;
>   	u64 valid_mask = 0;
> -	unsigned int i;
> +	int i;
>   
>   	if (!modeset->num_connectors)
>   		return false;

-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)



More information about the dri-devel mailing list