Merge branch 'master' into usbh_construct

1 files changed, 133 insertions, 58 deletions

diff --git a/documentation/coding_style_cn.md b/documentation/coding_style_cn.md
index 9c4019d647..96a4cf8383 100644
--- a/documentation/coding_style_cn.md
+++ b/documentation/coding_style_cn.md
@@ -1,25 +1,23 @@
 # RT-Thread 编程风格
 
-这是一份 RT-Thread 开发人员的开发指引。RT-Thread 做为一份开源软件，它需要由不同
-的人采用合作的方式完成，这份文档是开发人员的一个指引。RT-Thread 的开发人员请遵
-守这样的编程风格。同时对于使用 RT-Thread 的用户，也可通过这份文档了解 RT-Thread
-代码内部一些约定从而比较容易的把握到 RT-Thread 的实现方式。
+这是一份 RT-Thread 开发人员的开发指引。RT-Thread 做为一份开源软件，它需要由不同的人采用合作的方式完成，这份文档是开发人员的一个指引。RT-Thread 的开发人员请遵守这样的编程风格。同时对于使用 RT-Thread 的用户，也可通过这份文档了解 RT-Thread代码内部一些约定从而比较容易的把握到 RT-Thread 的实现方式。
 
 ## 1.目录名称
 
-目录名称如果无特殊的需求，请使用全小写的形式；目录名称应能够反映部分的意思，例
-如各芯片移植由其芯片名称构成或芯片类别构成；components 目录下能够反映组件的意义。
+目录名称如果无特殊的需求，请使用全小写的形式；目录名称应能够反映部分的意思，例如各芯片移植由其芯片名称构成或芯片类别构成；components 目录下能够反映组件的意义。
 
 ## 2.文件名称
 
-文件名称如果无特殊的需求(如果是引用其他地方，可以保留相应的名称)，请使用全小写
-的形式。另外为了避免文件名重名的问题，一些地方请尽量不要使用通用化、使用频率高
-的名称。
+文件名称如果无特殊的需求(如果是引用其他地方，可以保留相应的名称)，请使用全小写的形式。另外为了避免文件名重名的问题，一些地方请尽量不要使用通用化、使用频率高的名称。
+
+设备驱动源码文件：`drv_class.c` 的命名方式，如：
+
+- drv_spi.c
+- drv_gpio.c
 
 ## 3.头文件定义
 
-C语言头文件为了避免多次重复包含，需要定义一个符号。这个符号的定义形式请采用如下
-的风格：
+C 语言头文件为了避免多次重复包含，需要定义一个符号。这个符号的定义形式请采用如下的风格：
 
 ```c
     #ifndef __FILE_H__
@@ -28,8 +26,7 @@ C语言头文件为了避免多次重复包含，需要定义一个符号。这�
     #endif
 ```
 
-即定义的符号两侧采用 "__" 以避免重名，另外也可以根据文件名中是否包含多个词语而
-采用 "_" 连接起来。
+即定义的符号两侧采用 "__" 以避免重名，另外也可以根据文件名中是否包含多个词语而采用 "_" 连接起来。
 
 ## 4.文件头注释
 
@@ -62,7 +59,7 @@ C语言头文件为了避免多次重复包含，需要定义一个符号。这�
     };
 ```
 
-其中，"{"，"}" 独立占用一行，后面的成员定义使用缩进的方式定义。
+其中，**"{"，"}" 独立占用一行**，后面的成员定义使用缩进的方式定义。
 
 结构体等的类型定义请以结构体名称加上 "_t" 的形式作为名称，例如：
 
@@ -86,28 +83,103 @@ C语言头文件为了避免多次重复包含，需要定义一个符号。这�
 
 ## 7.函数名称、声明
 
-函数名称请使用小写英文的形式，单词之间使用 "_" 连接。提供给上层应用使用的 API
-接口，必须在相应的头文件中声明；如果函数入口参数是空，必须使用 void 作为入口参
-数，例如：
+函数名称请使用小写英文的形式，单词之间使用 "_" 连接。提供给上层应用使用的 API接口，必须在相应的头文件中声明；如果函数入口参数是空，必须使用 void 作为入口参数，例如：
+
+```c
+rt_thread_t rt_thread_self(void);
+```
+
+内部静态函数命名：以下划线开头，使用 `_class_method` 格式，不携带`_rt_`开头，如内核或驱动文件中的函数命名：
+
+```c
+/* IPC object init */
+static rt_err_t _ipc_object_init()
+
+/* UART driver ops */
+static rt_err_t _uart_configure()
+static rt_err_t _uart_control()                    
+```
+
+调用注册设备接口的函数命名：使用 `rt_hw_class_init()` 格式，举例：
 
 ```c
