public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] [PATCH V2] drivers/base: use tabs where possible in code indentation
@ 2015-03-08 20:33 Lavinia Tache
  2015-03-08 20:40 ` Joe Perches
  0 siblings, 1 reply; 2+ messages in thread
From: Lavinia Tache @ 2015-03-08 20:33 UTC (permalink / raw)
  To: gregkh; +Cc: linux-kernel

Linux kernel coding style require that tabs should be used instead of
spaces for code indentation.
Problem found using checkpatch.pl script.

Signed-off-by: Lavinia Tache <lavinia.tachee@gmail.com>

---
 Changes since v1: fix commit message as per Greg's suggestion
 
 drivers/base/soc.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/base/soc.c b/drivers/base/soc.c
index 72b5e72..39fca01 100644
--- a/drivers/base/soc.c
+++ b/drivers/base/soc.c
@@ -43,8 +43,8 @@ struct device *soc_device_to_device(struct soc_device *soc_dev)
 }
 
 static umode_t soc_attribute_mode(struct kobject *kobj,
-                                 struct attribute *attr,
-                                 int index)
+				struct attribute *attr,
+				int index)
 {
 	struct device *dev = container_of(kobj, struct device, kobj);
 	struct soc_device *soc_dev = container_of(dev, struct soc_device, dev);
@@ -60,7 +60,7 @@ static umode_t soc_attribute_mode(struct kobject *kobj,
 		return attr->mode;
 	if ((attr == &dev_attr_soc_id.attr)
 	    && (soc_dev->attr->soc_id != NULL))
-	        return attr->mode;
+		return attr->mode;
 
 	/* Unknown or unfilled attribute. */
 	return 0;
@@ -117,7 +117,7 @@ struct soc_device *soc_device_register(struct soc_device_attribute *soc_dev_attr
 
 	soc_dev = kzalloc(sizeof(*soc_dev), GFP_KERNEL);
 	if (!soc_dev) {
-	        ret = -ENOMEM;
+		ret = -ENOMEM;
 		goto out1;
 	}
 
@@ -135,7 +135,7 @@ struct soc_device *soc_device_register(struct soc_device_attribute *soc_dev_attr
 	} while (ret == -EAGAIN);
 
 	if (ret)
-	         goto out2;
+		goto out2;
 
 	soc_dev->attr = soc_dev_attr;
 	soc_dev->dev.bus = &soc_bus_type;
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] [PATCH V2] drivers/base: use tabs where possible in code indentation
  2015-03-08 20:33 [PATCH] [PATCH V2] drivers/base: use tabs where possible in code indentation Lavinia Tache
@ 2015-03-08 20:40 ` Joe Perches
  0 siblings, 0 replies; 2+ messages in thread
From: Joe Perches @ 2015-03-08 20:40 UTC (permalink / raw)
  To: Lavinia Tache; +Cc: gregkh, linux-kernel

On Sun, 2015-03-08 at 22:33 +0200, Lavinia Tache wrote:
> Linux kernel coding style require that tabs should be used instead of
> spaces for code indentation.
> Problem found using checkpatch.pl script.
[]
> diff --git a/drivers/base/soc.c b/drivers/base/soc.c
[]
> @@ -43,8 +43,8 @@ struct device *soc_device_to_device(struct soc_device *soc_dev)
>  }
>  
>  static umode_t soc_attribute_mode(struct kobject *kobj,
> -                                 struct attribute *attr,
> -                                 int index)
> +				struct attribute *attr,
> +				int index)

It'd be nicer to keep the alignment to the open parenthesis.



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-03-08 20:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-08 20:33 [PATCH] [PATCH V2] drivers/base: use tabs where possible in code indentation Lavinia Tache
2015-03-08 20:40 ` Joe Perches

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox