Re: [PATCH v3 0/9] ARM: dts: exynos: add support for hdmi subsystem

[Kukjin Kim <kgene.kim@samsung.com>]

On 08/14/13 20:26, Tomasz Figa wrote:

[...]

>> See my comments about making common stuff for exynos5 SoCs.
>
> I agree.
>
>> And please don't use title like following,
>>
>> 'ARM: dts: exynos0000: ....'.
>>
>> Because the relatively long prefix of title can cover _real_ title, So
>> just 'ARM: dts: .... in exynos000' is better.
>
> Well, I tend to disagree on this. The ARM: dts:<platform name>: prefix is
> commonly used in arch/arm/boot/dts (see git log --oneline
> arch/arm/boot/dts).
>
Hmm...it is not entirely true...

> As long as we don't cross the ~75 characters boundary in patch subject, I
> don't think this is any problem.
>
Let's see.

If we use the 'platform name' in prefixe for dt file,

"ARM: dts: exynos0000: add support for something"
or
"ARM: dts: universal_c210: add support for something"
or
"ARM: dts: exynos0000-pinctrl: add support for something"
...
most of them should spend over half of 75 characters as a prefix 
including 'add support for' in subject and in addition, there is '[PATCH 
v0 00/00]' in the email subject.

(+ Russell)

I remember there were discussions about subject and at that time the 
result was that keyword should be shown ahead as possible in subject. 
But we are still using too long word as a prefix...

One more note, even my e-mail client shows just 40~50 characters for 
subject because of relatively not big enough resolution and pre-read 
window and so on ;)

Kukjin