From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752530AbcAOPue (ORCPT ); Fri, 15 Jan 2016 10:50:34 -0500 Received: from metis.ext.4.pengutronix.de ([92.198.50.35]:48989 "EHLO metis.ext.4.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752045AbcAOPuW (ORCPT ); Fri, 15 Jan 2016 10:50:22 -0500 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" X-Mailer: Evolution 3.12.9-1+b1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 2001:67c:670:100:96de:80ff:fec2:9969 X-SA-Exim-Mail-From: p.zabel@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@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