[U-Boot] [PATCH 2/6 V5] EXYNOS: Add clock for SPI.

Minkyu Kang promsoft at gmail.com
Wed Oct 17 09:12:45 CEST 2012


Dear Hatim,

On 17 October 2012 15:26, Hatim Rv <rv.hatimali at gmail.com> wrote:
> Dear Minkyu Kang,
>
> On Wed, Oct 17, 2012 at 10:55 AM, Minkyu Kang <promsoft at gmail.com> wrote:
>> Dear Hatim,
>>
>> On 17 October 2012 14:21, Hatim Rv <rv.hatimali at gmail.com> wrote:
>>>>> diff --git a/arch/arm/include/asm/arch-exynos/clk.h b/arch/arm/include/asm/arch-exynos/clk.h
>>>>> index 5529025..4e51402 100644
>>>>> --- a/arch/arm/include/asm/arch-exynos/clk.h
>>>>> +++ b/arch/arm/include/asm/arch-exynos/clk.h
>>>>> @@ -22,6 +22,8 @@
>>>>>  #ifndef __ASM_ARM_ARCH_CLK_H_
>>>>>  #define __ASM_ARM_ARCH_CLK_H_
>>>>>
>>>>> +#include <asm/arch/pinmux.h>
>>>>
>>>> no. we don't need this header file.
>>>>
>>>
>>> Actually we do need the asm/arch/periph.h file as we are using the
>>> 'enum periph_id'. Otherwise it will give compilation errors.
>>
>> So, I commented.
>>
>> "please modify to exynos5_set_spi_clk(int periph_id"
>>
>> we don't have to use enum at parameter.
>
> Even if we make the enum as int in the clk.h we still need the enum
> definition in clock.c.

then, please include that file in clock.c

> Besides, making the enum as int in clk.h does not look good as it will
> make the code confusing (by passing  enum as  int and using int as
> enum).

No, it's not confusing.
enum is not a specific data type.
Please look pinmux.c we get paramter as int.

> So we require the periph.h declaration either in clk.h or clock.c file.
> If there is any better way to proceed, please let me know.

Actually, I want what you do not use periph_id at clock.c.
Why clock get dependency with pinmux?

Thanks.
Minkyu Kang.
-- 
from. prom.
www.promsoft.net


More information about the U-Boot mailing list