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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux