From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joonyoung Shim Date: Tue, 13 Jan 2015 13:30:14 +0900 Subject: [U-Boot] [PATCH] ARM: exynos: clock: remove clock_get_periph_rate() In-Reply-To: References: <1420702439-3796-1-git-send-email-jh80.chung@samsung.com> Message-ID: <54B49F56.8040202@samsung.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, On 01/13/2015 01:16 PM, Simon Glass wrote: > Hi, > > On 7 January 2015 at 23:33, Jaehoon Chung wrote: >> This api is wrong array bounds. >> >> arch/arm/cpu/armv7/exynos/clock.c: In function 'clock_get_periph_rate': >> arch/arm/cpu/armv7/exynos/clock.c:265:47: warning: array subscript is above array bounds [-Warray-bounds] >> struct clk_bit_info *bit_info = &clk_bit_info[peripheral]; >> >> And it doesn't use anywhere. It only used to get pwm clock. >> So it changes from clock_get_periph_rate() to get_pwm_clock. >> >> Signed-off-by: Jaehoon Chung >> --- >> arch/arm/cpu/armv7/exynos/clock.c | 173 +++++---------------------------- >> arch/arm/include/asm/arch-exynos/clk.h | 9 -- >> 2 files changed, 25 insertions(+), 157 deletions(-) > > As discussed on the other patch, can we fix this and use it, rather > than removing it? Lots of little functions for each n peripherals and > each m SoCs = n * m functions and a primitive API. > > This was a request made as part of the Snow and Pit developments, but > it didn't get very far. Can we finish it? > Who does follow up it? It's best solution if it's finished soon but if it takes long time, i think it is better to remove now. Thanks.