[PATCH v4 04/10] coresight: Appropriately disable programming clocks

[Leo Yan]

Hi,

On Mon, Jul 21, 2025 at 10:08:27PM +0800, hejunhao wrote:
> On 2025/6/27 19:51, Leo Yan wrote:
> > diff --git a/include/linux/coresight.h b/include/linux/coresight.h
> > index 4ac65c68bbf44b98db22c3dad2d83a224ce5278e..dd2b4cc7a2b70cf060a3207548fe80e3824c489f 100644
> > --- a/include/linux/coresight.h
> > +++ b/include/linux/coresight.h
> > @@ -480,26 +480,16 @@ static inline bool is_coresight_device(void __iomem *base)
> >    * Returns:
> >    *
> >    * clk   - Clock is found and enabled
> > - * NULL  - clock is not found
> >    * ERROR - Clock is found but failed to enable
> >    */
> >   static inline struct clk *coresight_get_enable_apb_pclk(struct device *dev)
> >   {
> >   	struct clk *pclk;
> > -	int ret;
> > -	pclk = clk_get(dev, "apb_pclk");
> > -	if (IS_ERR(pclk)) {
> > -		pclk = clk_get(dev, "apb");
> > -		if (IS_ERR(pclk))
> > -			return NULL;
> Hi,
> 
> Here, the function returns NULL, but the caller uses IS_ERR() to check the
> function return value.
> Yes, this patch has already been fixed this, and should we split this fix
> into a separate patch?

I am not sure if I understand this question correctly.

Are you suggesting that we should use IS_ERR_OR_NULL() instead of
IS_ERR() to check the returned clock pointer?

If so, the answer is that we should not change it. As Suzuki mentioned,
we need to tolerate the absence of pclk in the ACPI case. So keep using
IS_ERR() is the right thing to do.

Thanks,
Leo