diff mbox

[1/1] openfirmware: trivial: Standardise formatting of 'struct of_device_id' definition

Message ID 1390489239-29001-1-git-send-email-lee.jones@linaro.org
State Accepted
Commit 851c63e3b381fdbf5aca1a797f37d8606b5588d2
Headers show

Commit Message

Lee Jones Jan. 23, 2014, 3 p.m. UTC 
It's commonly accepted that when defining a struct the '{' is appended
onto the first line. This helps when searching for the definition using
syntax similar to 'struct of_device_id {'.

Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
 include/linux/mod_devicetable.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Lee Jones Feb. 10, 2014, 2:40 p.m. UTC | #1 
> It's commonly accepted that when defining a struct the '{' is appended
> onto the first line. This helps when searching for the definition using
> syntax similar to 'struct of_device_id {'.
> 
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
>  include/linux/mod_devicetable.h | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/include/linux/mod_devicetable.h b/include/linux/mod_devicetable.h
> index 45e9214..090e014 100644
> --- a/include/linux/mod_devicetable.h
> +++ b/include/linux/mod_devicetable.h
> @@ -220,8 +220,7 @@ struct serio_device_id {
>  /*
>   * Struct used for matching a device
>   */
> -struct of_device_id
> -{
> +struct of_device_id {
>  	char	name[32];
>  	char	type[32];
>  	char	compatible[128];

Is anyone interested in this patch?

Or should is it okay to just take it through my tree?
diff mbox

Patch

diff --git a/include/linux/mod_devicetable.h b/include/linux/mod_devicetable.h
index 45e9214..090e014 100644
--- a/include/linux/mod_devicetable.h
+++ b/include/linux/mod_devicetable.h
@@ -220,8 +220,7 @@  struct serio_device_id {
 /*
  * Struct used for matching a device
  */
-struct of_device_id
-{
+struct of_device_id {
 	char	name[32];
 	char	type[32];
 	char	compatible[128];