Re: [PATCH net-next v13 12/14] net: ptp: Move ptp_clock_index() to builtin symbol

[Kory Maincent]

Hello Sai,

On Thu, 30 May 2024 11:23:10 +0000
Sai Krishna Gajula <saikrishnag@xxxxxxxxxxx> wrote:

> > Move ptp_clock_index() to builtin symbols to prepare for supporting get and
> > set hardware timestamps from ethtool, which is builtin.
> > 
> > Signed-off-by: Kory Maincent <kory.maincent@xxxxxxxxxxx>

> >  }
> > +
> > +int ptp_clock_index(struct ptp_clock *ptp) {
> > +	return ptp->index;
> > +}
> > +EXPORT_SYMBOL(ptp_clock_index);
> >   
> Please check the "build_clang - FAILED", "build_32bit - FAILED" build errors.

Could you be more explicit? Which config are you using?
What is the build error?

I don't really see how this patch can bring a 32bit or clang build error. 

Regards,
-- 
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com