From nobody Mon Dec 15 21:54:12 2025 Received: from relay2-d.mail.gandi.net (relay2-d.mail.gandi.net [217.70.183.194]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 46EB11F0E32 for ; Tue, 21 Jan 2025 10:55:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.183.194 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737456945; cv=none; b=N7QJMumowdUKJPL/N3vOIsClsXRNHd78J4+rJ9pLQCF0xcRmHQo9pBOf1VcjH9vctaFmJPRlbrqMZ4IRyTNbdqxqBDmBxlEVqiZCOXbIBuMqa3w2FDzWTS9/LcKUW39dr60OJi8mIGcFCwkJ3eVHd9+p6cjXfm9MR8AK6s7qODc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737456945; c=relaxed/simple; bh=gsiygt2fQ2wFhE4Zgg0i+3O3oWlk+bFZUumT4XOlZdU=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=OFdDE/Szsowh2WeBOI4BUMgUV6SmgkrRrULMatqiEvGvSn+hFBmgIBTw052G5H4KB7HkCcH/Opt1yjPYcSaQ/cUWs0CxutohmZnCnDpoamYVpH9sAaA0av3y9yXpe9wnufsKF+Q5GTo8Zy3okNmvYdP7G5SfrzD2UperWtuQsRc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=NHQJxeRV; arc=none smtp.client-ip=217.70.183.194 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="NHQJxeRV" Received: by mail.gandi.net (Postfix) with ESMTPSA id 31C6040007; Tue, 21 Jan 2025 10:55:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1737456935; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=iMm1JXHZ3IwcVSCGDPx/XjO13zJJ9xnGIZllfHmkiI4=; b=NHQJxeRVdeAdw2pFDUzwZzA/7Ic9eZZJ/3x/Jp/mn9+II0mjgEnknqOGaMuHRDp3es1fY5 yk2GoHMSpXA47qL5baQxr3PeIua6+6nUpn1jplUoZamPq8tBHZHvdsU52OUd/KdsZJq4oc sUFrKiBNykNYxcSbUdS+IVf76HHEDkj4iXyvEuIkKcqM4N25CtNPX1uQtfkGX3k7wU+oDg gTGyXis5a9+c4wEWqkiO6P7eJagyZF4fW+RQj5jqF7C1RE5J+whvwBR47WjR9C2MYibCOt g9y8vKeQo18X1FELbKGxRYfdvrfzPM6sbf9y6dEET9LAnnuFJaC2k0/AwH0Rcw== From: Louis Chauvet Date: Tue, 21 Jan 2025 11:55:25 +0100 Subject: [PATCH v3 01/16] drm/vkms: Extract vkms_config header Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250121-google-remove-crtc-index-from-parameter-v3-1-cac00a3c3544@bootlin.com> References: <20250121-google-remove-crtc-index-from-parameter-v3-0-cac00a3c3544@bootlin.com> In-Reply-To: <20250121-google-remove-crtc-index-from-parameter-v3-0-cac00a3c3544@bootlin.com> To: =?utf-8?q?Ma=C3=ADra_Canal?= , Haneen Mohammed , Simona Vetter , Melissa Wen , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie Cc: arthurgrillo@riseup.net, jeremie.dautheribes@bootlin.com, miquel.raynal@bootlin.com, thomas.petazzoni@bootlin.com, seanpaul@google.com, nicolejadeyee@google.com, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Louis Chauvet , =?utf-8?q?Jos=C3=A9_Exp=C3=B3sito?= X-Mailer: b4 0.15-dev X-Developer-Signature: v=1; a=openpgp-sha256; l=8128; i=louis.chauvet@bootlin.com; h=from:subject:message-id; bh=gsiygt2fQ2wFhE4Zgg0i+3O3oWlk+bFZUumT4XOlZdU=; b=owEBbQKS/ZANAwAIASCtLsZbECziAcsmYgBnj30g6NBG9iIYnfgkAqQcgl2ahLsblntO0kYLg uADuzIq/ImJAjMEAAEIAB0WIQRPj7g/vng8MQxQWQQgrS7GWxAs4gUCZ499IAAKCRAgrS7GWxAs 4vIjEADWTC/tkJ09rL1yhXV4EtBcUc1C7MisH33176VJijVlQvUrjpZRP9p1c36hE1r7+T8VrdW 5tzXG5sTrl5e2WvYKYPEd1WVUS8Osy2JioVBKO6ARpPQ7cvlw9hEIsRLBuD0C03buoPwAHwSn2d hUUUc4qAaFoJ34Fbzv5nfKdu026fzvhJDFw3HZPkhybba64JUQ+N8cUPdWDQkhG3azusTLhKTdR TEbrpU2jcFBkAkxpQybuMEPaD+xvfOKi/usAHcg/6z2DeSdLlV6eDyAMmXAcezdtFeCyJ6W5u/e 5rlIje062Sl4tNTgRTHcGTUQVeCSPKCb78yaOE2qD+9xnLggu6wcDc8+VvOXmLSGRImVmkvOmt6 k4jtnacc/cY8BuUCsrQIGjOAmrleIw8jI34OZgiyBJAqv9UjOiWhCAEqlg6iuWlPkh14mYjVwQc zQQic5LhloXOdL8u/DSzj3rWii7niM06gTQOwMDoKKCKCR9trs5mY3PXykLtTsoMuYd+NurXzo/ +t7jNj3MjwPzxOubtzpi4WuCeAlQ80FH6wrg1/cbEskA7/RDxM+ykude4vgzgozV5aXEmgyTop6 5HL5ZeFqx6kQrwVLIE3TB34VjEOYm3e6xNmFy/54b1rBJ4QRakM9JmgZSWDodTxYuNVxdhQ7XVI OvZlmuesN5Ap0aA== X-Developer-Key: i=louis.chauvet@bootlin.com; a=openpgp; fpr=8B7104AE9A272D6693F527F2EC1883F55E0B40A5 X-GND-Sasl: louis.chauvet@bootlin.com Creating a new vkms_config structure will be more complex once we start adding more options. Extract the vkms_config structure to its own header and source files and add functions to create and delete a vkms_config and to initialize debugfs. Refactor, no functional changes. Signed-off-by: Louis Chauvet [Changes: Cherry picked and conflict solve] Signed-off-by: Jos=C3=A9 Exp=C3=B3sito --- drivers/gpu/drm/vkms/Makefile | 3 ++- drivers/gpu/drm/vkms/vkms_config.c | 48 ++++++++++++++++++++++++++++++++++= ++++ drivers/gpu/drm/vkms/vkms_config.h | 32 +++++++++++++++++++++++++ drivers/gpu/drm/vkms/vkms_drv.c | 41 +++++++++----------------------- drivers/gpu/drm/vkms/vkms_drv.h | 15 +----------- drivers/gpu/drm/vkms/vkms_output.c | 1 + 6 files changed, 95 insertions(+), 45 deletions(-) diff --git a/drivers/gpu/drm/vkms/Makefile b/drivers/gpu/drm/vkms/Makefile index 1b28a6a32948b557867dda51d2ccfdea687a2b62..b371b5d70ee31e76343c2f09e6d= c5a6e74a1c246 100644 --- a/drivers/gpu/drm/vkms/Makefile +++ b/drivers/gpu/drm/vkms/Makefile @@ -6,6 +6,7 @@ vkms-y :=3D \ vkms_formats.o \ vkms_crtc.o \ vkms_composer.o \ - vkms_writeback.o + vkms_writeback.o \ + vkms_config.o =20 obj-$(CONFIG_DRM_VKMS) +=3D vkms.o diff --git a/drivers/gpu/drm/vkms/vkms_config.c b/drivers/gpu/drm/vkms/vkms= _config.c new file mode 100644 index 0000000000000000000000000000000000000000..76355174a6ca54b880218c2bd45= 8c8339a3dabaa --- /dev/null +++ b/drivers/gpu/drm/vkms/vkms_config.c @@ -0,0 +1,48 @@ +// SPDX-License-Identifier: GPL-2.0+ + +#include +#include + +#include "vkms_config.h" +#include "vkms_drv.h" + +struct vkms_config *vkms_config_create(void) +{ + struct vkms_config *config; + + config =3D kzalloc(sizeof(*config), GFP_KERNEL); + if (!config) + return ERR_PTR(-ENOMEM); + + return config; +} +EXPORT_SYMBOL_IF_KUNIT(vkms_config_create); + +void vkms_config_destroy(struct vkms_config *config) +{ + kfree(config); +} +EXPORT_SYMBOL_IF_KUNIT(vkms_config_destroy); + +static int vkms_config_show(struct seq_file *m, void *data) +{ + struct drm_debugfs_entry *entry =3D m->private; + struct drm_device *dev =3D entry->dev; + struct vkms_device *vkmsdev =3D drm_device_to_vkms_device(dev); + + seq_printf(m, "writeback=3D%d\n", vkmsdev->config->writeback); + seq_printf(m, "cursor=3D%d\n", vkmsdev->config->cursor); + seq_printf(m, "overlay=3D%d\n", vkmsdev->config->overlay); + + return 0; +} + +static const struct drm_debugfs_info vkms_config_debugfs_list[] =3D { + { "vkms_config", vkms_config_show, 0 }, +}; + +void vkms_config_register_debugfs(struct vkms_device *vkms_device) +{ + drm_debugfs_add_files(&vkms_device->drm, vkms_config_debugfs_list, + ARRAY_SIZE(vkms_config_debugfs_list)); +} diff --git a/drivers/gpu/drm/vkms/vkms_config.h b/drivers/gpu/drm/vkms/vkms= _config.h new file mode 100644 index 0000000000000000000000000000000000000000..b284831738743f6d7c452be03f9= 17a7d3975d173 --- /dev/null +++ b/drivers/gpu/drm/vkms/vkms_config.h @@ -0,0 +1,32 @@ +/* SPDX-License-Identifier: GPL-2.0+ */ + +#ifndef _VKMS_CONFIG_H +#define _VKMS_CONFIG_H + +#include +#include "vkms_drv.h" + +/** + * struct vkms_config - General configuration for VKMS driver + * + * @writeback: If true, a writeback buffer can be attached to the CRTC + * @cursor: If true, a cursor plane is created in the VKMS device + * @overlay: If true, NUM_OVERLAY_PLANES will be created for the VKMS devi= ce + * @dev: Used to store the current vkms device. Only set when the device i= s instancied. + */ +struct vkms_config { + bool writeback; + bool cursor; + bool overlay; + struct vkms_device *dev; +}; + +/** + * vkms_config_register_debugfs() - Register the debugfs file to display c= urrent configuration + */ +void vkms_config_register_debugfs(struct vkms_device *vkms_device); + +struct vkms_config *vkms_config_create(void); +void vkms_config_destroy(struct vkms_config *config); + +#endif //_VKMS_CONFIG_H diff --git a/drivers/gpu/drm/vkms/vkms_drv.c b/drivers/gpu/drm/vkms/vkms_dr= v.c index 7c142bfc3bd9de9556621db3e7f570dc0a4fab3a..6b0e5fd0e26f04359b0d1025249= 6c52b15d83896 100644 --- a/drivers/gpu/drm/vkms/vkms_drv.c +++ b/drivers/gpu/drm/vkms/vkms_drv.c @@ -28,6 +28,7 @@ #include =20 #include "vkms_drv.h" +#include "vkms_config.h" =20 #include #include @@ -81,22 +82,6 @@ static void vkms_atomic_commit_tail(struct drm_atomic_st= ate *old_state) drm_atomic_helper_cleanup_planes(dev, old_state); } =20 -static int vkms_config_show(struct seq_file *m, void *data) -{ - struct drm_debugfs_entry *entry =3D m->private; - struct drm_device *dev =3D entry->dev; - struct vkms_device *vkmsdev =3D drm_device_to_vkms_device(dev); - - seq_printf(m, "writeback=3D%d\n", vkmsdev->config->writeback); - seq_printf(m, "cursor=3D%d\n", vkmsdev->config->cursor); - seq_printf(m, "overlay=3D%d\n", vkmsdev->config->overlay); - - return 0; -} - -static const struct drm_debugfs_info vkms_config_debugfs_list[] =3D { - { "vkms_config", vkms_config_show, 0 }, -}; =20 static const struct drm_driver vkms_driver =3D { .driver_features =3D DRIVER_MODESET | DRIVER_ATOMIC | DRIVER_GEM, @@ -208,8 +193,7 @@ static int vkms_create(struct vkms_config *config) if (ret) goto out_devres; =20 - drm_debugfs_add_files(&vkms_device->drm, vkms_config_debugfs_list, - ARRAY_SIZE(vkms_config_debugfs_list)); + vkms_config_register_debugfs(vkms_device); =20 ret =3D drm_dev_register(&vkms_device->drm, 0); if (ret) @@ -229,21 +213,18 @@ static int vkms_create(struct vkms_config *config) static int __init vkms_init(void) { int ret; - struct vkms_config *config; - - config =3D kmalloc(sizeof(*config), GFP_KERNEL); - if (!config) - return -ENOMEM; =20 - default_config =3D config; + default_config =3D vkms_config_create(); + if (IS_ERR(default_config)) + return PTR_ERR(default_config); =20 - config->cursor =3D enable_cursor; - config->writeback =3D enable_writeback; - config->overlay =3D enable_overlay; + default_config->cursor =3D enable_cursor; + default_config->writeback =3D enable_writeback; + default_config->overlay =3D enable_overlay; =20 - ret =3D vkms_create(config); + ret =3D vkms_create(default_config); if (ret) - kfree(config); + vkms_config_destroy(default_config); =20 return ret; } @@ -272,7 +253,7 @@ static void __exit vkms_exit(void) if (default_config->dev) vkms_destroy(default_config); =20 - kfree(default_config); + vkms_config_destroy(default_config); } =20 module_init(vkms_init); diff --git a/drivers/gpu/drm/vkms/vkms_drv.h b/drivers/gpu/drm/vkms/vkms_dr= v.h index abbb652be2b5389f96cec78837117ceb9acef656..af7081c940d6c074dc01cf81805= 56a3d902e386d 100644 --- a/drivers/gpu/drm/vkms/vkms_drv.h +++ b/drivers/gpu/drm/vkms/vkms_drv.h @@ -189,20 +189,7 @@ struct vkms_output { spinlock_t composer_lock; }; =20 -/** - * struct vkms_config - General configuration for VKMS driver - * - * @writeback: If true, a writeback buffer can be attached to the CRTC - * @cursor: If true, a cursor plane is created in the VKMS device - * @overlay: If true, NUM_OVERLAY_PLANES will be created for the VKMS devi= ce - * @dev: Used to store the current VKMS device. Only set when the device i= s instantiated. - */ -struct vkms_config { - bool writeback; - bool cursor; - bool overlay; - struct vkms_device *dev; -}; +struct vkms_config; =20 /** * struct vkms_device - Description of a VKMS device diff --git a/drivers/gpu/drm/vkms/vkms_output.c b/drivers/gpu/drm/vkms/vkms= _output.c index 22f0d678af3ac8177e43c4ea730af3f2109de5f3..78e389d832c16e4dbaccda34fbf= 3215f34adeadf 100644 --- a/drivers/gpu/drm/vkms/vkms_output.c +++ b/drivers/gpu/drm/vkms/vkms_output.c @@ -1,6 +1,7 @@ // SPDX-License-Identifier: GPL-2.0+ =20 #include "vkms_drv.h" +#include "vkms_config.h" #include #include #include --=20 2.47.1