From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753410AbdKFOS5 (ORCPT ); Mon, 6 Nov 2017 09:18:57 -0500 Received: from mga06.intel.com ([134.134.136.31]:13056 "EHLO mga06.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752829AbdKFOS4 (ORCPT ); Mon, 6 Nov 2017 09:18:56 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,352,1505804400"; d="scan'208";a="332660677" From: Andy Shevchenko To: Darren Hart , platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Andy Shevchenko , Pierre-Louis Bossart Subject: [PATCH v1] platform/x86: pmc_atom: Fix parent clocks Date: Mon, 6 Nov 2017 16:17:52 +0200 Message-Id: <20171106141752.11827-1-andriy.shevchenko@linux.intel.com> X-Mailer: git-send-email 2.14.2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org PLL or XTAL is a choice of parents. They are not dependent to each other (direct citation): The source of the frequencies can be XTAL or PLL, depending on the configuration. Any of the two available frequencies can be selected for each of the platform clocks. According to datasheet on hand CherryTrail has them, though they both provide 19.2MHz frequency. Fixes: 282a4e4ce5f9 ("platform/x86: Enable Atom PMC platform clocks") Cc: Pierre-Louis Bossart Signed-off-by: Andy Shevchenko --- drivers/platform/x86/pmc_atom.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/pmc_atom.c b/drivers/platform/x86/pmc_atom.c index 77bac859342d..56734112e5fe 100644 --- a/drivers/platform/x86/pmc_atom.c +++ b/drivers/platform/x86/pmc_atom.c @@ -65,7 +65,7 @@ static const struct pmc_clk byt_clks[] = { { .name = "pll", .freq = 19200000, - .parent_name = "xtal", + .parent_name = NULL, }, {}, }; @@ -76,6 +76,11 @@ static const struct pmc_clk cht_clks[] = { .freq = 19200000, .parent_name = NULL, }, + { + .name = "pll", + .freq = 19200000, + .parent_name = NULL, + }, {}, }; -- 2.14.2