From mboxrd@z Thu Jan 1 00:00:00 1970 From: jbrunet@baylibre.com (Jerome Brunet) Date: Tue, 26 Jun 2018 13:19:45 +0200 Subject: [PATCH 0/2] clk: meson: drop clk-audio-divider In-Reply-To: <20180620100610.29010-1-jbrunet@baylibre.com> References: <20180620100610.29010-1-jbrunet@baylibre.com> Message-ID: <1530011985.2900.45.camel@baylibre.com> To: linus-amlogic@lists.infradead.org List-Id: linus-amlogic.lists.infradead.org On Wed, 2018-06-20 at 12:06 +0200, Jerome Brunet wrote: > As explained in the commit descriptions, clk-audio-divider was a poor > solution to make sure PLL rates would be high enough while using CCF > rate propagation to determine audio clock rates. > > Stopping rate propagation before the PLLs is actually simpler and provides > better results. It also allow to get rid of clk-audio-divider. > > Off tree sound cards using these clocks should add the following to the > sound card DT to properly setup the PLLs: > > assigned-clocks = <&clkc CLKID_MPLL2>, > <&clkc CLKID_MPLL0>, > <&clkc CLKID_MPLL1>; > assigned-clock-parents = <0>, <0>, <0>; > assigned-clock-rates = <294912000>, > <270950400>, > <393216000>; > > * MPLL2: 294912000Hz: 192Khz * 32 * 24 * 2 > used 24, 48, 96, 192Khz > > * MPLL0: 270950400Hz: 176.4Khz * 32 * 24 * 2 > used for 5.512, 10.025, 20.05, 44.1, 88.2 and 176,4Khz > > * MPLL1: 393216000Hz: 64Khz * 32 * 24 * 8 > used for 8, 16, 32 and 64Khz > > Jerome Brunet (2): > clk: meson: stop rate propagation for audio clocks > clk: meson: remove unused clk-audio-divider driver > > drivers/clk/meson/Makefile | 2 +- > drivers/clk/meson/clk-audio-divider.c | 110 ---------------------------------- > drivers/clk/meson/clkc.h | 7 --- > drivers/clk/meson/gxbb.c | 16 +++-- > 4 files changed, 8 insertions(+), 127 deletions(-) > delete mode 100644 drivers/clk/meson/clk-audio-divider.c > Rebased and applied with Neil's ack. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f68.google.com ([74.125.82.68]:52955 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933876AbeFZLTs (ORCPT ); Tue, 26 Jun 2018 07:19:48 -0400 Received: by mail-wm0-f68.google.com with SMTP id p126-v6so1471263wmb.2 for ; Tue, 26 Jun 2018 04:19:48 -0700 (PDT) Message-ID: <1530011985.2900.45.camel@baylibre.com> Subject: Re: [PATCH 0/2] clk: meson: drop clk-audio-divider From: Jerome Brunet To: Neil Armstrong , Kevin Hilman , Carlo Caione Cc: linux-amlogic@lists.infradead.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org Date: Tue, 26 Jun 2018 13:19:45 +0200 In-Reply-To: <20180620100610.29010-1-jbrunet@baylibre.com> References: <20180620100610.29010-1-jbrunet@baylibre.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-clk-owner@vger.kernel.org List-ID: On Wed, 2018-06-20 at 12:06 +0200, Jerome Brunet wrote: > As explained in the commit descriptions, clk-audio-divider was a poor > solution to make sure PLL rates would be high enough while using CCF > rate propagation to determine audio clock rates. > > Stopping rate propagation before the PLLs is actually simpler and provides > better results. It also allow to get rid of clk-audio-divider. > > Off tree sound cards using these clocks should add the following to the > sound card DT to properly setup the PLLs: > > assigned-clocks = <&clkc CLKID_MPLL2>, > <&clkc CLKID_MPLL0>, > <&clkc CLKID_MPLL1>; > assigned-clock-parents = <0>, <0>, <0>; > assigned-clock-rates = <294912000>, > <270950400>, > <393216000>; > > * MPLL2: 294912000Hz: 192Khz * 32 * 24 * 2 > used 24, 48, 96, 192Khz > > * MPLL0: 270950400Hz: 176.4Khz * 32 * 24 * 2 > used for 5.512, 10.025, 20.05, 44.1, 88.2 and 176,4Khz > > * MPLL1: 393216000Hz: 64Khz * 32 * 24 * 8 > used for 8, 16, 32 and 64Khz > > Jerome Brunet (2): > clk: meson: stop rate propagation for audio clocks > clk: meson: remove unused clk-audio-divider driver > > drivers/clk/meson/Makefile | 2 +- > drivers/clk/meson/clk-audio-divider.c | 110 ---------------------------------- > drivers/clk/meson/clkc.h | 7 --- > drivers/clk/meson/gxbb.c | 16 +++-- > 4 files changed, 8 insertions(+), 127 deletions(-) > delete mode 100644 drivers/clk/meson/clk-audio-divider.c > Rebased and applied with Neil's ack.