Thread (17 messages) 17 messages, 2 authors, 2015-07-20

[PATCH v2 1/7] pinctrl: UniPhier: add UniPhier pinctrl core support

From: Masahiro Yamada <hidden>
Date: 2015-07-19 04:19:54
Also in: linux-gpio, lkml

Hi Linus,



2015-07-16 16:20 GMT+09:00 Linus Walleij [off-list ref]:
On Tue, Jul 14, 2015 at 4:40 AM, Masahiro Yamada
[off-list ref] wrote:
quoted
The core support for the pinctrl drivers for all the UniPhier SoCs.

Signed-off-by: Masahiro Yamada <redacted>
---

Changes in v2:
  - drop vogus THIS_MODULE because this file is always built-in
  - drop vogus "include <linux/module.h> because this file is
    always built-in
This looks nice and uses all generic facilities we have.

Patch applied.

Yours,
Linus Walleij


I saw the applied patch and noticed "Changes in v2"
was moved to git-description.


commit 6e908892025885b07e804dc6c05aab6ce1e06832
Author: Masahiro Yamada [off-list ref]
Date:   Tue Jul 14 11:40:01 2015 +0900

    pinctrl: UniPhier: add UniPhier pinctrl core support

    The core support for the pinctrl drivers for all the UniPhier SoCs.

    Changes in v2:
      - drop vogus THIS_MODULE because this file is always built-in
      - drop vogus "include <linux/module.h> because this file is
        always built-in

    Signed-off-by: Masahiro Yamada [off-list ref]
    Signed-off-by: Linus Walleij [off-list ref]




Do we need to record such stuff in the git history?

In my opinion, the difference between patch versions
should be mentioned to help reviewers, but not included in the git repository.

Early versions are often so immature that they must be improved
in the review process.
No point to record what was updated from those versions, I think.

This is why I put "changed in v2" below the "---".



-- 
Best Regards
Masahiro Yamada
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help