[PATCH 042/199] Documentation/i2c/busses/i2c-ali1535: Checkpatch cleanup

[PATCH 042/199] Documentation/i2c/busses/i2c-ali1535: Checkpatch cleanup

[Andrea Gelmini]

Documentation/i2c/busses/i2c-ali1535:9: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-ali1535:14: ERROR: trailing whitespace

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 Documentation/i2c/busses/i2c-ali1535 |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/i2c/busses/i2c-ali1535 b/Documentation/i2c/busses/i2c-ali1535
index 0db3b4c..acbc65a 100644
--- a/Documentation/i2c/busses/i2c-ali1535
+++ b/Documentation/i2c/busses/i2c-ali1535
@@ -6,12 +6,12 @@ Supported adapters:
 	http://www.ali.com.tw/eng/support/datasheet_request.php
 
 Authors:
-	Frodo Looijaard <frodol@dds.nl>, 
+	Frodo Looijaard <frodol@dds.nl>,
 	Philip Edelbrock <phil@netroedge.com>,
 	Mark D. Studebaker <mdsxyz123@yahoo.com>,
 	Dan Eaton <dan.eaton@rocketlogix.com>,
 	Stephen Rousset<stephen.rousset@rocketlogix.com>
-												
+
 Description
 -----------
 
-- 
1.7.1.251.gf80a2

Re: [PATCH 042/199] Documentation/i2c/busses/i2c-ali1535: Checkpatch cleanup

[Jean Delvare]

Hi Andrea,

On Sun, 23 May 2010 21:56:10 +0200, Andrea Gelmini wrote:
> Documentation/i2c/busses/i2c-ali1535:9: ERROR: trailing whitespace
> Documentation/i2c/busses/i2c-ali1535:14: ERROR: trailing whitespace
> 
> Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
> ---
>  Documentation/i2c/busses/i2c-ali1535 |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/i2c/busses/i2c-ali1535 b/Documentation/i2c/busses/i2c-ali1535
> index 0db3b4c..acbc65a 100644
> --- a/Documentation/i2c/busses/i2c-ali1535
> +++ b/Documentation/i2c/busses/i2c-ali1535
> @@ -6,12 +6,12 @@ Supported adapters:
>  	http://www.ali.com.tw/eng/support/datasheet_request.php
>  
>  Authors:
> -	Frodo Looijaard <frodol@dds.nl>, 
> +	Frodo Looijaard <frodol@dds.nl>,
>  	Philip Edelbrock <phil@netroedge.com>,
>  	Mark D. Studebaker <mdsxyz123@yahoo.com>,
>  	Dan Eaton <dan.eaton@rocketlogix.com>,
>  	Stephen Rousset<stephen.rousset@rocketlogix.com>
> -												
> +
>  Description
>  -----------
>  

For such a trivial change, I definitely prefer a single patch for all
files. I'll merge them all and apply the result.

Thanks,
-- 
Jean Delvare

Re: [PATCH 042/199] Documentation/i2c/busses/i2c-ali1535: Checkpatch cleanup

[Randy Dunlap]

On Sun, 23 May 2010 23:14:55 +0200 Jean Delvare wrote:

> Hi Andrea,
> 
> On Sun, 23 May 2010 21:56:10 +0200, Andrea Gelmini wrote:
> > Documentation/i2c/busses/i2c-ali1535:9: ERROR: trailing whitespace
> > Documentation/i2c/busses/i2c-ali1535:14: ERROR: trailing whitespace
> > 
> > Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
> > ---
> >  Documentation/i2c/busses/i2c-ali1535 |    4 ++--
> >  1 files changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/Documentation/i2c/busses/i2c-ali1535 b/Documentation/i2c/busses/i2c-ali1535
> > index 0db3b4c..acbc65a 100644
> > --- a/Documentation/i2c/busses/i2c-ali1535
> > +++ b/Documentation/i2c/busses/i2c-ali1535
> > @@ -6,12 +6,12 @@ Supported adapters:
> >  	http://www.ali.com.tw/eng/support/datasheet_request.php
> >  
> >  Authors:
> > -	Frodo Looijaard <frodol@dds.nl>, 
> > +	Frodo Looijaard <frodol@dds.nl>,
> >  	Philip Edelbrock <phil@netroedge.com>,
> >  	Mark D. Studebaker <mdsxyz123@yahoo.com>,
> >  	Dan Eaton <dan.eaton@rocketlogix.com>,
> >  	Stephen Rousset<stephen.rousset@rocketlogix.com>
> > -												
> > +
> >  Description
> >  -----------
> >  
> 
> For such a trivial change, I definitely prefer a single patch for all
> files. I'll merge them all and apply the result.

