[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