On Fri, Jan 14, 2022 at 10:45:38PM +0300, Sergey Shtylyov wrote:
On 1/13/22 10:43 PM, Uwe Kleine-König wrote:
The subsystems regulator, clk and gpio have the concept of a dummy resource. For regulator, clk and gpio there is a semantic difference between the regular _get() function and the _get_optional() variant. (One might return the dummy resource, the other won't. Unfortunately which one implements which isn't the same for these three.) The difference between platform_get_irq() and platform_get_irq_optional() is only that the former might emit an error message and the later won't.
To prevent people's expectations that there is a semantic difference between these too, rename platform_get_irq_optional() to platform_get_irq_silent() to make the actual difference more obvious.
The #define for the old name can and should be removed once all patches currently in flux still relying on platform_get_irq_optional() are fixed.
Hm... I'm afraid that with this #define they would never get fixed... :-)
Agree. The problems I have listed in another reply should be fixed at once by the same series.