Re: [PATCH v2] drm/vkms: add module parameter to set background color

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

Subject: Re: [PATCH v2] drm/vkms: add module parameter to set background color
From: Simon Ser <contact@xxxxxxxxxxx>
Date: Mon, 10 Apr 2023 16:56:46 +0000
Cc: Haneen Mohammed <hamohammed.sa@xxxxxxxxx>, Rodrigo Siqueira <rodrigosiqueiramelo@xxxxxxxxx>, dri-devel@xxxxxxxxxxxxxxxxxxxxx, Melissa Wen <mwen@xxxxxxxxxx>, Igor Matheus Andrade Torrente <igormtorrente@xxxxxxxxx>
Feedback-id: 1358184:user:proton
In-reply-to: <20230410125435.128689-1-mcanal@igalia.com>
References: <20230410125435.128689-1-mcanal@igalia.com>

I think this should be a KMS property instead of a module parameter.
Is there a reason why this patch uses a module parameter? It breaks
user-space expectations.

Follow-Ups:
- Re: [PATCH v2] drm/vkms: add module parameter to set background color
  - From: Melissa Wen

References:
- [PATCH v2] drm/vkms: add module parameter to set background color
  - From: Maíra Canal

Prev by Date: Re: [Intel-gfx] [PATCH 1/8] drm/i915/mtl: Define MOCS and PAT tables for MTL
Next by Date: [PATCH] drm/msm/a6xx: initialize GMU mutex earlier
Previous by thread: [PATCH v2] drm/vkms: add module parameter to set background color
Next by thread: Re: [PATCH v2] drm/vkms: add module parameter to set background color
Index(es):
- Date
- Thread

[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