Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1316723
| From | Mark Brown <broonie@kernel.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional |
| Date | 2016-01-25 15:00 +0100 |
| Message-ID | <qUPQu-4Ie-17@gated-at.bofh.it> (permalink) |
| References | (5 earlier) <qTCLE-80H-3@gated-at.bofh.it> <qTML1-6v3-17@gated-at.bofh.it> <qUOB4-3Tm-33@gated-at.bofh.it> <qUOKL-3WU-45@gated-at.bofh.it> <qUOUq-40h-5@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
[Multipart message — attachments visible in raw view] - view raw
On Mon, Jan 25, 2016 at 01:51:09PM +0100, Lothar Waßmann wrote: > On Mon, 25 Jan 2016 12:41:23 +0000 Mark Brown wrote: > > 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) I'm not convinced that justifies constantly polling, if the system designers cared you'd hope they'd have wired it up to a working interrupt. People commonly don't, realistically thermal warnings are usually set near the point where the silicon will be physically damaged and typically by the time they fire the system is already experiencing catastrophic problems at a system level. The polling is at the very least a separate change, and making the interrupt work would be a much better option.
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
PATCH [0/2] ARM: dts: am33xx: add support for Ka-Ro electronics TX48-7020 module Lothar Waßmann <LW@KARO-electronics.de> - 2016-01-20 13:40 +0100
[PATCH 1/2] regulator: ltc3589: make IRQ optional Lothar Waßmann <LW@KARO-electronics.de> - 2016-01-20 13:40 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Mark Brown <broonie@kernel.org> - 2016-01-20 17:50 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Lothar Waßmann <LW@KARO-electronics.de> - 2016-01-21 08:10 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Mark Brown <broonie@kernel.org> - 2016-01-21 11:30 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Lothar Waßmann <LW@KARO-electronics.de> - 2016-01-21 11:30 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Mark Brown <broonie@kernel.org> - 2016-01-21 12:20 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Lothar Waßmann <LW@KARO-electronics.de> - 2016-01-21 12:40 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Mark Brown <broonie@kernel.org> - 2016-01-21 17:30 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Lothar Waßmann <LW@KARO-electronics.de> - 2016-01-22 06:50 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Mark Brown <broonie@kernel.org> - 2016-01-22 17:30 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Lothar Waßmann <LW@KARO-electronics.de> - 2016-01-25 13:40 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Mark Brown <broonie@kernel.org> - 2016-01-25 13:50 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Lothar Waßmann <LW@KARO-electronics.de> - 2016-01-25 14:00 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Mark Brown <broonie@kernel.org> - 2016-01-25 15:00 +0100
Re: [PATCH 1/2] regulator: ltc3589: make IRQ optional Grygorii Strashko <grygorii.strashko@ti.com> - 2016-01-20 18:40 +0100
csiph-web