[PATCH 1/3] This patch adds support HR-Timer(High Resolution Timer) and dynamic tick system for S5P SoCs. There are many clock sources for HR-Timer on S5P SoCs. The PWM timer, RTC, System Timer, and MCT can be used for clock source. This patch can only support PWM timer for clocksource of S5P64x0 and S5PV210.

Russell King - ARM Linux linux at arm.linux.org.uk
Tue Mar 8 04:49:55 EST 2011


> Subject: [PATCH 1/3] This patch adds support HR-Timer(High Resolution Timer)
        and dynamic tick system for S5P SoCs. There are many clock
        sources for HR-Timer on S5P SoCs. The PWM timer, RTC, System Timer,
        and MCT can be used for clock source. This patch can only support PWM
        timer for clocksource of S5P64x0 and S5PV210.

Please don't fill the subject line with the entire patch description.
Subject lines are always supposed to be a brief summary of the message
being sent, or with patches a brief summary of what the patch is about.

Put the rest of the patch description in the body of the message.

Don't say things like "This patch adds", "Adding support for", etc.  When
the patch has been merged, years later it doesn't make sense to read such
patch descriptions.

"S5P: Common HRT and NO_HZ support" is sufficient to summarise the patch.

Note that some mail clients only show you the first 40 or so characters of
the subject line in their message index.  To illustrate:

6472 N   Mar 07 Sangbeom Kim    (  19) [PATCH V3 0/3] ARM: S5P: Add HRT support
6473     Mar 07 Sangbeom Kim    ( 530) ├─>[PATCH 1/3] This patch adds support HR
6474 N   Mar 07 Sangbeom Kim    ( 208) ├─>[PATCH 2/3] This patch modify s5pv210_
6475 N   Mar 07 Sangbeom Kim    ( 112) └─>[PATCH 3/3] This patch update mach-s5p

So to have half the subject space wasted by "This patch modify" is not
very productive.



More information about the linux-arm-kernel mailing list