Re: [PATCH v9 01/13] rust: hrtimer: introduce hrtimer support
From: Andreas Hindborg
Date: Wed Feb 26 2025 - 14:47:47 EST
"Frederic Weisbecker" <frederic@xxxxxxxxxx> writes:
> Le Mon, Feb 24, 2025 at 08:52:35PM +0100, Andreas Hindborg a écrit :
>> > It's of course up to hrtimer maintainers. But I personally nack this
>> > kconfig, because it's not necessary, and hrtimer API has been stable for
>> > a while.
>>
>> Having the switch is fine for me, removing it is fine as well. It's just
>> an added convenience that might come in handy. But having this kconfig
>> very close to zero overhead, so I do not really understand your
>> objection. I would like to better understand your reasoning.
>
> If you choose to make a such a Kconfig switch, it would only make sense
> in order to spare some bytes when no drivers use it for example. But if
> you're afraid that the Rust binding is on the way while the core is
> changing some API then I guess simply disabling Rust would be enough for
> testing.
>
> I don't think it's necessary (unless it's strictly selected by drivers).
> But it's your call.
I that case, I will drop it. Thanks for chiming in.
Best regards,
Andreas Hindborg