[PATCH v5 04/11] drm/xe/configfs: Drop redundant init() error message

[John Harrison]

On 7/29/2025 4:42 AM, Michal Wajdeczko wrote:
> There is no need to print separate error message since we will
> also print one in xe_init(). Also drop temporary variable, which
> was likely just taken from the example code.
>
> Signed-off-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
> Cc: Lucas De Marchi <lucas.demarchi at intel.com>
> Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
Reviewed by: John Harrison <John.C.Harrison at Intel.com>

> ---
>   drivers/gpu/drm/xe/xe_configfs.c | 5 +----
>   1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_configfs.c b/drivers/gpu/drm/xe/xe_configfs.c
> index 56fbf4c1c37f..6aa0531bcf76 100644
> --- a/drivers/gpu/drm/xe/xe_configfs.c
> +++ b/drivers/gpu/drm/xe/xe_configfs.c
> @@ -392,15 +392,12 @@ u64 xe_configfs_get_engines_allowed(struct pci_dev *pdev)
>   
>   int __init xe_configfs_init(void)
>   {
> -	struct config_group *root = &xe_configfs.su_group;
>   	int ret;
>   
> -	config_group_init(root);
> +	config_group_init(&xe_configfs.su_group);
>   	mutex_init(&xe_configfs.su_mutex);
>   	ret = configfs_register_subsystem(&xe_configfs);
>   	if (ret) {
> -		pr_err("Error %d while registering %s subsystem\n",
> -		       ret, root->cg_item.ci_namebuf);
>   		mutex_destroy(&xe_configfs.su_mutex);
>   		return ret;
>   	}