From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: MIME-version: 1.0 Content-type: text/plain; charset="UTF-8" Message-id: <5A820B19.8080902@samsung.com> Date: Tue, 13 Feb 2018 06:46:01 +0900 From: Chanwoo Choi To: Sylwester Nawrocki , linux-clk@vger.kernel.org Cc: sboyd@codeaurora.org, mturquette@baylibre.com, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, krzk@kernel.org, b.zolnierkie@samsung.com, m.szyprowski@samsung.com Subject: Re: [PATCH v2] clk: exynos5433: Extend list of available AUD_PLL output frequencies In-reply-to: <20180212155227.20605-1-s.nawrocki@samsung.com> References: <20180212155227.20605-1-s.nawrocki@samsung.com> List-ID: Hi, On 2018년 02월 13일 00:52, Sylwester Nawrocki wrote: > Add one more entry to the exynos5433_aud_pll_rates table, this allows > to support audio sample rates: 48000, 96000, 192000 Hz with minimum > error. The M, P, S, K values re confirmed by the HW team. > > Signed-off-by: Sylwester Nawrocki > --- > drivers/clk/samsung/clk-exynos5433.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/clk/samsung/clk-exynos5433.c b/drivers/clk/samsung/clk-exynos5433.c > index ebd18586e325..b08a9028653f 100644 > --- a/drivers/clk/samsung/clk-exynos5433.c > +++ b/drivers/clk/samsung/clk-exynos5433.c > @@ -765,6 +765,7 @@ static const struct samsung_pll_rate_table exynos5433_aud_pll_rates[] __initcons > PLL_36XX_RATE(294912000U, 98, 1, 3, 19923), > PLL_36XX_RATE(288000000U, 96, 1, 3, 0), > PLL_36XX_RATE(252000000U, 84, 1, 3, 0), > + PLL_36XX_RATE(196608001U, 197, 3, 3, -25690), > { /* sentinel */ } > }; Looks good to me. Acked-by: Chanwoo Choi -- Best Regards, Chanwoo Choi Samsung Electronics