Skip to content

Commit 93e905f

Browse files
committed
drm: apple: av: Warn only once about failed calls
Reduce log spam while errors are still likely due missing state checks.
1 parent 7fce142 commit 93e905f

1 file changed

Lines changed: 11 additions & 4 deletions

File tree

  • drivers/gpu/drm/apple

drivers/gpu/drm/apple/av.c

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,9 @@ struct audiosrv_data {
5959
struct work_struct start_av_service_wq;
6060

6161
struct dcp_av_audio_cmds cmds;
62+
63+
bool warned_get_elements;
64+
bool warned_get_product_attrs;
6265
};
6366

6467
static void av_interface_init(struct apple_epic_service *service, const char *name,
@@ -207,10 +210,12 @@ int dcp_audiosrv_get_elements(struct device *dev, void *elements, size_t maxsize
207210
elements, maxsize, &size);
208211
up_write(&asrv->srv_rwsem);
209212

210-
if (ret)
213+
if (ret && asrv->warned_get_elements) {
211214
dev_err(dev, "audiosrv: error getting elements: %d\n", ret);
212-
else
215+
asrv->warned_get_elements = true;
216+
} else {
213217
dev_dbg(dev, "audiosrv: got %zd bytes worth of elements\n", size);
218+
}
214219

215220
return ret;
216221
}
@@ -228,10 +233,12 @@ int dcp_audiosrv_get_product_attrs(struct device *dev, void *attrs, size_t maxsi
228233
maxsize, &size);
229234
up_write(&asrv->srv_rwsem);
230235

231-
if (ret)
236+
if (ret && asrv->warned_get_product_attrs) {
232237
dev_err(dev, "audiosrv: error getting product attributes: %d\n", ret);
233-
else
238+
asrv->warned_get_product_attrs = true;
239+
} else {
234240
dev_dbg(dev, "audiosrv: got %zd bytes worth of product attributes\n", size);
241+
}
235242

236243
return ret;
237244
}

0 commit comments

Comments
 (0)