-    rt_thread_t rt_thread_self(void);
+int rt_hw_uart_init(void)
+int rt_hw_spi_init(void)
 ```
 
 ## 8.注释编写
 
-请使用英文做为注释，使用中文注释将意味着在编写代码时需要来回不停的切换中英文输
-入法从而打断编写代码的思路。并且使用英文注释也能够比较好的与中国以外的技术者进
-行交流。
+请使用**英文**做为注释，使用中文注释将意味着在编写代码时需要来回不停的切换中英文输入法从而打断编写代码的思路。并且使用英文注释也能够比较好的与中国以外的技术者进行交流。
 
-源代码的注释不应该过多，更多的说明应该是代码做了什么，仅当个别关键点才需要一些
-相应提示性的注释以解释一段复杂的算法它是如何工作的。对语句的注释只能写在它的上
-方或右方，其他位置都是非法的。
+**语句注释**：
+
+源代码的注释不应该过多，更多的说明应该是代码做了什么，仅当个别关键点才需要一些相应提示性的注释以解释一段复杂的算法它是如何工作的。对语句的注释只能写在它的**上方或右方**，其他位置都是非法的。
+
+```c
+/* 你的英文注释 */
+```
+
+**函数注释**：
+
+注释以 `/**` 开头，以 `  */` 结尾，中间写入函数注释，组成元素如下，每个元素描述之间空一行，且首列对齐：
+
+- @brief + 简述函数作用。在描述中，着重说明该函数的作用，每句话首字母大写，句尾加英文句号。
+- @note + 函数说明。在上述简述中未能体现到的函数功能或作用的一些点，可以做解释说明，每句话首字母大写，句尾加英文句号。
+- @see + 相关 API 罗列。若有与当前函数相关度较高的 API，可以进行列举。
+- @param + 以参数为主语 + be 动词 + 描述，说明参数的意义或来源。
+- @return + 枚举返回值 + 返回值的意思，若返回值为数据，则直接介绍数据的功能。
+- @warning + 函数使用注意要点。在函数使用时，描述需要注意的事项，如使用环境、使用方式等。每句话首字母大写，句尾加英文句号。
+
+注释模版请参见：rt-thread/src/ipc.c 源码文件，英文注释请参考使用 grammarly 以及谷歌翻译。
+
+```C
+/**
+ * @brief    The function will initialize a static event object.
+ *
+ * @note     For the static event object, its memory space is allocated by the compiler during compiling,
+ *           and shall placed on the read-write data segment or on the uninitialized data segment.
+ *           By contrast, the rt_event_create() function will allocate memory space automatically
+ *           and initialize the event.
+ *
+ * @see      rt_event_create()
+ *
+ * @param    event is a pointer to the event to initialize. It is assumed that storage for the event
+ *           will be allocated in your application.
+ *
+ * @param    name is a pointer to the name that given to the event.
+ *
+ * @param    value is the initial value for the event.
+ *           If want to share resources, you should initialize the value as the number of available resources.
+ *           If want to signal the occurrence of an event, you should initialize the value as 0.
+ *
+ * @param    flag is the event flag, which determines the queuing way of how multiple threads wait
+ *           when the event is not available.
+ *           The event flag can be ONE of the following values:
+ *
+ *               RT_IPC_FLAG_PRIO          The pending threads will queue in order of priority.
+ *
+ *               RT_IPC_FLAG_FIFO          The pending threads will queue in the first-in-first-out method
+ *                                         (also known as first-come-first-served (FCFS) scheduling strategy).
+ *
+ *               NOTE: RT_IPC_FLAG_FIFO is a non-real-time scheduling mode. It is strongly recommended to
+ *               use RT_IPC_FLAG_PRIO to ensure the thread is real-time UNLESS your applications concern about
+ *               the first-in-first-out principle, and you clearly understand that all threads involved in
+ *               this event will become non-real-time threads.
+ *
+ * @return   Return the operation status. When the return value is RT_EOK, the initialization is successful.
+ *           If the return value is any other values, it represents the initialization failed.
+ *
+ * @warning  This function can ONLY be called from threads.
+ */
+rt_err_t rt_event_init(rt_event_t event, const char *name, rt_uint8_t flag)
+{
+   ... 
+}
+```
 
 ## 9.缩进及分行
 
-缩进请采用 4 个空格的方式。如果没有什么特殊意义，请在 "{" 后进行分行，并在下一
-行都采用缩进的方式，例如：
+缩进请采用 4 个空格的方式。如果没有什么特殊意义，请在 "{" 后进行分行，并在下一行都采用缩进的方式，例如：
 
 ```c
     if (condition)
