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 X-Spam-Level: X-Spam-Status: No, score=-15.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4CB93C433DB for ; Thu, 4 Mar 2021 23:07:23 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 00F2E64F79 for ; Thu, 4 Mar 2021 23:07:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231626AbhCDXHV (ORCPT ); Thu, 4 Mar 2021 18:07:21 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34132 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230403AbhCDXHV (ORCPT ); Thu, 4 Mar 2021 18:07:21 -0500 Received: from mail-lf1-x135.google.com (mail-lf1-x135.google.com [IPv6:2a00:1450:4864:20::135]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 831E9C061574 for ; Thu, 4 Mar 2021 15:07:19 -0800 (PST) Received: by mail-lf1-x135.google.com with SMTP id e7so664630lft.2 for ; Thu, 04 Mar 2021 15:07:19 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=4xPZrQKI9DzyQgI8l1KHtECGn6OvKmEcjIaCH3tRIG0=; b=gp193NP4aQsovNv6sTt76j2EtGF5dS+erNhe/XKhkWD1fK9TUgG/v9bOtIKmCes6IJ eX28hhrl8od93drGWqI8d2Ja0M7vIz76V9ZPgHMjQYOTfvTnwQT4LLFKd5/l9NHLzrLi DfPiEzRtLrZlD7v6Psd2xvn4DM6r6vWBq4DCEtYxAi/GJYxHrMiJstJyFOHEiJAi96ZZ RsO0uMtqqMg05quPedHfRvZ4OV9YCIOM72frf4fSJX0FUut3BdgJJyXJbyoJGFi269pv A+wFKARPjxNCLYxZC5PVxgaEw5AHc/YO7key9LPTLAzgVH9bv3cLcqOIZ3eT/j9QsgAJ 6HHg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=4xPZrQKI9DzyQgI8l1KHtECGn6OvKmEcjIaCH3tRIG0=; b=sCWdJFgVSt5+7Z049n6DKdIp+Os9wMk2o5AOdvsv1svTh9Zj9icRRynbBaG8aWFQah TNcLvJXv6R9qJu/es1DQSkFGVnLmPFVyCE7wWoTtpTMNMjmUWSddXwDSCQrKMDek9YoU Uo4FqTQMjWrZNjm7G49Qgs1t7Jn2LHtltcciJyHMv45yCyuBIYdxz2glQ8diXW6qE3qJ S5HEhBVipOs2p7/Eefqm/t/QuNgIqIMsp33IkHZ4FrRlGK9GsWVYvFyaZdeyJH9XYh3X ia1kojl/VJx8p1bUXMZw3EWazjzVxArpeA73Av52UVkUTD7NhadjeFcSBTiq25BChYDf zs2A== X-Gm-Message-State: AOAM530F5OSMEbX96yuzR57dA9pAth2sHiGyZR1PExJFetXieprYxW0h DUjQS/9xTPzantulGXlfo0oZTns66QY= X-Google-Smtp-Source: ABdhPJyFs+iZnPrdGgaBZhKn+g7iLdh1vBSF+AuDTfX76w0LSp9Kj6P0BekF17aSIyahVpPomhmUUw== X-Received: by 2002:ac2:47f4:: with SMTP id b20mr3499410lfp.524.1614899238094; Thu, 04 Mar 2021 15:07:18 -0800 (PST) Received: from localhost.localdomain (h-98-128-229-129.NA.cust.bahnhof.se. [98.128.229.129]) by smtp.gmail.com with ESMTPSA id w8sm84825lft.44.2021.03.04.15.07.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 04 Mar 2021 15:07:17 -0800 (PST) From: Rikard Falkeborn To: Lee Jones Cc: Maxime Coquelin , Alexandre Torgue , linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Rikard Falkeborn , Andy Shevchenko Subject: [PATCH v2] mfd: stmpe: Revert "Constify static struct resource" Date: Fri, 5 Mar 2021 00:07:09 +0100 Message-Id: <20210304230709.3217-1-rikard.falkeborn@gmail.com> X-Mailer: git-send-email 2.30.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org In stmpe_devices_init(), the start and end field of these structs are modified, so they can not be const. Add a comment to those structs that lacked it to reduce the risk that this happens again. This reverts commit 8d7b3a6dac4eae22c58b0853696cbd256966741b. Fixes: 8d7b3a6dac4e ("mfd: stmpe: Constify static struct resource") Reported-by: Andy Shevchenko Signed-off-by: Rikard Falkeborn --- Changes V1-V2: Add comments to the structs that they are dynamically modified Rewrite commit message. V1: https://lore.kernel.org/lkml/20210302234710.74455-1-rikard.falkeborn@gmail.com/ drivers/mfd/stmpe.c | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/drivers/mfd/stmpe.c b/drivers/mfd/stmpe.c index 90f3292230c9..1dd39483e7c1 100644 --- a/drivers/mfd/stmpe.c +++ b/drivers/mfd/stmpe.c @@ -312,7 +312,7 @@ EXPORT_SYMBOL_GPL(stmpe_set_altfunc); * GPIO (all variants) */ -static const struct resource stmpe_gpio_resources[] = { +static struct resource stmpe_gpio_resources[] = { /* Start and end filled dynamically */ { .flags = IORESOURCE_IRQ, @@ -336,7 +336,8 @@ static const struct mfd_cell stmpe_gpio_cell_noirq = { * Keypad (1601, 2401, 2403) */ -static const struct resource stmpe_keypad_resources[] = { +static struct resource stmpe_keypad_resources[] = { + /* Start and end filled dynamically */ { .name = "KEYPAD", .flags = IORESOURCE_IRQ, @@ -357,7 +358,8 @@ static const struct mfd_cell stmpe_keypad_cell = { /* * PWM (1601, 2401, 2403) */ -static const struct resource stmpe_pwm_resources[] = { +static struct resource stmpe_pwm_resources[] = { + /* Start and end filled dynamically */ { .name = "PWM0", .flags = IORESOURCE_IRQ, @@ -445,7 +447,8 @@ static struct stmpe_variant_info stmpe801_noirq = { * Touchscreen (STMPE811 or STMPE610) */ -static const struct resource stmpe_ts_resources[] = { +static struct resource stmpe_ts_resources[] = { + /* Start and end filled dynamically */ { .name = "TOUCH_DET", .flags = IORESOURCE_IRQ, @@ -467,7 +470,8 @@ static const struct mfd_cell stmpe_ts_cell = { * ADC (STMPE811) */ -static const struct resource stmpe_adc_resources[] = { +static struct resource stmpe_adc_resources[] = { + /* Start and end filled dynamically */ { .name = "STMPE_TEMP_SENS", .flags = IORESOURCE_IRQ, -- 2.30.1 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 X-Spam-Level: X-Spam-Status: No, score=-13.7 required=3.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED,DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E54E4C433E0 for ; Thu, 4 Mar 2021 23:09:17 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6FC5064F79 for ; Thu, 4 Mar 2021 23:09:17 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6FC5064F79 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Transfer-Encoding :Content-Type:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:Cc:To:From: Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender :Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=FsP7WiPV3AKtgddEXAF9V3Ttu98N+y6vY3dBa+2cPcM=; b=QutAU9biE2h3fLbO3ihEXwDGof cz/iFSOUC2Sc2tzn7oPuT84QEZLYYlwTfuMCKdQUkifPOuXmfbYMtelJKg/FqOnwM9tAqNP+R//Sy DuE+jDipUyMbSiIb/xWPG2KUIBzzti64um9+ewKYD2kaNUQ3HL9dtNk6SM+pkAe2u6AKRLXNfHLNV 0rcT3YiqG1WDxQlEf6oUprGo0qP2Vwk1tMZmE85w4Vr72ULDQOs82GBBUlZ3/oQP7ts6RVfKnZ6AL Lx+AyC4K2NO81SqqdIKAMLMSwP2jdHpEzgajmC4bKhmlZPIOTkxExlFUmGNn3mr1SeTE42lxdjZ7Q BJ2GwNfg==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lHx42-00AeAC-ON; Thu, 04 Mar 2021 23:07:27 +0000 Received: from mail-lf1-x133.google.com ([2a00:1450:4864:20::133]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lHx3x-00Ae8U-EG for linux-arm-kernel@lists.infradead.org; Thu, 04 Mar 2021 23:07:24 +0000 Received: by mail-lf1-x133.google.com with SMTP id k9so564429lfo.12 for ; Thu, 04 Mar 2021 15:07:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=4xPZrQKI9DzyQgI8l1KHtECGn6OvKmEcjIaCH3tRIG0=; b=gp193NP4aQsovNv6sTt76j2EtGF5dS+erNhe/XKhkWD1fK9TUgG/v9bOtIKmCes6IJ eX28hhrl8od93drGWqI8d2Ja0M7vIz76V9ZPgHMjQYOTfvTnwQT4LLFKd5/l9NHLzrLi DfPiEzRtLrZlD7v6Psd2xvn4DM6r6vWBq4DCEtYxAi/GJYxHrMiJstJyFOHEiJAi96ZZ RsO0uMtqqMg05quPedHfRvZ4OV9YCIOM72frf4fSJX0FUut3BdgJJyXJbyoJGFi269pv A+wFKARPjxNCLYxZC5PVxgaEw5AHc/YO7key9LPTLAzgVH9bv3cLcqOIZ3eT/j9QsgAJ 6HHg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=4xPZrQKI9DzyQgI8l1KHtECGn6OvKmEcjIaCH3tRIG0=; b=frMCT7RZPygr8FUuwIBWu8UgM5QxqE8IZHkKy0pLFcbB3GqxxKCNkAt4UMZmteDAIi oycqsakUtLU4c6q+oPoZnDbJUFJAPUYghSjd30Iebe7Le8u43HNClRoX75uvqqm+Y99t KFpLucdSChjosKjRlzLl+Z8tzO+8OsUzxT2IoKHVevAReR/L7uHJMOpTP4mMjxM7qr09 IRnirX51hgVJmA3OUOYR7Ixln/i51Gcc26IcIpVemOXmbWpOBnkLv4uxIH9PsxRIwaM6 XNAdnTPrFzcknc5plVwQkg/jb9ytJgqGrYqkWFoea/BXnnBk6eNle9yuIf42pjlZhL5X H/NQ== X-Gm-Message-State: AOAM532HRy68BdknT03PpeJTNtb003Slqe5PoIdlY/Gek6/QP/wNdu+c f0c4adEK62krYxCmk2jj0hU= X-Google-Smtp-Source: ABdhPJyFs+iZnPrdGgaBZhKn+g7iLdh1vBSF+AuDTfX76w0LSp9Kj6P0BekF17aSIyahVpPomhmUUw== X-Received: by 2002:ac2:47f4:: with SMTP id b20mr3499410lfp.524.1614899238094; Thu, 04 Mar 2021 15:07:18 -0800 (PST) Received: from localhost.localdomain (h-98-128-229-129.NA.cust.bahnhof.se. [98.128.229.129]) by smtp.gmail.com with ESMTPSA id w8sm84825lft.44.2021.03.04.15.07.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 04 Mar 2021 15:07:17 -0800 (PST) From: Rikard Falkeborn To: Lee Jones Cc: Maxime Coquelin , Alexandre Torgue , linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Rikard Falkeborn , Andy Shevchenko Subject: [PATCH v2] mfd: stmpe: Revert "Constify static struct resource" Date: Fri, 5 Mar 2021 00:07:09 +0100 Message-Id: <20210304230709.3217-1-rikard.falkeborn@gmail.com> X-Mailer: git-send-email 2.30.1 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210304_230722_473976_72F5023F X-CRM114-Status: GOOD ( 11.74 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org In stmpe_devices_init(), the start and end field of these structs are modified, so they can not be const. Add a comment to those structs that lacked it to reduce the risk that this happens again. This reverts commit 8d7b3a6dac4eae22c58b0853696cbd256966741b. Fixes: 8d7b3a6dac4e ("mfd: stmpe: Constify static struct resource") Reported-by: Andy Shevchenko Signed-off-by: Rikard Falkeborn --- Changes V1-V2: Add comments to the structs that they are dynamically modified Rewrite commit message. V1: https://lore.kernel.org/lkml/20210302234710.74455-1-rikard.falkeborn@gmail.com/ drivers/mfd/stmpe.c | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/drivers/mfd/stmpe.c b/drivers/mfd/stmpe.c index 90f3292230c9..1dd39483e7c1 100644 --- a/drivers/mfd/stmpe.c +++ b/drivers/mfd/stmpe.c @@ -312,7 +312,7 @@ EXPORT_SYMBOL_GPL(stmpe_set_altfunc); * GPIO (all variants) */ -static const struct resource stmpe_gpio_resources[] = { +static struct resource stmpe_gpio_resources[] = { /* Start and end filled dynamically */ { .flags = IORESOURCE_IRQ, @@ -336,7 +336,8 @@ static const struct mfd_cell stmpe_gpio_cell_noirq = { * Keypad (1601, 2401, 2403) */ -static const struct resource stmpe_keypad_resources[] = { +static struct resource stmpe_keypad_resources[] = { + /* Start and end filled dynamically */ { .name = "KEYPAD", .flags = IORESOURCE_IRQ, @@ -357,7 +358,8 @@ static const struct mfd_cell stmpe_keypad_cell = { /* * PWM (1601, 2401, 2403) */ -static const struct resource stmpe_pwm_resources[] = { +static struct resource stmpe_pwm_resources[] = { + /* Start and end filled dynamically */ { .name = "PWM0", .flags = IORESOURCE_IRQ, @@ -445,7 +447,8 @@ static struct stmpe_variant_info stmpe801_noirq = { * Touchscreen (STMPE811 or STMPE610) */ -static const struct resource stmpe_ts_resources[] = { +static struct resource stmpe_ts_resources[] = { + /* Start and end filled dynamically */ { .name = "TOUCH_DET", .flags = IORESOURCE_IRQ, @@ -467,7 +470,8 @@ static const struct mfd_cell stmpe_ts_cell = { * ADC (STMPE811) */ -static const struct resource stmpe_adc_resources[] = { +static struct resource stmpe_adc_resources[] = { + /* Start and end filled dynamically */ { .name = "STMPE_TEMP_SENS", .flags = IORESOURCE_IRQ, -- 2.30.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel