From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from foss.arm.com ([217.140.101.70]:51900 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750927AbcBPKqQ (ORCPT ); Tue, 16 Feb 2016 05:46:16 -0500 From: Andre Przywara To: Maxime Ripard , Emilio Lopez , Michael Turquette , Chen-Yu Tsai Cc: Stephen Boyd , linux-clk@vger.kernel.org, linux-sunxi@googlegroups.com, linux-arm-kernel@lists.infradead.org Subject: [PATCH v2 0/3] clk: sunxi: error checking on clock setup Date: Tue, 16 Feb 2016 10:46:05 +0000 Message-Id: <1455619568-1906-1-git-send-email-andre.przywara@arm.com> Sender: linux-clk-owner@vger.kernel.org List-ID: Setting up the clocks properly is quite critical to the system's operation, but currently our error handling is not very verbose. This series adds error handling and reporting to the sunxi clocks, so that any errors are correctly detected and reported. Also previous actions are rolled back in case something went wrong. This proves to be helpful in debugging clock tree issues, especially when adding support for new SoCs. I tested this on a BananaPi by deliberately misspelling "clock-output-names". The resulting kernel crash in sun4i_timer_interrupt is totally misleading, but now there is a line in the dmesg before saying: ======= sunxi_divider_clk_setup: could not read clock-output-names for "apb0" ======= This applies on top of Maxime's sunxi/for-next branch. Changes v1 .. v2: - in all three patches: - use of_clock_full_name() instead of node->name - don't report the uninitialized clk_name - handle error in final check in if-clause Andre Przywara (3): clk: sunxi: improve mux_clk error handling and reporting clk: sunxi: improve divider_clk error handling and reporting clk: sunxi: Improve divs_clk error handling and reporting drivers/clk/sunxi/clk-sunxi.c | 75 ++++++++++++++++++++++++++++++++++++------- 1 file changed, 63 insertions(+), 12 deletions(-) -- 2.6.4