* [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