Re: [PATCH] drm: fix minor number range calculation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

On Thu, Aug 8, 2013 at 7:10 PM, David Herrmann <dh.herrmann@xxxxxxxxx> wrote:
> Currently, both ranges overlap. Fix the limits so both ranges are mutually
> exclusive. Also use the occasion to convert whitespaces to tabs.
>
> Signed-off-by: Kristian Høgsberg <krh@xxxxxxxxxxxxx>
> (fixed up tabs and adjust commit-msg accordingly)
> Signed-off-by: David Herrmann <dh.herrmann@xxxxxxxxx>

Umh, somehow this dropped the author line. It's supposed to be:
  Author: Kristian Høgsberg <krh@xxxxxxxxxxxxx>

Regards
David

> ---
>  drivers/gpu/drm/drm_stub.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_stub.c b/drivers/gpu/drm/drm_stub.c
> index d663f7d..ca24ad0 100644
> --- a/drivers/gpu/drm/drm_stub.c
> +++ b/drivers/gpu/drm/drm_stub.c
> @@ -113,12 +113,12 @@ static int drm_minor_get_id(struct drm_device *dev, int type)
>         int base = 0, limit = 63;
>
>         if (type == DRM_MINOR_CONTROL) {
> -                base += 64;
> -                limit = base + 127;
> -        } else if (type == DRM_MINOR_RENDER) {
> -                base += 128;
> -                limit = base + 255;
> -        }
> +               base += 64;
> +               limit = base + 63;
> +       } else if (type == DRM_MINOR_RENDER) {
> +               base += 128;
> +               limit = base + 63;
> +       }
>
>         mutex_lock(&dev->struct_mutex);
>         ret = idr_alloc(&drm_minors_idr, NULL, base, limit, GFP_KERNEL);
> --
> 1.8.3.4
>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel





[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux