From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id EC953CD6E58 for ; Wed, 11 Oct 2023 10:08:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234819AbjJKKIt (ORCPT ); Wed, 11 Oct 2023 06:08:49 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35846 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234001AbjJKKIg (ORCPT ); Wed, 11 Oct 2023 06:08:36 -0400 Received: from muru.com (muru.com [72.249.23.125]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id F009F1732; Wed, 11 Oct 2023 03:07:28 -0700 (PDT) Received: from localhost (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 606E680E1; Wed, 11 Oct 2023 10:07:28 +0000 (UTC) Date: Wed, 11 Oct 2023 13:07:26 +0300 From: Tony Lindgren To: Andreas Kemnade Cc: dmitry.torokhov@gmail.com, Jonathan.Cameron@huawei.com, robh@kernel.org, frank.li@vivo.com, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org Subject: Re: [RFC PATCH] Input: omap4-keypad: react on keypresses if device is runtime-suspended Message-ID: <20231011100726.GD34982@atomide.com> References: <20230929200046.2300333-1-andreas@kemnade.info> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230929200046.2300333-1-andreas@kemnade.info> Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org Hi, * Andreas Kemnade [230929 23:01]: > According to SWPU235AB, table 26-6, fclk is required to generate events > at least on OMAP4460, so keep fclk enabled all the time the device > is opened. Sorry for the delay, the patch looks good to me: Reviewed-by: Tony Lindgren > Open questions: > - Should we rather (or also use) padconf irqs? The keypad is in the wakeup domain and has SYSC_OMAP2_ENAWAKEUP set in the dts, so padconf irqs should not be needed as long as the device is clocked. As the source clock is sys_32k_ck, not sure if gating it would save any power, so padconf interrupts are probably not needed. > - It seems not to be required everywhere. Does it harm somewhere? Sounds like on 4430 the gate bit does not do anything while on 4460 it does. The source clock seems to be sys_32k_ck so it's enabled anyways and should not affect power management. Regards, Tony