Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757520AbeAHUw6 (ORCPT + 1 other); Mon, 8 Jan 2018 15:52:58 -0500 Received: from mail.kmu-office.ch ([178.209.48.109]:43984 "EHLO mail.kmu-office.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755717AbeAHUw4 (ORCPT ); Mon, 8 Jan 2018 15:52:56 -0500 MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Date: Mon, 08 Jan 2018 21:52:36 +0100 From: Stefan Agner To: Gary Bisson Cc: linus.walleij@linaro.org, shawnguo@kernel.org, kernel@pengutronix.de, fabio.estevam@nxp.com, aisheng.dong@nxp.com, robh+dt@kernel.org, mark.rutland@arm.com, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Arvind Yadav Subject: Re: [PATCH v2 1/5] pinctrl: imx: use struct imx_pinctrl_soc_info as a const In-Reply-To: <20180108164855.GA6504@t450s.lan> References: <20180106142553.15322-1-stefan@agner.ch> <20180106142553.15322-2-stefan@agner.ch> <20180108164855.GA6504@t450s.lan> Message-ID: User-Agent: Roundcube Webmail/1.2.5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On 2018-01-08 17:48, Gary Bisson wrote: > Hi Stefan, > > On Sat, Jan 06, 2018 at 03:25:49PM +0100, Stefan Agner wrote: >> For some SoCs the struct imx_pinctrl_soc_info is passed through >> of_device_id.data which is const. Most variables are already const >> or otherwise not written. However, some fields are modified at >> runtime. Move those fields to the dynamically allocated struct >> imx_pinctrl. >> >> Fixes: b3060044e495 ("pinctrl: freescale: imx7d: make of_device_ids const") >> Cc: Shawn Guo >> Cc: Arvind Yadav >> Cc: Dong Aisheng >> Cc: Gary Bisson >> Signed-off-by: Stefan Agner > > This is actually more or less a revert of a previous commit: > b28742be4709 pinctrl: imx: remove const qualifier of imx_pinctrl_soc_info Hm, I see. However, back then imx_pinctrl_probe still consumed a non-const struct imx_pinctrl_soc_info pointer. So this constifies all the way through. > > Note that the idea for this commit was to get dt-overlays working and > able to do pinctrl changes using configfs interface to load an overlay > (using Pantelis patch). Not sure where we stand on loading such overlay > from user-space, is it still something that will happen? I am all for dt-overlays and actually also hope that it will make it completely into mainline. So whatever prevents using device tree overlays should be addressed. It seems that ngroups is now part of struct pinctrl_dev (num_groups), which is still writable. So we should be fine? As far as I can tell all remaining properties come from the drivers directly and should not be influenced by anything in the device tree.... -- Stefan