Kees,
>> This patch seems to be lost. Gustavo reviewed it on January 15, 2024
>> but the patch has not been applied since.
>
> This looks correct to me. I can pick this up if no one else snags it?
I guess my original reply didn't make it out, I don't see it in the
archives.
My objections were:
1. The original code is more readable to me than the proposed
replacement.
2. The original code has worked since introduced in 2012. Nobody has
touched it since, presumably it's fine.
3. I don't have the hardware and thus no way of validating the proposed
changes.
So what is the benefit of me accepting this patch? We have had several
regressions in these conversions. Had one just last week, almost
identical in nature to the one at hand.
I am all for fixing code which is undergoing active use and development.
But I really don't see the benefit of updating a legacy driver which
hasn't seen updates in ages. Why risk introducing a regression?
--
Martin K. Petersen Oracle Linux Engineering
