[PATCH v2 1/2] ARM: dts: aspeed: Add gpio-keys-polled for fan presence
Joel Stanley
joel at jms.id.au
Thu Jul 20 13:53:08 AEST 2017
On Thu, Jul 20, 2017 at 1:24 AM, Christopher Bostic
<cbostic at linux.vnet.ibm.com> wrote:
>
>
> On 7/19/17 4:28 AM, Joel Stanley wrote:
>>
>> On Wed, Jul 19, 2017 at 6:13 AM, Christopher Bostic
>> <cbostic at linux.vnet.ibm.com> wrote:
>>>
>>> + gpio-keys-polled {
>>> + compatible = "gpio-keys-polled";
>>> + #address-cells = <1>;
>>> + #size-cells = <0>;
>>> + #poll-interval = <100>;
>>
>> Are you sure that this needs to be polled this often?
>>
>> We should consider how much traffic this generates on the bus (how
>> often it is contended), and how that fits in with other devices on
>> that master. If you've done these calculations then please amend your
>> commit message with that information.
>
> Discussed with Brad Bishop and Matt Spinler - agreed to increase period to 1
> second.
Okay. You forgot to add this to the commit message.
Cheers,
Joel
More information about the openbmc
mailing list