Thanks, Jean.

Other subsystem maintainers can make their own choices about this
and either merge them as is, merge them into one subsystem patch,
or ask Andrea to merge them into one patch per subsystem.

If they are merged into one patch per subsystem, I'll likely merge them
if subsystem owners do not do so.  As is, I won't.  It's just too much work
for too little gain.

---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

Re: [PATCH 042/199] Documentation/i2c/busses/i2c-ali1535: Checkpatch cleanup

[Andrea Gelmini]

2010/5/23 Randy Dunlap <rdunlap@xenotime.net>:
> If they are merged into one patch per subsystem, I'll likely merge them
> if subsystem owners do not do so.  As is, I won't.  It's just too much work
> for too little gain.

I agree.
If you prefer I can check all Documentation/ and put patches in big
chunks (trailing spaces, and so on).

Thanks a lot for your time,
Andrea

Re: [PATCH 042/199] Documentation/i2c/busses/i2c-ali1535: Checkpatch cleanup

[Randy Dunlap]

On Sun, 23 May 2010 23:50:23 +0200 Andrea Gelmini wrote:

> 2010/5/23 Randy Dunlap <rdunlap@xenotime.net>:
> > If they are merged into one patch per subsystem, I'll likely merge them
> > if subsystem owners do not do so.  As is, I won't.  It's just too much work
> > for too little gain.
> 
> I agree.
> If you prefer I can check all Documentation/ and put patches in big
> chunks (trailing spaces, and so on).

They don't even have to be trailing-spaces-specific (for example).
Just all of Documentation/ as one patch or multiple Documentation/<subsystem>/
patches.

I think that I would prefer one patch per subsystem.

thanks,
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

Re: [PATCH 042/199] Documentation/i2c/busses/i2c-ali1535: Checkpatch cleanup

[Andrea Gelmini]

2010/5/23 Jean Delvare <khali@linux-fr.org>:
> For such a trivial change, I definitely prefer a single patch for all
> files. I'll merge them all and apply the result.

No prob, as you prefer.

Thanks a lot for your work,
Andrea

[PATCH 043/199] Documentation/i2c/busses/i2c-ali1563: Checkpatch cleanup

[Andrea Gelmini]

Documentation/i2c/busses/i2c-ali1563:21: ERROR: trailing whitespace

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 Documentation/i2c/busses/i2c-ali1563 |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/Documentation/i2c/busses/i2c-ali1563 b/Documentation/i2c/busses/i2c-ali1563
index 99ad4b9..5469169 100644
--- a/Documentation/i2c/busses/i2c-ali1563
+++ b/Documentation/i2c/busses/i2c-ali1563
@@ -18,7 +18,7 @@ For an overview of these chips see http://www.acerlabs.com
 The M1563 southbridge is deceptively similar to the M1533, with a few
 notable exceptions. One of those happens to be the fact they upgraded the
 i2c core to be SMBus 2.0 compliant, and happens to be almost identical to
-the i2c controller found in the Intel 801 south bridges. 
+the i2c controller found in the Intel 801 south bridges.
 
 Features
 --------
-- 
1.7.1.251.gf80a2

[PATCH 046/199] Documentation/i2c/busses/i2c-sis5595: Checkpatch cleanup

[Andrea Gelmini]

