* [PATCH] mfd: Fix the header file name in some copyrights section
@ 2013-12-19 13:08 Laszlo Papp
2013-12-19 13:42 ` Lee Jones
0 siblings, 1 reply; 2+ messages in thread
From: Laszlo Papp @ 2013-12-19 13:08 UTC (permalink / raw)
To: sameo, lee.jones; +Cc: linux-kernel, Laszlo Papp
The original author(s) probably only made a copy/paste from the existing public
header, and that is how this issue could sneak in. It is now more consistent
with the rest where this file name is proper in the copyrights section.
Arguably, the file name could be removed from there, but this is now just done
for catching up with the way how it is done in certain other similar headers.
Signed-off-by: Laszlo Papp <lpapp@kde.org>
---
include/linux/mfd/max77686-private.h | 2 +-
include/linux/mfd/max8997-private.h | 2 +-
include/linux/mfd/max8998-private.h | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/include/linux/mfd/max77686-private.h b/include/linux/mfd/max77686-private.h
index d327d49..8c75a9c 100644
--- a/include/linux/mfd/max77686-private.h
+++ b/include/linux/mfd/max77686-private.h
@@ -1,5 +1,5 @@
/*
- * max77686.h - Voltage regulator driver for the Maxim 77686
+ * max77686-private.h - Voltage regulator driver for the Maxim 77686
*
* Copyright (C) 2012 Samsung Electrnoics
* Chiwoong Byun <woong.byun@samsung.com>
diff --git a/include/linux/mfd/max8997-private.h b/include/linux/mfd/max8997-private.h
index fb465df..ad1ae7f 100644
--- a/include/linux/mfd/max8997-private.h
+++ b/include/linux/mfd/max8997-private.h
@@ -1,5 +1,5 @@
/*
- * max8997.h - Voltage regulator driver for the Maxim 8997
+ * max8997-private.h - Voltage regulator driver for the Maxim 8997
*
* Copyright (C) 2010 Samsung Electrnoics
* MyungJoo Ham <myungjoo.ham@samsung.com>
diff --git a/include/linux/mfd/max8998-private.h b/include/linux/mfd/max8998-private.h
index 84844e0..4ecb24b 100644
--- a/include/linux/mfd/max8998-private.h
+++ b/include/linux/mfd/max8998-private.h
@@ -1,5 +1,5 @@
/*
- * max8998.h - Voltage regulator driver for the Maxim 8998
+ * max8998-private.h - Voltage regulator driver for the Maxim 8998
*
* Copyright (C) 2009-2010 Samsung Electrnoics
* Kyungmin Park <kyungmin.park@samsung.com>
--
1.8.5.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] mfd: Fix the header file name in some copyrights section
2013-12-19 13:08 [PATCH] mfd: Fix the header file name in some copyrights section Laszlo Papp
@ 2013-12-19 13:42 ` Lee Jones
0 siblings, 0 replies; 2+ messages in thread
From: Lee Jones @ 2013-12-19 13:42 UTC (permalink / raw)
To: Laszlo Papp; +Cc: sameo, linux-kernel
On Thu, 19 Dec 2013, Laszlo Papp wrote:
> The original author(s) probably only made a copy/paste from the existing public
> header, and that is how this issue could sneak in. It is now more consistent
> with the rest where this file name is proper in the copyrights section.
> Arguably, the file name could be removed from there, but this is now just done
> for catching up with the way how it is done in certain other similar headers.
>
> Signed-off-by: Laszlo Papp <lpapp@kde.org>
> ---
> include/linux/mfd/max77686-private.h | 2 +-
> include/linux/mfd/max8997-private.h | 2 +-
> include/linux/mfd/max8998-private.h | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
I guess we all have to start somewhere. ;)
More useful patches are more gratefully received, however.
Applied with some small changes to the commit message.
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-12-19 13:43 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-19 13:08 [PATCH] mfd: Fix the header file name in some copyrights section Laszlo Papp
2013-12-19 13:42 ` Lee Jones
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox