Commit a897f65b authored by Ben Skeggs's avatar Ben Skeggs Committed by Danilo Krummrich

drm/nouveau/nvif: remove client device arg

This was once used by userspace tools (with nvkm built as a library), as
a way to select a "default device".

The DRM code doesn't need this at all as clients only have access to a
single device already, so inherit the value from its parent.
Signed-off-by: default avatarBen Skeggs <bskeggs@nvidia.com>
Signed-off-by: default avatarDanilo Krummrich <dakr@kernel.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20240726043828.58966-20-bskeggs@nvidia.com
parent 7e0d8de4
...@@ -10,8 +10,7 @@ struct nvif_client { ...@@ -10,8 +10,7 @@ struct nvif_client {
u64 version; u64 version;
}; };
int nvif_client_ctor(struct nvif_client *parent, const char *name, u64 device, int nvif_client_ctor(struct nvif_client *parent, const char *name, struct nvif_client *);
struct nvif_client *);
void nvif_client_dtor(struct nvif_client *); void nvif_client_dtor(struct nvif_client *);
int nvif_client_ioctl(struct nvif_client *, void *, u32); int nvif_client_ioctl(struct nvif_client *, void *, u32);
int nvif_client_suspend(struct nvif_client *); int nvif_client_suspend(struct nvif_client *);
......
...@@ -5,7 +5,6 @@ ...@@ -5,7 +5,6 @@
struct nvif_client_v0 { struct nvif_client_v0 {
__u8 version; __u8 version;
__u8 pad01[7]; __u8 pad01[7];
__u64 device;
char name[32]; char name[32];
}; };
......
...@@ -241,7 +241,6 @@ nouveau_cli_init(struct nouveau_drm *drm, const char *sname, ...@@ -241,7 +241,6 @@ nouveau_cli_init(struct nouveau_drm *drm, const char *sname,
{ NVIF_CLASS_VMM_NV04 , -1 }, { NVIF_CLASS_VMM_NV04 , -1 },
{} {}
}; };
u64 device = nouveau_name(drm->dev);
int ret; int ret;
snprintf(cli->name, sizeof(cli->name), "%s", sname); snprintf(cli->name, sizeof(cli->name), "%s", sname);
...@@ -254,8 +253,7 @@ nouveau_cli_init(struct nouveau_drm *drm, const char *sname, ...@@ -254,8 +253,7 @@ nouveau_cli_init(struct nouveau_drm *drm, const char *sname,
if (cli != &drm->master) { if (cli != &drm->master) {
mutex_lock(&drm->master.lock); mutex_lock(&drm->master.lock);
ret = nvif_client_ctor(&drm->master.base, cli->name, device, ret = nvif_client_ctor(&drm->master.base, cli->name, &cli->base);
&cli->base);
mutex_unlock(&drm->master.lock); mutex_unlock(&drm->master.lock);
} }
if (ret) { if (ret) {
......
...@@ -55,10 +55,9 @@ nvif_client_dtor(struct nvif_client *client) ...@@ -55,10 +55,9 @@ nvif_client_dtor(struct nvif_client *client)
} }
int int
nvif_client_ctor(struct nvif_client *parent, const char *name, u64 device, nvif_client_ctor(struct nvif_client *parent, const char *name, struct nvif_client *client)
struct nvif_client *client)
{ {
struct nvif_client_v0 args = { .device = device }; struct nvif_client_v0 args = {};
struct { struct {
struct nvif_ioctl_v0 ioctl; struct nvif_ioctl_v0 ioctl;
struct nvif_ioctl_nop_v0 nop; struct nvif_ioctl_nop_v0 nop;
......
...@@ -36,5 +36,5 @@ nvif_driver_init(const char *drv, const char *cfg, const char *dbg, ...@@ -36,5 +36,5 @@ nvif_driver_init(const char *drv, const char *cfg, const char *dbg,
if (ret) if (ret)
return ret; return ret;
return nvif_client_ctor(client, name, device, client); return nvif_client_ctor(client, name, client);
} }
...@@ -42,7 +42,7 @@ nvkm_uclient_new(const struct nvkm_oclass *oclass, void *argv, u32 argc, ...@@ -42,7 +42,7 @@ nvkm_uclient_new(const struct nvkm_oclass *oclass, void *argv, u32 argc,
if (!(ret = nvif_unpack(ret, &argv, &argc, args->v0, 0, 0, false))){ if (!(ret = nvif_unpack(ret, &argv, &argc, args->v0, 0, 0, false))){
args->v0.name[sizeof(args->v0.name) - 1] = 0; args->v0.name[sizeof(args->v0.name) - 1] = 0;
ret = nvkm_client_new(args->v0.name, args->v0.device, NULL, ret = nvkm_client_new(args->v0.name, oclass->client->device, NULL,
NULL, oclass->client->event, &client); NULL, oclass->client->event, &client);
if (ret) if (ret)
return ret; return ret;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment