From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <1452873018.6067.79.camel@pengutronix.de> Subject: Re: [PATCH v2 02/26] reset: Make reset_control_ops const From: Philipp Zabel To: Maxime Ripard Cc: Mike Turquette , Stephen Boyd , David Airlie , Thierry Reding , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-sunxi@googlegroups.com, Laurent Pinchart , Chen-Yu Tsai , Hans de Goede , Alexander Kaplan , Boris Brezillon , Wynter Woods , Thomas Petazzoni , Rob Clark , Daniel Vetter Date: Fri, 15 Jan 2016 16:50:18 +0100 In-Reply-To: <1452785109-6172-3-git-send-email-maxime.ripard@free-electrons.com> References: <1452785109-6172-1-git-send-email-maxime.ripard@free-electrons.com> <1452785109-6172-3-git-send-email-maxime.ripard@free-electrons.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 List-ID: Am Donnerstag, den 14.01.2016, 16:24 +0100 schrieb Maxime Ripard: > The ops pointer is holding a pointer to a structure that is usually not > modified. Make it const. > > Signed-off-by: Maxime Ripard > --- > include/linux/reset-controller.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/reset-controller.h b/include/linux/reset-controller.h > index ce6b962ffed4..a3a5bcdb1d02 100644 > --- a/include/linux/reset-controller.h > +++ b/include/linux/reset-controller.h > @@ -38,7 +38,7 @@ struct of_phandle_args; > * @nr_resets: number of reset controls in this reset controller device > */ > struct reset_controller_dev { > - struct reset_control_ops *ops; > + const struct reset_control_ops *ops; > struct module *owner; > struct list_head list; > struct device_node *of_node; Applied, thanks. regards Philipp From mboxrd@z Thu Jan 1 00:00:00 1970 From: p.zabel@pengutronix.de (Philipp Zabel) Date: Fri, 15 Jan 2016 16:50:18 +0100 Subject: [PATCH v2 02/26] reset: Make reset_control_ops const In-Reply-To: <1452785109-6172-3-git-send-email-maxime.ripard@free-electrons.com> References: <1452785109-6172-1-git-send-email-maxime.ripard@free-electrons.com> <1452785109-6172-3-git-send-email-maxime.ripard@free-electrons.com> Message-ID: <1452873018.6067.79.camel@pengutronix.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Am Donnerstag, den 14.01.2016, 16:24 +0100 schrieb Maxime Ripard: > The ops pointer is holding a pointer to a structure that is usually not > modified. Make it const. > > Signed-off-by: Maxime Ripard > --- > include/linux/reset-controller.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/reset-controller.h b/include/linux/reset-controller.h > index ce6b962ffed4..a3a5bcdb1d02 100644 > --- a/include/linux/reset-controller.h > +++ b/include/linux/reset-controller.h > @@ -38,7 +38,7 @@ struct of_phandle_args; > * @nr_resets: number of reset controls in this reset controller device > */ > struct reset_controller_dev { > - struct reset_control_ops *ops; > + const struct reset_control_ops *ops; > struct module *owner; > struct list_head list; > struct device_node *of_node; Applied, thanks. regards Philipp From mboxrd@z Thu Jan 1 00:00:00 1970 From: Philipp Zabel Subject: Re: [PATCH v2 02/26] reset: Make reset_control_ops const Date: Fri, 15 Jan 2016 16:50:18 +0100 Message-ID: <1452873018.6067.79.camel@pengutronix.de> References: <1452785109-6172-1-git-send-email-maxime.ripard@free-electrons.com> <1452785109-6172-3-git-send-email-maxime.ripard@free-electrons.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1452785109-6172-3-git-send-email-maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Maxime Ripard Cc: Mike Turquette , Stephen Boyd , David Airlie , Thierry Reding , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-clk-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org, linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org, Laurent Pinchart , Chen-Yu Tsai , Hans de Goede , Alexander Kaplan , Boris Brezillon , Wynter Woods , Thomas Petazzoni , Rob Clark , Daniel Vetter List-Id: devicetree@vger.kernel.org Am Donnerstag, den 14.01.2016, 16:24 +0100 schrieb Maxime Ripard: > The ops pointer is holding a pointer to a structure that is usually not > modified. Make it const. > > Signed-off-by: Maxime Ripard > --- > include/linux/reset-controller.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/reset-controller.h b/include/linux/reset-controller.h > index ce6b962ffed4..a3a5bcdb1d02 100644 > --- a/include/linux/reset-controller.h > +++ b/include/linux/reset-controller.h > @@ -38,7 +38,7 @@ struct of_phandle_args; > * @nr_resets: number of reset controls in this reset controller device > */ > struct reset_controller_dev { > - struct reset_control_ops *ops; > + const struct reset_control_ops *ops; > struct module *owner; > struct list_head list; > struct device_node *of_node; Applied, thanks. regards Philipp -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html