Re: [PATCH v2] media: v4l2-common: Always register clock with device-specific name

[Mehdi Djait]

Hi Paul,


[..]

> > 	if (id)
> > 		clk_id = kasprintf(GFP_KERNEL, "clk-%s-%s",
> > dev_name(dev), id);
> > 	else
> > 		clk_id = kasprintf(GFP_KERNEL, "clk-%s-%s",
> > dev_name(dev));
> > 
> > 	if (!clk_id)
> > 		return ERR_PTR(-ENOMEM);
> 
> In the case where "id" is NULL then the name will be e.g.
> "clk-ov2680@1c-(null)" which I assumed was fine - even with a valid
> "id" the clock name wouldn't be very pretty anyway.
> 
> But I can't update it to your suggestion and send a v3.

Yes, please.

--
Kind Regards
Mehdi Djait