최규호 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 <kmp...@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....@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
