Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional

From: Lothar WaÃmann
Date: Mon Jan 25 2016 - 07:51:36 EST


Hi,

On Mon, 25 Jan 2016 12:41:23 +0000 Mark Brown wrote:
> On Mon, Jan 25, 2016 at 01:37:31PM +0100, Lothar WaÃmann wrote:
> > On Fri, 22 Jan 2016 16:26:10 +0000 Mark Brown wrote:
>
> > > You're not just making the interrupt optional, you are also implementing
> > > polling support. That's really unusual and there's no clear reason for
>
> > Any suggestions how to handle this case in a more sensible way?
>
> The above, for example - make the interrupt optional.
>
This will make it impossible to notify the system about
overtemperature (and undervoltage).
I implemented polling to be able to get at least overtemperature
warnings.
(Undervoltage cannot be handled sensibly without interrupt anyway)


Lothar WaÃmann