Documentation/i2c/busses/i2c-sis5595:3: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:6: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:12: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:14: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:15: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:16: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:17: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:18: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:19: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:20: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:21: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:22: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:23: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:24: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:25: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:26: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:27: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:28: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:29: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:30: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:31: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:32: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:33: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:34: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:35: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:36: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:37: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis5595:38: ERROR: trailing whitespace

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 Documentation/i2c/busses/i2c-sis5595 |   58 +++++++++++++++++-----------------
 1 files changed, 29 insertions(+), 29 deletions(-)

diff --git a/Documentation/i2c/busses/i2c-sis5595 b/Documentation/i2c/busses/i2c-sis5595
index cc47db7..ecd21fb 100644
--- a/Documentation/i2c/busses/i2c-sis5595
+++ b/Documentation/i2c/busses/i2c-sis5595
@@ -1,41 +1,41 @@
 Kernel driver i2c-sis5595
 
-Authors: 
+Authors:
 	Frodo Looijaard <frodol@dds.nl>,
         Mark D. Studebaker <mdsxyz123@yahoo.com>,
-	Philip Edelbrock <phil@netroedge.com> 
+	Philip Edelbrock <phil@netroedge.com>
 
 Supported adapters:
   * Silicon Integrated Systems Corp. SiS5595 Southbridge
     Datasheet: Publicly available at the Silicon Integrated Systems Corp. site.
 
-Note: all have mfr. ID 0x1039. 
-
-   SUPPORTED            PCI ID           
-        5595            0008 
- 
-   Note: these chips contain a 0008 device which is incompatible with the 
-         5595. We recognize these by the presence of the listed 
-         "blacklist" PCI ID and refuse to load. 
- 
-   NOT SUPPORTED        PCI ID          BLACKLIST PCI ID         
-         540            0008            0540 
-         550            0008            0550 
-        5513            0008            5511 
-        5581            0008            5597 
-        5582            0008            5597 
-        5597            0008            5597 
-        5598            0008            5597/5598 
-         630            0008            0630 
-         645            0008            0645 
-         646            0008            0646 
-         648            0008            0648 
-         650            0008            0650 
-         651            0008            0651 
-         730            0008            0730 
-         735            0008            0735 
-         745            0008            0745 
-         746            0008            0746 
+Note: all have mfr. ID 0x1039.
+
+   SUPPORTED            PCI ID
+        5595            0008
+
+   Note: these chips contain a 0008 device which is incompatible with the
+         5595. We recognize these by the presence of the listed
+         "blacklist" PCI ID and refuse to load.
+
+   NOT SUPPORTED        PCI ID          BLACKLIST PCI ID
+         540            0008            0540
+         550            0008            0550
+        5513            0008            5511
+        5581            0008            5597
+        5582            0008            5597
+        5597            0008            5597
+        5598            0008            5597/5598
+         630            0008            0630
+         645            0008            0645
+         646            0008            0646
+         648            0008            0648
+         650            0008            0650
+         651            0008            0651
+         730            0008            0730
+         735            0008            0735
+         745            0008            0745
+         746            0008            0746
 
 Module Parameters
 -----------------
-- 
1.7.1.251.gf80a2

[PATCH 044/199] Documentation/i2c/busses/i2c-ali15x3: Checkpatch cleanup

[Andrea Gelmini]

Documentation/i2c/busses/i2c-ali15x3:9: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-ali15x3:10: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-ali15x3:43: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-ali15x3:44: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-ali15x3:46: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-ali15x3:80: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-ali15x3:90: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-ali15x3:91: ERROR: trailing whitespace

Signed-off-by: Andrea Gelmini <andrea.gelmini-PT7Us6ch6G9eoWH0uzbU5w@public.gmane.org>
---
 Documentation/i2c/busses/i2c-ali15x3 |   16 ++++++++--------
 1 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/Documentation/i2c/busses/i2c-ali15x3 b/Documentation/i2c/busses/i2c-ali15x3
index ff28d38..600da90 100644
--- a/Documentation/i2c/busses/i2c-ali15x3
+++ b/Documentation/i2c/busses/i2c-ali15x3
@@ -6,8 +6,8 @@ Supported adapters:
 	http://www.ali.com.tw/eng/support/datasheet_request.php
 
 Authors:
