Thread (32 messages) 32 messages, 4 authors, 2020-03-23

Re: [PATCH v2 05/10] media: adv748x: add support for HDMI audio

From: Alex Riesen <hidden>
Date: 2020-03-20 08:58:10
Also in: linux-clk, linux-media, linux-renesas-soc, lkml

Hi Geert,

Geert Uytterhoeven, Fri, Mar 20, 2020 09:43:29 +0100:
CC linux-clk for the clock provider.
Thanks!
quoted
+int adv748x_dai_init(struct adv748x_dai *dai)
+{
+       int ret;
+       struct adv748x_state *state = adv748x_dai_to_state(dai);
+
+       dai->mclk = clk_register_fixed_rate(state->dev,
+                                           "adv748x-hdmi-i2s-mclk",
I assume there can be multiple adv748x instances in the system?
Hence the clock name should be unique for each instance.
I think that can happen.

Is it alright to derive the clock name from the device name? E.g.:
adv748x.4-0070-mclk? Where "adv748x.4-0070" is a struct device->name.

Regards,
Alex
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help