[OpenWrt-Devel] [PATCH v2] ath79: ag71xx: apply interface mode to MII0/1_CTRL on ar71xx/ar913x

Mathias Kresin dev at kresin.me
Sun Aug 19 04:26:39 EDT 2018


19.08.2018 09:02, Chuanhong Guo:
> Signed-off-by: Chuanhong Guo <gch981213 at gmail.com>

Please add a commit message. I'm sure you explained the issue and your 
fix somewhere. Not sure if I saw it in the forum or somewhere on github.

Good commit messages serve at least three important purposes:

  - To speed up the reviewing process.
  - To help us write a good release note.
  - To help the future maintainers, say five years into the future, to
    find out why a particular change was made to the code or why a
    specific feature was added.

A good commit message should answer three questions about a patch:

  - Why is it necessary? It may fix a bug, it may add a feature, it may
    improve performance, reliabilty, stability, or just be a change for
    the sake of correctness.
  - How does it address the issue? For short obvious patches this part
    can be omitted, but it should be a high level description of what the
    approach was.
  - What effects does the patch have? (In addition to the obvious ones,
    this may include benchmarks, side effects, etc.)

Mathias

_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel



More information about the openwrt-devel mailing list