-	Frodo Looijaard <frodol-B0qZmFHriGg@public.gmane.org>, 
-	Philip Edelbrock <phil-KXOFo5pg7o1l57MIdRCFDg@public.gmane.org>, 
+	Frodo Looijaard <frodol-B0qZmFHriGg@public.gmane.org>,
+	Philip Edelbrock <phil-KXOFo5pg7o1l57MIdRCFDg@public.gmane.org>,
 	Mark D. Studebaker <mdsxyz123-/E1597aS9LQAvxtiuMwx3w@public.gmane.org>
 
 Module Parameters
@@ -40,10 +40,10 @@ M1541 and M1543C South Bridges.
 The M1543C is a South bridge for desktop systems.
 The M1541 is a South bridge for portable systems.
 They are part of the following ALI chipsets:
-   
- * "Aladdin Pro 2" includes the M1621 Slot 1 North bridge with AGP and 
+
+ * "Aladdin Pro 2" includes the M1621 Slot 1 North bridge with AGP and
  		100MHz CPU Front Side bus
- * "Aladdin V" includes the M1541 Socket 7 North bridge with AGP and 100MHz 
+ * "Aladdin V" includes the M1541 Socket 7 North bridge with AGP and 100MHz
  		CPU Front Side bus
    Some Aladdin V motherboards:
 	Asus P5A
@@ -77,7 +77,7 @@ output of lspci will show something similar to the following:
 ** then run lspci.
 ** If you see the 1533 and 5229 devices but NOT the 7101 device,
 ** then you must enable ACPI, the PMU, SMB, or something similar
-** in the BIOS. 
+** in the BIOS.
 ** The driver won't work if it can't find the M7101 device.
 
 The SMB controller is part of the M7101 device, which is an ACPI-compliant
@@ -87,8 +87,8 @@ The whole M7101 device has to be enabled for the SMB to work. You can't
 just enable the SMB alone. The SMB and the ACPI have separate I/O spaces.
 We make sure that the SMB is enabled. We leave the ACPI alone.
 
-Features 
--------- 
+Features
+--------
 
 This driver controls the SMB Host only. The SMB Slave
 controller on the M15X3 is not enabled. This driver does not use
-- 
1.7.1.251.gf80a2

[PATCH 045/199] Documentation/i2c/busses/i2c-pca-isa: Checkpatch cleanup

[Andrea Gelmini]

Documentation/i2c/busses/i2c-pca-isa:4: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-pca-isa:5: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-pca-isa:7: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-pca-isa:15: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-pca-isa:16: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-pca-isa:22: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-pca-isa:23: ERROR: trailing whitespace

Signed-off-by: Andrea Gelmini <andrea.gelmini-PT7Us6ch6G9eoWH0uzbU5w@public.gmane.org>
---
 Documentation/i2c/busses/i2c-pca-isa |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/Documentation/i2c/busses/i2c-pca-isa b/Documentation/i2c/busses/i2c-pca-isa
index 6fc8f4c..b044e52 100644
--- a/Documentation/i2c/busses/i2c-pca-isa
+++ b/Documentation/i2c/busses/i2c-pca-isa
@@ -1,10 +1,10 @@
 Kernel driver i2c-pca-isa
 
 Supported adapters:
-This driver supports ISA boards using the Philips PCA 9564 
-Parallel bus to I2C bus controller 
+This driver supports ISA boards using the Philips PCA 9564
+Parallel bus to I2C bus controller
 
-Author: Ian Campbell <icampbell-2sJRl1BP9u0AvxtiuMwx3w@public.gmane.org>, Arcom Control Systems 
+Author: Ian Campbell <icampbell-2sJRl1BP9u0AvxtiuMwx3w@public.gmane.org>, Arcom Control Systems
 
 Module Parameters
 -----------------
@@ -12,12 +12,12 @@ Module Parameters
 * base int
  I/O base address
 * irq int
- IRQ interrupt 
-* clock int 
+ IRQ interrupt
+* clock int
  Clock rate as described in table 1 of PCA9564 datasheet
 
 Description
 -----------
 
