Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752360AbbBMHXM (ORCPT ); Fri, 13 Feb 2015 02:23:12 -0500 Received: from bhuna.collabora.co.uk ([93.93.135.160]:50973 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751625AbbBMHXJ (ORCPT ); Fri, 13 Feb 2015 02:23:09 -0500 Message-ID: <54DDA657.7030407@collabora.co.uk> Date: Fri, 13 Feb 2015 08:23:03 +0100 From: Javier Martinez Canillas User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.2.0 MIME-Version: 1.0 To: Stephen Boyd , Mike Turquette CC: Tomeu Vizoso , =?windows-1252?Q?Emilio_?= =?windows-1252?Q?L=F3pez?= , Peter De Schrijver , Robert Jarzmik , Giuseppe Cavallaro , linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 3/3] clk: Replace explicit clk assignment with __clk_hw_set_clk References: <1423749510-14525-1-git-send-email-javier.martinez@collabora.co.uk> <1423749510-14525-4-git-send-email-javier.martinez@collabora.co.uk> <54DD051D.8020701@codeaurora.org> In-Reply-To: <54DD051D.8020701@codeaurora.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1214 Lines: 46 Hello Stephen, On 02/12/2015 08:55 PM, Stephen Boyd wrote: > On 02/12/15 05:58, Javier Martinez Canillas wrote: >> >> The changes were made using the following cocinelle semantic patch: >> >> @i@ >> @@ >> >> @depends on i@ >> identifier dst; >> @@ >> >> - dst->clk = hw->clk; >> + __clk_hw_set_clk(dst, hw); >> >> @depends on i@ >> identifier dst; >> @@ >> >> - dst->hw.clk = hw->clk; >> + __clk_hw_set_clk(&dst->hw, hw); >> >> Fixes: 035a61c314eb3 ("clk: Make clk API return per-user struct clk instances") >> Signed-off-by: Javier Martinez Canillas > > Reviewed-by: Stephen Boyd > Thanks a lot for your review. > Did you run this on all files that include clk-provider.h? I hope there > aren't similar situations in arch/arm/ for example. > Yes, I did run spatch against all the files that include clk-provider.h but only were matches in the drivers/clk files changed by $subject. Best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/