[PATCH] ARM: S5PV210: Add Torbreck board support

Kukjin Kim kgene.kim at samsung.com
Tue Sep 28 01:55:15 EDT 2010


최규호 wrote:
Hi, 

Welcome to Linux mainline ;-)
I have some comments about your patches.

Firstly could you please use English character in the representing name in e-mail client not Korean character :-)
And make sure it's text type.

>Hi,
>Thank you for your interesting.
>On Mon, Sep 27, 2010 at 11:58 AM, Kyungmin Park <kmpark at infradead.org> wrote:

(snip)

>> +#define TORBRECK_UFCON_DEFAULT (S3C2410_UFCON_FIFOMODE |       \
>> +                                S5PV210_UFCON_TXTRIG4 |        \
>> +                                S5PV210_UFCON_RXTRIG4)
>Any reason to use TRIG4? just use the full trigger e.g., 256.
> 
>Okay, I'll fix it.
> 
Hmm...Kyungmin, any reason to use full trigger here?

It depends on board...so it doesn't matter TRIG4 or anything else if there is no problem on your board.
It means the maximum value is not best condition...only depends on your situation/condition.

>> +
>> +static struct s3c2410_uartcfg torbreck_uartcfgs[] __initdata = {
>> +       [0] = {
>> +               .hwport         = 0,
>> +               .flags          = 0,
>There's no code for flags, please remove it all.
> 
>Okay, I'll remove it.
> 
I think no need to modify it.
Actually I said many times about this...

And as Ben Dooks said in other patch, the format will be changed soon.

>> +               .ucon           = TORBRECK_UCON_DEFAULT,
>> +               .ulcon          = TORBRECK_ULCON_DEFAULT,
>> +               .ufcon          = TORBRECK_UFCON_DEFAULT,
>> +       },
>> +       [1] = {
>> +               .hwport         = 1,
>> +               .flags          = 0,
>> +               .ucon           = TORBRECK_UCON_DEFAULT,
>> +               .ulcon          = TORBRECK_ULCON_DEFAULT,
>> +               .ufcon          = TORBRECK_UFCON_DEFAULT,
>> +       },
>> +       [2] = {
>> +               .hwport         = 2,
>> +               .flags          = 0,
>> +               .ucon           = TORBRECK_UCON_DEFAULT,
>> +               .ulcon          = TORBRECK_ULCON_DEFAULT,
>> +               .ufcon          = TORBRECK_UFCON_DEFAULT,
>> +       },
>> +       [3] = {
>> +               .hwport         = 3,
>> +               .flags          = 0,
>> +               .ucon           = TORBRECK_UCON_DEFAULT,
>> +               .ulcon          = TORBRECK_ULCON_DEFAULT,
>> +               .ufcon          = TORBRECK_UFCON_DEFAULT,
>> +       },
>> +};
>> +

(snip)

>> --
>> 1.5.6.3
>>
If possible, please use later version git.
It doesn't mean latest git is best...
This is just private opinion. :-)

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.




More information about the linux-arm-kernel mailing list