@@ -116,8 +188,7 @@ C语言头文件为了避免多次重复包含，需要定义一个符号。这�
     }
 ```
 
-唯一的例外是 switch 语句，switch-case 语句采用 case 语句与 switch 对齐的方式，
-例如：
+唯一的例外是 switch 语句，switch-case 语句采用 case 语句与 switch 对齐的方式，例如：
 
 ```c
     switch (value)
@@ -127,9 +198,7 @@ C语言头文件为了避免多次重复包含，需要定义一个符号。这�
     }
 ```
 
-case 语句与前面的 switch 语句对齐，后续的语句则采用缩进的方式。
-
-分行上，如果没有什么特殊考虑，请**不要在代码中连续使用两个以上的空行**。
+case 语句与前面的 switch 语句对齐，后续的语句则采用缩进的方式。分行上，如果没有什么特殊考虑，请**不要在代码中连续使用两个以上的空行**。
 
 ## 10.大括号与空格
 
@@ -142,9 +211,7 @@ case 语句与前面的 switch 语句对齐，后续的语句则采用缩进的�
     }
 ```
 
-匹配的大括号单独占用一行，代码阅读起来就会有相应的层次而不会容易出现混淆的情况。
-
-空格建议在非函数方式的括号调用前留一个空格以和前面的进行区分，例如：
+匹配的大括号单独占用一行，代码阅读起来就会有相应的层次而不会容易出现混淆的情况。空格建议在非函数方式的括号调用前留一个空格以和前面的进行区分，例如：
 
 ```c
     if (x <= y)
@@ -158,8 +225,7 @@ case 语句与前面的 switch 语句对齐，后续的语句则采用缩进的�
     }
 ```
 
