Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932386AbcLGPW1 (ORCPT ); Wed, 7 Dec 2016 10:22:27 -0500 Received: from mail-wj0-f178.google.com ([209.85.210.178]:34441 "EHLO mail-wj0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751722AbcLGPW0 (ORCPT ); Wed, 7 Dec 2016 10:22:26 -0500 From: Bartosz Golaszewski To: Kevin Hilman , Michael Turquette , Sekhar Nori , Peter Ujfalusi , Russell King , Viresh Kumar , Boris Brezillon , "Rafael J. Wysocki" , Richard Weinberger , David Woodhouse , Brian Norris , Marek Vasut , Cyrille Pitchen Cc: LKML , arm-soc , linux-pm@vger.kernel.org, Bartosz Golaszewski Subject: [PATCH v4 0/4] ARM: da850: fix pll0 rate setting Date: Wed, 7 Dec 2016 16:22:14 +0100 Message-Id: <1481124138-27337-1-git-send-email-bgolaszewski@baylibre.com> X-Mailer: git-send-email 2.1.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1392 Lines: 36 While trying to set the pll0 rate from the kernel I noticed there are two issues with da850 clocks. The first patch fixes an infinite loop in propagate_rate(). The third fixes an oops in da850_set_pll0rate(). The second patch is just a coding style fix, while we're at it. v1 -> v2: - change the approach in 1/3: create a new clock for nand inheriting the rate from the aemif clock (verified that nand still works on da850-lcdk) - patch 3/3: also update the davinci_cpufreq driver - the only (indirect) user of da850_set_pll0rate() - s/requested_rate/rate in 3/3 v2 -> v3: - 1/3: keep the "aemif" connector id for the nand clock - 3/3: instead of multiplying freq->frequency, divide rate by 1000 - retested both davinci_nand and clk_set_rate() for pll0 v3 -> v4: - 1/4: add a comment explaining why we're leaving a NULL dev_id - 2/4: emac_clk is added twice to the clock lookup table too - fix it and verify that davinci_mmdio gets a functional clock - 4/4: s/opp == NULL/!opp/ Bartosz Golaszewski (4): ARM: da850: fix infinite loop in clk_set_rate() ARM: da850: don't add the emac clock to the clock lookup table twice ARM: da850: coding style fix ARM: da850: fix da850_set_pll0rate() arch/arm/mach-davinci/da850.c | 51 +++++++++++++++++++++++++++++++++------ drivers/cpufreq/davinci-cpufreq.c | 2 +- 2 files changed, 45 insertions(+), 8 deletions(-) -- 2.9.3