From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sylwester Nawrocki Subject: Re: [PATCH] ARM: dts: Specify default clocks for Exynos4 FIMC devices Date: Thu, 25 Sep 2014 20:05:46 +0200 Message-ID: <5424597A.7040904@samsung.com> References: <1410367054-30926-1-git-send-email-s.nawrocki@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: Received: from mailout4.w1.samsung.com ([210.118.77.14]:42586 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753234AbaIYSGD (ORCPT ); Thu, 25 Sep 2014 14:06:03 -0400 Received: from eucpsbgm2.samsung.com (unknown [203.254.199.245]) by mailout4.w1.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0NCG0063CYENRQ10@mailout4.w1.samsung.com> for linux-samsung-soc@vger.kernel.org; Thu, 25 Sep 2014 19:08:47 +0100 (BST) In-reply-to: Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Daniel Drake Cc: Kukjin Kim , linux-arm-kernel@lists.infradead.org, linux-samsung-soc , Marek Szyprowski , Jacek Anaszewski , Kamil Debski Hi Daniel, On 18/09/14 21:27, Daniel Drake wrote: > On Wed, Sep 10, 2014 at 10:37 AM, Sylwester Nawrocki > wrote: >> > The default mux and divider clocks are specified in device tree >> > so that the FIMC devices in Exynos4210 and Exynos4x12 SoCs are >> > clocked from recommended clock source and with maximum supported >> > frequency. If needed these settings could be overrode in board >> > specific dts files, however they are in practice optimal in most >> > cases. > > Just curious about the Exynos4x12 situation here. > You set the FIMC clocks as 176MHz, child of MPLL, which works for > ODROID with a divider: > > 880MHz MPLL / 5 = 176MHz > > However, talking of recommended frequencies... Is 880MHz really the > standard there? > Isn't 800MHz the more common one? AFAIK 880 MHz is recommended MPLL frequency for Exynos4412 EVT2.0, which is revision of the Exynos4412 SoC the Odroid U3 boards are populated with. You can read the main/sub revision information from the chip ID register (at 0x10000000). The frequencies can always be overwritten in board specific dts or DTB could be amended by bootloader if needed. > Also, if you happen to know, I would be curious about the equivalent > and recommended situation for the sclk_mfc clock. On the vendor kernel > it is clocked at 880/4 = 220MHz. When booting mainline on an odroid it > is 880/16 = 55MHz :/ I think we should add similar entry in device tree for the MFC devices. AFAIR now the frequency has fixed value in the driver. I saw some changes in s5p-mfc driver WRT to clock handling recently though, possibly Jacek or Kamil could explain what current situation is. -- Regards, Sylwester From mboxrd@z Thu Jan 1 00:00:00 1970 From: s.nawrocki@samsung.com (Sylwester Nawrocki) Date: Thu, 25 Sep 2014 20:05:46 +0200 Subject: [PATCH] ARM: dts: Specify default clocks for Exynos4 FIMC devices In-Reply-To: References: <1410367054-30926-1-git-send-email-s.nawrocki@samsung.com> Message-ID: <5424597A.7040904@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Daniel, On 18/09/14 21:27, Daniel Drake wrote: > On Wed, Sep 10, 2014 at 10:37 AM, Sylwester Nawrocki > wrote: >> > The default mux and divider clocks are specified in device tree >> > so that the FIMC devices in Exynos4210 and Exynos4x12 SoCs are >> > clocked from recommended clock source and with maximum supported >> > frequency. If needed these settings could be overrode in board >> > specific dts files, however they are in practice optimal in most >> > cases. > > Just curious about the Exynos4x12 situation here. > You set the FIMC clocks as 176MHz, child of MPLL, which works for > ODROID with a divider: > > 880MHz MPLL / 5 = 176MHz > > However, talking of recommended frequencies... Is 880MHz really the > standard there? > Isn't 800MHz the more common one? AFAIK 880 MHz is recommended MPLL frequency for Exynos4412 EVT2.0, which is revision of the Exynos4412 SoC the Odroid U3 boards are populated with. You can read the main/sub revision information from the chip ID register (at 0x10000000). The frequencies can always be overwritten in board specific dts or DTB could be amended by bootloader if needed. > Also, if you happen to know, I would be curious about the equivalent > and recommended situation for the sclk_mfc clock. On the vendor kernel > it is clocked at 880/4 = 220MHz. When booting mainline on an odroid it > is 880/16 = 55MHz :/ I think we should add similar entry in device tree for the MFC devices. AFAIR now the frequency has fixed value in the driver. I saw some changes in s5p-mfc driver WRT to clock handling recently though, possibly Jacek or Kamil could explain what current situation is. -- Regards, Sylwester