Missing subsystem in subject line. Please use the same git brief
description format that has been used previously for files you want to
patch. You can view previous commits for a file using

        git log --pretty=oneline --abbrev --reverse

On Mon, Nov 20, 2017 at 03:14:21PM -0600, zebmccor...@disr.it wrote:
> From: Zebulon McCorkle <zebmccor...@disr.it>
> 
> The olpc_dcon driver had some slight style issues, mostly pertaining to
> indentation in function calls and definitions. I've solved those, and
> plan to work on the issues in the TODO.

Please read Documentation/process/submitting-patches.rst for tips on how to
write git commit log messages. Especially section '2) Describe your changes'.

> Signed-off-by: Zebulon McCorkle <zebmccor...@disr.it>
> ---
>  drivers/staging/olpc_dcon/olpc_dcon.c      | 30 
> ++++++++++++++++++++----------
>  drivers/staging/olpc_dcon/olpc_dcon.h      | 30 
> +++++++++++++++---------------
>  drivers/staging/olpc_dcon/olpc_dcon_xo_1.c |  2 +-
>  3 files changed, 36 insertions(+), 26 deletions(-)
> 
> diff --git a/drivers/staging/olpc_dcon/olpc_dcon.c 
> b/drivers/staging/olpc_dcon/olpc_dcon.c
> index 82bffd911435..2744c9f0920e 100644
> --- a/drivers/staging/olpc_dcon/olpc_dcon.c
> +++ b/drivers/staging/olpc_dcon/olpc_dcon.c
> @@ -393,7 +393,8 @@ static void dcon_set_source_sync(struct dcon_priv *dcon, 
> int arg)
>  }
>  
>  static ssize_t dcon_mode_show(struct device *dev,
> -     struct device_attribute *attr, char *buf)
> +                           struct device_attribute *attr,
> +                           char *buf)
>  {
>       struct dcon_priv *dcon = dev_get_drvdata(dev);
>  
> @@ -401,7 +402,8 @@ static ssize_t dcon_mode_show(struct device *dev,
>  }
>  
>  static ssize_t dcon_sleep_show(struct device *dev,
> -     struct device_attribute *attr, char *buf)
> +                            struct device_attribute *attr,
> +                            char *buf)
>  {
>       struct dcon_priv *dcon = dev_get_drvdata(dev);
>  
> @@ -409,7 +411,8 @@ static ssize_t dcon_sleep_show(struct device *dev,
>  }
>  
>  static ssize_t dcon_freeze_show(struct device *dev,
> -     struct device_attribute *attr, char *buf)
> +                             struct device_attribute *attr,
> +                             char *buf)
>  {
>       struct dcon_priv *dcon = dev_get_drvdata(dev);
>  
> @@ -417,7 +420,8 @@ static ssize_t dcon_freeze_show(struct device *dev,
>  }
>  
>  static ssize_t dcon_mono_show(struct device *dev,
> -     struct device_attribute *attr, char *buf)
> +                           struct device_attribute *attr,
> +                           char *buf)
>  {
>       struct dcon_priv *dcon = dev_get_drvdata(dev);
>  
> @@ -425,13 +429,15 @@ static ssize_t dcon_mono_show(struct device *dev,
>  }
>  
>  static ssize_t dcon_resumeline_show(struct device *dev,
> -     struct device_attribute *attr, char *buf)
> +                                 struct device_attribute *attr,
> +                                 char *buf)
>  {
>       return sprintf(buf, "%d\n", resumeline);
>  }
>  
>  static ssize_t dcon_mono_store(struct device *dev,
> -     struct device_attribute *attr, const char *buf, size_t count)
> +                            struct device_attribute *attr,
> +                            const char *buf, size_t count)
>  {
>       unsigned long enable_mono;
>       int rc;
> @@ -446,7 +452,8 @@ static ssize_t dcon_mono_store(struct device *dev,
>  }
>  
>  static ssize_t dcon_freeze_store(struct device *dev,
> -     struct device_attribute *attr, const char *buf, size_t count)
> +                              struct device_attribute *attr,
> +                              const char *buf, size_t count)
>  {
>       struct dcon_priv *dcon = dev_get_drvdata(dev);
>       unsigned long output;
> @@ -474,7 +481,8 @@ static ssize_t dcon_freeze_store(struct device *dev,
>  }
>  
>  static ssize_t dcon_resumeline_store(struct device *dev,
> -     struct device_attribute *attr, const char *buf, size_t count)
> +                                  struct device_attribute *attr,
> +                                  const char *buf, size_t count)
>  {
>       unsigned short rl;
>       int rc;
> @@ -490,7 +498,8 @@ static ssize_t dcon_resumeline_store(struct device *dev,
>  }
>  
>  static ssize_t dcon_sleep_store(struct device *dev,
> -     struct device_attribute *attr, const char *buf, size_t count)
> +                             struct device_attribute *attr,
> +                             const char *buf, size_t count)
>  {
>       unsigned long output;
>       int ret;
> @@ -641,7 +650,8 @@ static int dcon_probe(struct i2c_client *client, const 
> struct i2c_device_id *id)
>       /* Add the backlight device for the DCON */
>       dcon_bl_props.brightness = dcon->bl_val;
>       dcon->bl_dev = backlight_device_register("dcon-bl", &dcon_device->dev,
> -             dcon, &dcon_bl_ops, &dcon_bl_props);
> +                                              dcon, &dcon_bl_ops,
> +                                              &dcon_bl_props);
>       if (IS_ERR(dcon->bl_dev)) {
>               dev_err(&client->dev, "cannot register backlight dev (%ld)\n",
>                       PTR_ERR(dcon->bl_dev));
> diff --git a/drivers/staging/olpc_dcon/olpc_dcon.h 
> b/drivers/staging/olpc_dcon/olpc_dcon.h
> index 8fbde5d3b4a6..fa89bb97c7b0 100644
> --- a/drivers/staging/olpc_dcon/olpc_dcon.h
> +++ b/drivers/staging/olpc_dcon/olpc_dcon.h
> @@ -10,18 +10,18 @@
>  #define DCON_REG_ID           0
>  #define DCON_REG_MODE                 1
>  
> -#define MODE_PASSTHRU        (1<<0)
> -#define MODE_SLEEP   (1<<1)
> -#define MODE_SLEEP_AUTO      (1<<2)
> -#define MODE_BL_ENABLE       (1<<3)
> -#define MODE_BLANK   (1<<4)
> -#define MODE_CSWIZZLE        (1<<5)
> -#define MODE_COL_AA  (1<<6)
> -#define MODE_MONO_LUMA       (1<<7)
> -#define MODE_SCAN_INT        (1<<8)
> -#define MODE_CLOCKDIV        (1<<9)
> -#define MODE_DEBUG   (1<<14)
> -#define MODE_SELFTEST        (1<<15)
> +#define MODE_PASSTHRU        BIT(0)
> +#define MODE_SLEEP   BIT(1)
> +#define MODE_SLEEP_AUTO      BIT(2)
> +#define MODE_BL_ENABLE       BIT(3)
> +#define MODE_BLANK   BIT(4)
> +#define MODE_CSWIZZLE        BIT(5)
> +#define MODE_COL_AA  BIT(6)
> +#define MODE_MONO_LUMA       BIT(7)
> +#define MODE_SCAN_INT        BIT(8)
> +#define MODE_CLOCKDIV        BIT(9)
> +#define MODE_DEBUG   BIT(14)
> +#define MODE_SELFTEST        BIT(15)

This is a different change to the others in this patch. One change per
patch please. (The other change is re-align to match parenthesis).

Also, perhaps you could align all the BIT() calls.

Hope this helps,
Tobin.
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to