On Thu, Jan 09, 2025 at 11:00:43AM +0000, Shiju Jose wrote:
> The min_ and max_ attributes of the control attributes are added for your
> feedback on V15 to expose supported ranges of these control attributes to the user,
> in the following links.
Sure, but you can make that differently:
cat /sys/bus/edac/devices/<dev-name>/mem_repairX/bank
[x:y]
which is the allowed range.
echo ...
then writes in the bank.
> ... so we would propose we do not add max_ and min_ for now and see how the
> use cases evolve.
Yes, you should apply that same methodology to the rest of the new features
you're adding: only add functionality for the stuff that is actually being
used now. You can always extend it later.
Changing an already user-visible API is a whole different story and a lot lot
harder, even impossible.
So I'd suggest you prune the EDAC patches from all the hypothetical usage and
then send only what remains so that I can try to queue them.
Thx.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette
