Re: [EXTERNAL] [PATCH 2/3] net: ti: icss-iep: Enable compare events

[Diogo Ivo]

Hi Sunil,

On 5/31/24 6:12 AM, Sunil Kovvuri Goutham wrote:
> > From: Diogo Ivo <diogo.ivo@xxxxxxxxxxx>
> > Sent: Wednesday, May 29, 2024 9:35 PM
> >
> > +	iep->cap_cmp_irq = platform_get_irq_byname_optional(pdev,
> > "iep_cap_cmp");
> > +	if (iep->cap_cmp_irq < 0) {
> > +		if (iep->cap_cmp_irq == -EPROBE_DEFER)
> > +			return iep->cap_cmp_irq;
>
> This info is coming from DT, is PROBE_DIFFER error return value possible ?

From my understanding -EPROBE_DEFER is a possible error code.
platform_get_irq_byname_optional() will eventually call of_irq_get()
where we get -EPROBE_DEFER if the IRQ domain still hasn't been
initialized.

> > +		iep->cap_cmp_irq = 0;
> > +	} else {
> > +		ret = devm_request_irq(dev, iep->cap_cmp_irq,
> > +				       icss_iep_cap_cmp_irq,
> > IRQF_TRIGGER_HIGH,
> > +				       "iep_cap_cmp", iep);
> > +		if (ret)
> > +			return dev_err_probe(iep->dev, ret,
> > +					     "Request irq failed for cap_cmp\n");
>
> Can't this driver live without this feature ?

Yes it can! I'll rework this logic so that we do not fail to probe
here and simply continue without PPS event support.

Thank you for the review!

Best regards,
Diogo