From patchwork Thu Jul 13 16:06:46 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jeff LaBundy X-Patchwork-Id: 702541 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 63AECC001B0 for ; Thu, 13 Jul 2023 16:07:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230468AbjGMQG7 (ORCPT ); Thu, 13 Jul 2023 12:06:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59722 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229754AbjGMQG6 (ORCPT ); Thu, 13 Jul 2023 12:06:58 -0400 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2074.outbound.protection.outlook.com [40.107.223.74]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6773C1BEB for ; Thu, 13 Jul 2023 09:06:56 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XufVWq5WBl0M7/+yE87iwUt3wJniX7qonbUh3D4S3+szkjo2QWqVDs9Tn/iH2S0XbrlUYO5Evb78W7w9NrHo82P1PnyIz/YxxxDlhCfUmdCPy/FWRQueZRrzeCXkfpksnRqSE46zvCoLd9deGzmcNCTWYxWJOy44a/FHlBfOq9KesQYPD5wpNwvghFFu2HRZRLzVxaV8nO3S4HCmDIHFUyULMcMyis4gnZhbXG5P3whXiwz7qPLa5RYo3ZPPbw+juoGtmakpGHTObDAb/kcud8J7pM7PBjXpvq+OwofwHEFz5ZCa9sKHUrjkLTrWTG/6hrIYZ1pxI+eiMXnP5a7FaA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=0FhaXKmVh+588uR+NvUJBLu8C6T5hiuNaAv2FT+0hAk=; b=AOOEHchHZWE0a7qk5xUi54eZ+cubcISa3yTViSAQD0vO1zAPMFXtRvpah5C2M+mdgy3X7BS8NUawlrOVAdOIzPa6uVJrOEV9CLhfE4jJpJTRcXauYM7N3jfmHvJNdLfvl4NbStQ6RubHTkKcVIXad8WgHcUliAdp3cP86I8KyeP/fpG8XwBV32gYRkLHjC5ULQP2B2X2f7tIydxstjlXC3ftAdHGHhUik7uHBqQeNBehRSdWkAW/nLGfdZSARgCUZhwTRGH2gltBqSwcdC6ZYPeyK5xhFtI6k2s/UjAFbK2CnxlUWFkZEtF2Py/sBtkRB+mkmmyPiOzs58pIJbG7oA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=labundy.com; dmarc=pass action=none header.from=labundy.com; dkim=pass header.d=labundy.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=NETORG5796793.onmicrosoft.com; s=selector1-NETORG5796793-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=0FhaXKmVh+588uR+NvUJBLu8C6T5hiuNaAv2FT+0hAk=; b=ghHug61ZKZc/XQZ4ceALbH42BiRf9uejhe2O8FRSrXAVJoGOJ7UwWzj+hF7R/99AT8r8zK0OURIHDLMIM75Fwx/CJDzgawrk057XbIXqHwk37a/QtDReILWVgLpOcfe8802d9+rIt73UtfhQ5XIqdhA4ZkxBoBMse3bx69tKC7M= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=labundy.com; Received: from SN4PR0801MB3774.namprd08.prod.outlook.com (2603:10b6:803:43::21) by CH3PR08MB9339.namprd08.prod.outlook.com (2603:10b6:610:1cd::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6565.31; Thu, 13 Jul 2023 16:06:51 +0000 Received: from SN4PR0801MB3774.namprd08.prod.outlook.com ([fe80::a1c0:edd6:d172:7ff7]) by SN4PR0801MB3774.namprd08.prod.outlook.com ([fe80::a1c0:edd6:d172:7ff7%6]) with mapi id 15.20.6565.028; Thu, 13 Jul 2023 16:06:51 +0000 Date: Thu, 13 Jul 2023 11:06:46 -0500 From: Jeff LaBundy To: dmitry.torokhov@gmail.com Cc: linux-input@vger.kernel.org, jeff@labundy.com Subject: [PATCH] Input: iqs7211 - point to match data directly Message-ID: Content-Disposition: inline X-ClientProxiedBy: SN6PR08CA0014.namprd08.prod.outlook.com (2603:10b6:805:66::27) To SN4PR0801MB3774.namprd08.prod.outlook.com (2603:10b6:803:43::21) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN4PR0801MB3774:EE_|CH3PR08MB9339:EE_ X-MS-Office365-Filtering-Correlation-Id: 6bd16f85-4e11-4ccb-ae4c-08db83bb2afe X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: aJ8vngD4WCJBmc07TLkzJ4vjxrrR4Hwy682lQPj26s2k24NNhZD8uZ0QO2GGICARX1+mZPOmj8BOXm7EsHjJII59YvXGRsFX7AoB6g2E1EYk49P76GaRDMRwTaDNV32xzZLWxIGSmLW0nNY+RolapAydJfDpvp0RsX35ndpqE83vJfFLJV4e7DSpY3t3DVrcm4juJZ31S/T5TuRGmMD6OkSr+DZEVolxh4OAWvLRceftNs9uTFBaTXThu6Tz7MRyDyDy2rSgijO6Ke3tRWO3nwf5oMu8QNiFtHBDEFH+qicPqSx2uhE1uvSnz4f+pSe1Pll3d4euYKCdqzA3unHUgJ+TUsafm343I+wg++oo+PdO+t39RUnwKj2QQq3S2oM5F0UXwT7B8VOJ0jJEUqnHrHg9QVPYgHQyjcLmJEyTx1ugECIxEtcQAqVvdu+2IOcm2EEjgmHfEgVsPswUtlf8mqUvv0hSp6J1bntegoYLUKHro7WRJjkDsR8+vD/VBMgLXHk7Gei4uTroLzIx6dO7eSzm/sSgEBejSM6RZa65Je8= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SN4PR0801MB3774.namprd08.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(7916004)(136003)(396003)(39830400003)(366004)(376002)(346002)(451199021)(6666004)(478600001)(83380400001)(966005)(186003)(6506007)(26005)(9686003)(107886003)(6512007)(6486002)(316002)(2906002)(41300700001)(38100700002)(66946007)(66476007)(66556008)(4326008)(6916009)(86362001)(8936002)(8676002)(5660300002)(33716001); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: MkTCBL8nTP9WeWGdV1onRWlYp2uQvgMQfrsRMnea6eGnvESX9QjrLdd2D9DxLk1vLRViMc3Dj9gsvZH4upsBTzlUhN0j9h0LS6X7ZrTYhzwk+tlyVPKAm9ceLYj6HKYthJeLsi0DR+x3xslYCyXnSIbJaKnsCXzPWIPjiuJxR+t9sqqUkWtSyWeSBOQ5Al6+YxJoNfy1KNqLWiTGL55bcwIiJN63MfNNghHfpOfLtXifgjIzgSPOACiQaxG/A2LVG/7DhwqzMHj4YetWNR0JTcqk4iQFWM5JoCkLJuT/ObbVqG/z+q10YAzwwCCwnUiC8Y1j+I4JsX3v47N6TFobrRCbuaaJZfkcJhpJiKtD3TqluRPu9V+M+7lQYjiyKBKMbEgXklBje9u/SUSHiM48M3k6F7iRzmKlHdSrn8clqTB0KwUU4ZLTKoN+G4Sa131nCVVqF3V+hVM4JxHEizu/uLWrs0hIZqcIWegu9yFxY9He6ATI2B0ozH95clL98aLphvv97AWWr8vRuV5pwi91p187h7/kq8b/8/CUQJZwDejQpu0udwFEC0xhLju0ZD8jmBu2qbZYwKZM9pd1LqR7A/DW5AMDt4HHlbSBVhnUvwK7mpOdyxun+PpCy+pYyztO8fHKaeWIuEDMek6jUV2662GaD/7vXAli3RBfmtQsWvmCRlzOWk/QjkL7mUOj3n16EyeAkX4BSZ1aDk4sgS/WPGx7S2O3ldrCK1E1uHOKMyEERaFSMImOye+0CiwB0AGXaUQk5fwXuWeXcL32emLvmvLeoRPEYQt/yo7YgLj1WDbQzzu+t/PzzmBQyoNv63ICAJNYUQ7+NBI2WnUe0YgPJPUL5pQQJf8e7UAxDl2ORdkSF56Ah9aqAPb8F1x9/2aY9CY3N7ClLHft6JU03dGXJ0xL+UaIQLvVzFNQ6iMd6oBQ7FJI//bww6Nzgv/A2hW3yL7XM8kWZQ28pAjuGFHMVitSlhtDPJ9ac3cSv3aQBgI7uAAawVhjGEyClFTCYWQN8oz3JfYpXJ66i3nOc43B5XBJ0HXRWXRdv2Y7cQILOfF17F7dHL9fBafozcMil3u+LrjzoPiX+zEw/kYaLxwO+fJfIv/Y0iM4aYGDnzO1HordK65u7HDCWwuUTeV6Z2WE52kXWW6R+b5wf84+dNHoOs40vd+5cfJL/OSll4qNnxq/1heytYzDlWtEaMzlInA3+k+/IWMRUGDGw8/+KbPWEQ8DC+/2chdyyRakguwcNSPlMiYI2DwRfJPgSiCzFXVWsbRzsnGzrcU4y0Kml8t1qGr/iO2S+dX0+Wn0huHeQGCC0mhAxK40hW+G9qyc/5z9z7ZPx89YT72km7Tz0TUzD5cP4wFrt1ASt5pqgS+jbSI75d5wanMSU0rpUKtX/LDiOVJVvUEET3aXRKXTPzC1trpPz9f8CJ2arbTm1UY5O4kSorqbP2Ds1202AdUabtkH66cRWefi5fqu3LwDWnLQTPoyDmVpwUWl4MtQ95PD6lpLd81gOkVgAIF2FpxEdZ5JLDu0+zz4qwfh8hnA2LCdj7IfImFo+VVoiwuQME+K91m0TgJeqYw77szi8PBecu4E X-OriginatorOrg: labundy.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6bd16f85-4e11-4ccb-ae4c-08db83bb2afe X-MS-Exchange-CrossTenant-AuthSource: SN4PR0801MB3774.namprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Jul 2023 16:06:50.9222 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 00b69d09-acab-4585-aca7-8fb7c6323e6f X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 7E0v30QKVU37fmGqC9tKIAEFlrlCi4V4n/eFBIBk+/Jp7WgWhfAn22HDxYQyPWWqDq+TMhnP8/Lb7oh+YKbsbQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR08MB9339 Precedence: bulk List-ID: X-Mailing-List: linux-input@vger.kernel.org Point the OF match table directly to the struct that describes the device as opposed to an intermediate enum; doing so simplifies the code and avoids a clang warning. As part of this change, the I2C device ID table is removed, as the device cannot probe without an OF node due to the unique nature of the hardware's interrupt pin. Fixes: f2ba47e65f3b ("Input: add support for Azoteq IQS7210A/7211A/E") Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202307131717.LtwApG0z-lkp@intel.com/ Signed-off-by: Jeff LaBundy --- drivers/input/touchscreen/iqs7211.c | 26 +++++++------------------- 1 file changed, 7 insertions(+), 19 deletions(-) diff --git a/drivers/input/touchscreen/iqs7211.c b/drivers/input/touchscreen/iqs7211.c index f60316d37f45..d41bbdd096e7 100644 --- a/drivers/input/touchscreen/iqs7211.c +++ b/drivers/input/touchscreen/iqs7211.c @@ -2420,33 +2420,24 @@ ATTRIBUTE_GROUPS(iqs7211); static const struct of_device_id iqs7211_of_match[] = { { .compatible = "azoteq,iqs7210a", - .data = (void *)IQS7210A, + .data = &iqs7211_devs[IQS7210A], }, { .compatible = "azoteq,iqs7211a", - .data = (void *)IQS7211A, + .data = &iqs7211_devs[IQS7211A], }, { .compatible = "azoteq,iqs7211e", - .data = (void *)IQS7211E, + .data = &iqs7211_devs[IQS7211E], }, { } }; MODULE_DEVICE_TABLE(of, iqs7211_of_match); -static const struct i2c_device_id iqs7211_id[] = { - { "iqs7210a", IQS7210A }, - { "iqs7211a", IQS7211A }, - { "iqs7211e", IQS7211E }, - { } -}; -MODULE_DEVICE_TABLE(i2c, iqs7211_id); - static int iqs7211_probe(struct i2c_client *client) { struct iqs7211_private *iqs7211; enum iqs7211_reg_grp_id reg_grp; - enum iqs7211_dev_id dev_id; unsigned long irq_flags; bool shared_irq; int error, irq; @@ -2460,13 +2451,11 @@ static int iqs7211_probe(struct i2c_client *client) INIT_LIST_HEAD(&iqs7211->reg_field_head); - if (client->dev.of_node) - dev_id = (enum iqs7211_dev_id)of_device_get_match_data(&client->dev); - else - dev_id = i2c_match_id(iqs7211_id, client)->driver_data; + iqs7211->dev_desc = of_device_get_match_data(&client->dev); + if (!iqs7211->dev_desc) + return -ENODEV; - shared_irq = iqs7211_devs[dev_id].num_ctx == IQS7211_MAX_CTX; - iqs7211->dev_desc = &iqs7211_devs[dev_id]; + shared_irq = iqs7211->dev_desc->num_ctx == IQS7211_MAX_CTX; /* * The RDY pin behaves as an interrupt, but must also be polled ahead @@ -2554,7 +2543,6 @@ static int iqs7211_probe(struct i2c_client *client) static struct i2c_driver iqs7211_i2c_driver = { .probe = iqs7211_probe, - .id_table = iqs7211_id, .driver = { .name = "iqs7211", .of_match_table = iqs7211_of_match,