-建议在括号前留出一个空格(涉及的包括 if、for、while、switch 语句)，而运算表达式
-中，运算符与字符串间留一个空格。另外，不要在括号的表达式两侧留空格，例如：
+建议在括号前留出一个空格(涉及的包括 if、for、while、switch 语句)，而运算表达式中，运算符与字符串间留一个空格。另外，不要在括号的表达式两侧留空格，例如：
 
 ```c
     if ( x <= y )
@@ -172,26 +238,19 @@ case 语句与前面的 switch 语句对齐，后续的语句则采用缩进的�
 
 ## 11.trace、log信息
 
-在 RT-Thread 中，普遍使用的 log 方式是 rt_kprintf。rt_kprintf 在 RT-Thread 被实
-现成一个采用轮询、非中断方式的字串输出，能够适合于在中断这类"即时"显示日志的场
-合。因为这种轮询方式的存在，也必然会影响到日志输出的时序关系。
+在 RT-Thread 中，普遍使用的 log 方式是 rt_kprintf。rt_kprintf 在 RT-Thread 被实现成一个采用轮询、非中断方式的字串输出，能够适合于在中断这类"即时"显示日志的场合。因为这种轮询方式的存在，也必然会影响到日志输出的时序关系。
 
-建议在代码中不要频繁的使用 rt_kprintf 作为日志输出，除非你真正的明白，你的代码
-运行占用的时间多一些也没什么关系。
+建议在代码中不要频繁的使用 rt_kprintf 作为日志输出，除非你真正的明白，你的代码运行占用的时间多一些也没什么关系。
 
-日志输出应该被设计成正常情况下是关闭状态(例如通过一个变量或宏就能够开启)，并且
-当真正输出日志时，日志是易懂易定位问题的方式。"天书式"的日志系统是糟糕的，不合
-理的。
+日志输出应该被设计成正常情况下是关闭状态(例如通过一个变量或宏就能够开启)，并且当真正输出日志时，日志是易懂易定位问题的方式。"天书式"的日志系统是糟糕的，不合理的。
 
 ## 12.函数
 
-在内核编程中，函数应该尽量精简，仅完成相对独立的简单功能。函数的实现不应该太长
-，函数实现太长，应该反思能够如何修改(或拆分)使得函数更为精简、易懂。
+在内核编程中，函数应该尽量精简，仅完成相对独立的简单功能。函数的实现不应该太长，函数实现太长，应该反思能够如何修改(或拆分)使得函数更为精简、易懂。
 
 ## 13.对象
 
-RT-Thread 内核采用了 C 语言对象化技术，命名表现形式是：对象名结构体表示类定义、
-对象名 + 动词短语形式表示类方法，例如：
+RT-Thread 内核采用了 C 语言对象化技术，命名表现形式是：对象名结构体表示类定义、对象名 + 动词短语形式表示类方法，例如：
 
 ```c
     struct rt_timer
@@ -205,22 +264,28 @@ RT-Thread 内核采用了 C 语言对象化技术，命名表现形式是：对�
 结构体定义 rt_timer 代表了 timer 对象的类定义；
 
 ```c
-    rt_timer_t rt_timer_create(const char* name,
-        void (*timeout)(void* parameter), void* parameter,
-        rt_tick_t time, rt_uint8_t flag);
-    rt_err_t rt_timer_delete(rt_timer_t timer);
-    rt_err_t rt_timer_start(rt_timer_t timer);
-    rt_err_t rt_timer_stop(rt_timer_t timer);
+rt_timer_t rt_timer_create(const char* name,
+                           void (*timeout)(void* parameter), 
+                           void* parameter,
+                           rt_tick_t time, rt_uint8_t flag);
+rt_err_t rt_timer_delete(rt_timer_t timer);
+rt_err_t rt_timer_start(rt_timer_t timer);
+rt_err_t rt_timer_stop(rt_timer_t timer);
 ```
 
 rt_timer + 动词短语的形式表示能够应用于 timer 对象的方法。
 
-在创建一个新的对象时，应该思考好，对象的内存操作处理：是否允许一个静态对象存在
-，或仅仅支持从堆中动态分配的对象。
+在创建一个新的对象时，应该思考好，对象的内存操作处理：是否允许一个静态对象存在，或仅仅支持从堆中动态分配的对象。
+
+## 14.格式化代码
+
+格式化代码是指通过脚本自动整理你的代码，并使其符合 RT-Thread 的编码规范。本文提供以下两种自动格式化代码方法，可以自行选择或配合使用。
 
-## 14. 用 astyle 自动格式化代码
+### 使用 astyle 格式化
 
-    参数：--style=allman
+用 astyle 自动格式化代码，参数如下：
+
+          --style=allman
           --indent=spaces=4
           --indent-preproc-block
           --pad-oper
@@ -231,3 +296,13 @@ rt_timer + 动词短语的形式表示能够应用于 timer 对象的方法。
           --lineend=linux
           --convert-tabs
           --verbose
+
+能满足函数空格、缩进、函数语句等的规范。
+
+### 使用 formatting 格式化
+
+使用 [formatting](https://github.com/mysterywolf/formatting) 扫描文件来格式化代码：formatting 可以满足编码规则的基本要求，如：
+
+- 将源文件编码统一为 UTF-8
+- 将 TAB 键替换为 4 空格
+- 将每行末尾多余的空格删除，并统一换行符为 '\n'