-This driver supports ISA boards using the Philips PCA 9564 
-Parallel bus to I2C bus controller 
+This driver supports ISA boards using the Philips PCA 9564
+Parallel bus to I2C bus controller
-- 
1.7.1.251.gf80a2

[PATCH 047/199] Documentation/i2c/busses/i2c-sis630: Checkpatch cleanup

[Andrea Gelmini]

Documentation/i2c/busses/i2c-sis630:17: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis630:18: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis630:19: ERROR: trailing whitespace
Documentation/i2c/busses/i2c-sis630:47: ERROR: trailing whitespace

Signed-off-by: Andrea Gelmini <andrea.gelmini-PT7Us6ch6G9eoWH0uzbU5w@public.gmane.org>
---
 Documentation/i2c/busses/i2c-sis630 |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/i2c/busses/i2c-sis630 b/Documentation/i2c/busses/i2c-sis630
index 9aca688..629ea2c 100644
--- a/Documentation/i2c/busses/i2c-sis630
+++ b/Documentation/i2c/busses/i2c-sis630
@@ -14,9 +14,9 @@ Module Parameters
 * force = [1|0] Forcibly enable the SIS630. DANGEROUS!
 		This can be interesting for chipsets not named
 		above to check if it works for you chipset, but DANGEROUS!
-		
-* high_clock = [1|0] Forcibly set Host Master Clock to 56KHz (default, 
-			what your BIOS use). DANGEROUS! This should be a bit 
+
+* high_clock = [1|0] Forcibly set Host Master Clock to 56KHz (default,
+			what your BIOS use). DANGEROUS! This should be a bit
 			faster, but freeze some systems (i.e. my Laptop).
 
 
@@ -44,6 +44,6 @@ Philip Edelbrock <phil-KXOFo5pg7o1l57MIdRCFDg@public.gmane.org>
 - testing SiS730 support
 Mark M. Hoffman <mhoffman-xQSgfq/1h4JiLUuM0BA3LQ@public.gmane.org>
 - bug fixes
- 
+
 To anyone else which I forgot here ;), thanks!
 
-- 
1.7.1.251.gf80a2

[PATCH 048/199] Documentation/i2c/ten-bit-addresses: Checkpatch cleanup

[Andrea Gelmini]

Documentation/i2c/ten-bit-addresses:1: ERROR: trailing whitespace
Documentation/i2c/ten-bit-addresses:7: ERROR: trailing whitespace
Documentation/i2c/ten-bit-addresses:14: ERROR: trailing whitespace

Signed-off-by: Andrea Gelmini <andrea.gelmini@gelma.net>
---
 Documentation/i2c/ten-bit-addresses |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/i2c/ten-bit-addresses b/Documentation/i2c/ten-bit-addresses
index 200074f..e989070 100644
--- a/Documentation/i2c/ten-bit-addresses
+++ b/Documentation/i2c/ten-bit-addresses
@@ -1,17 +1,17 @@
-The I2C protocol knows about two kinds of device addresses: normal 7 bit 
+The I2C protocol knows about two kinds of device addresses: normal 7 bit
 addresses, and an extended set of 10 bit addresses. The sets of addresses
 do not intersect: the 7 bit address 0x10 is not the same as the 10 bit
 address 0x10 (though a single device could respond to both of them). You
 select a 10 bit address by adding an extra byte after the address
 byte:
-  S Addr7 Rd/Wr ....  
+  S Addr7 Rd/Wr ....
 becomes
   S 11110 Addr10 Rd/Wr
 S is the start bit, Rd/Wr the read/write bit, and if you count the number
 of bits, you will see the there are 8 after the S bit for 7 bit addresses,
 and 16 after the S bit for 10 bit addresses.
 
-WARNING! The current 10 bit address support is EXPERIMENTAL. There are 
+WARNING! The current 10 bit address support is EXPERIMENTAL. There are
 several places in the code that will cause SEVERE PROBLEMS with 10 bit
 addresses, even though there is some basic handling and hooks. Also,
 almost no supported adapter handles the 10 bit addresses correctly.
-- 
1.7.1.251.gf80a2