From patchwork Thu Apr 30 13:43:50 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Valek, Andrej" X-Patchwork-Id: 11520409 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 2AA8981 for ; Thu, 30 Apr 2020 14:04:57 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 1C062206D9 for ; Thu, 30 Apr 2020 14:04:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728651AbgD3OE4 (ORCPT ); Thu, 30 Apr 2020 10:04:56 -0400 Received: from david.siemens.de ([192.35.17.14]:41668 "EHLO david.siemens.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728628AbgD3NxJ (ORCPT ); Thu, 30 Apr 2020 09:53:09 -0400 X-Greylist: delayed 506 seconds by postgrey-1.27 at vger.kernel.org; Thu, 30 Apr 2020 09:53:07 EDT Received: from mail1.siemens.de (mail1.siemens.de [139.23.33.14]) by david.siemens.de (8.15.2/8.15.2) with ESMTPS id 03UDidUo014835 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 30 Apr 2020 15:44:39 +0200 Received: from dev.vm7.ccp.siemens.com ([158.226.192.214]) by mail1.siemens.de (8.15.2/8.15.2) with ESMTP id 03UDicKW011528; Thu, 30 Apr 2020 15:44:39 +0200 Received: from dev.vm7.ccp.siemens.com (localhost [127.0.0.1]) by dev.vm7.ccp.siemens.com (Postfix) with ESMTP id BD3EE7097C3; Thu, 30 Apr 2020 15:44:36 +0200 (CEST) From: Andrej Valek To: linux-input@vger.kernel.org Cc: hadess@hadess.net, andrej.valek@siemens.com Subject: [PATCH] Input: goodix - add option to disable firmware loading Date: Thu, 30 Apr 2020 15:43:50 +0200 Message-Id: <20200430134350.23746-1-andrej.valek@siemens.com> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 Sender: linux-input-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org Firmware file loadind for GT911 controller takes too much time (~60s). There is no check that configuration is the same which is already present. This happens always during boot, which makes touchscreen unusable. Add there an option to prevent firmware file loading, but keep it enabled by default. Signed-off-by: Andrej Valek --- drivers/input/touchscreen/goodix.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c index 02c75ea385e0..7a75e5137547 100644 --- a/drivers/input/touchscreen/goodix.c +++ b/drivers/input/touchscreen/goodix.c @@ -941,7 +941,9 @@ static int goodix_get_gpio_config(struct goodix_ts_data *ts) default: if (ts->gpiod_int && ts->gpiod_rst) { ts->reset_controller_at_probe = true; - ts->load_cfg_from_disk = true; + /* Prevent cfg loading for each start */ + ts->load_cfg_from_disk = !device_property_read_bool(dev, + "touchscreen-do-not-load-fw"); ts->irq_pin_access_method = IRQ_PIN_ACCESS_GPIO; } }