[PATCH 1/8] cyclic: Prevent corruption of cyclic list on reassignment

Marek Vasut marek.vasut at mailbox.org
Mon Jan 27 12:44:50 CET 2025


On 1/26/25 10:49 PM, Rasmus Villemoes wrote:

[...]

>>> Also, I think I'd prefer if double cyclic_register() is allowed and
>>> always succeeds; this could be used to change the period of an already
>>> registered instance, for example.
>>
>> This would be terribly overloaded interface, no, let's not do that.
> 
> I disagree.

Can you please elaborate on that ?

>> Better introduce a dedicated function for that kind of period adjustment.
>>
>>> Also, that avoids making the
>>> interfaces fallible.
> 
> And I believe that _this_ is an important property to get/preserve.

What exactly does this mean in this case ?

[...]

I'll see what I can come up with in V2 for the rest.


More information about the U-Boot mailing list