Messages in this thread | | | Date | Thu, 23 Jan 2020 12:02:40 +0000 | From | Charles Keepax <> | Subject | Re: [PATCH RESEND 1/2] regulator: arizona-ldo1: Improve handling of regulator unbinding |
| |
On Thu, Jan 23, 2020 at 11:48:05AM +0000, Mark Brown wrote: > On Thu, Jan 23, 2020 at 09:26:39AM +0000, Charles Keepax wrote: > > > 3) We could look at doing something in regmap IRQ to change when > > it does PM runtime calls, it is regmap doing runtime gets when > > drivers remove IRQs that causes the issue. But my accessment was > > that what regmap is doing makes perfect sense, so I don't think > > this is a good approach. > > Why do you even care about the errors? It's not like this device is > going to get removed in a production system and the primary IRQ will be > disabled when the core is removed, this is just something that happens > during development isn't it?
We do have certain customers who test unbinding the driver and complain when it throws errors. Admittedly, you are correct that it is a little bit of a stretch to imagine a situation where this is a massive problem in production. Best I can offer is one of our CODECs gets into a laptop and someone wants to unbind/bind the driver, to clear some issue or something.
Its been a while since I was looking at this issue, so I will double check to see if the issue is purely one of error messages or if it actually can cause problems (memory leaks or issues rebinding). But on a personal note I would be somewhat happier if we could come up with some acceptable way to make the driver unbind cleanly.
I am more than happy to do the leg work if we really don't like this solution. Do either you or Lee have any thoughts on my selective MFD remove helpers? That seemed like the most promising alternative solution to me.
Thanks, Charles
| |