linux-alpha.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/4] alpha: Converse all space errors into tabs in mkbb.c
@ 2014-07-16  4:13 Nicholas Krause
  2014-07-16  4:14 ` [PATCH 2/4] alpha: checkpatch errors in mkbb for if statements Nicholas Krause
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Nicholas Krause @ 2014-07-16  4:13 UTC (permalink / raw)
  To: rth; +Cc: ink, mattst88, linux-alpha, linux-kernel

This patch fixes all the checkpatch errors I get when running it
on mkbb.c for spaces at beginning of lines. I converted then all
to tabs to fix these checkpatch warnings.
---
 arch/alpha/boot/tools/mkbb.c | 124 +++++++++++++++++++++----------------------
 1 file changed, 61 insertions(+), 63 deletions(-)

diff --git a/arch/alpha/boot/tools/mkbb.c b/arch/alpha/boot/tools/mkbb.c
index 1185778..6f1b957 100644
--- a/arch/alpha/boot/tools/mkbb.c
+++ b/arch/alpha/boot/tools/mkbb.c
@@ -34,49 +34,49 @@
 #endif
 
 struct disklabel {
-    u32	d_magic;				/* must be DISKLABELMAGIC */
-    u16	d_type, d_subtype;
-    u8	d_typename[16];
-    u8	d_packname[16];
-    u32	d_secsize;
-    u32	d_nsectors;
-    u32	d_ntracks;
-    u32	d_ncylinders;
-    u32	d_secpercyl;
-    u32	d_secprtunit;
-    u16	d_sparespertrack;
-    u16	d_sparespercyl;
-    u32	d_acylinders;
-    u16	d_rpm, d_interleave, d_trackskew, d_cylskew;
-    u32	d_headswitch, d_trkseek, d_flags;
-    u32	d_drivedata[5];
-    u32	d_spare[5];
-    u32	d_magic2;				/* must be DISKLABELMAGIC */
-    u16	d_checksum;
-    u16	d_npartitions;
-    u32	d_bbsize, d_sbsize;
-    struct d_partition {
+	u32	d_magic;	/* must be DISKLABELMAGIC */
+	u16	d_type, d_subtype;
+	u8	d_typename[16];
+	u8	d_packname[16];
+	u32	d_secsize;
+	u32	d_nsectors;
+	u32	d_ntracks;
+	u32	d_ncylinders;
+	u32	d_secpercyl;
+	u32	d_secprtunit;
+	u16	d_sparespertrack;
+	u16	d_sparespercyl;
+	u32	d_acylinders;
+	u16	d_rpm, d_interleave, d_trackskew, d_cylskew;
+	u32	d_headswitch, d_trkseek, d_flags;
+	u32	d_drivedata[5];
+	u32	d_spare[5];
+	u32	d_magic2;				/* must be DISKLABELMAGIC */
+	u16	d_checksum;
+	u16	d_npartitions;
+	u32	d_bbsize, d_sbsize;
+	struct d_partition {
 	u32	p_size;
 	u32	p_offset;
 	u32	p_fsize;
 	u8	p_fstype;
 	u8	p_frag;
 	u16	p_cpg;
-    } d_partitions[MAXPARTITIONS];
+	} d_partitions[MAXPARTITIONS];
 };
 
 
 typedef union __bootblock {
-    struct {
-        char			__pad1[64];
-        struct disklabel	__label;
-    } __u1;
-    struct {
+	struct {
+	char			__pad1[64];
+	struct disklabel	__label;
+	} __u1;
+	struct {
 	unsigned long		__pad2[63];
 	unsigned long		__checksum;
-    } __u2;
-    char		bootblock_bytes[512];
-    unsigned long	bootblock_quadwords[64];
+	} __u2;
+	char		bootblock_bytes[512];
+	unsigned long	bootblock_quadwords[64];
 } bootblock;
 
 #define	bootblock_label		__u1.__label
@@ -84,69 +84,67 @@ typedef union __bootblock {
 
 int main(int argc, char ** argv)
 {
-    bootblock		bootblock_from_disk;
-    bootblock		bootloader_image;
-    int			dev, fd;
-    int			i;
-    int			nread;
+	bootblock		bootblock_from_disk;
+	bootblock		bootloader_image;
+	int			dev, fd;
+	int			i;
+	int			nread;
 
     /* Make sure of the arg count */
-    if(argc != 3) {
+	if(argc != 3) {
 	fprintf(stderr, "Usage: %s device lxboot\n", argv[0]);
 	exit(0);
-    }
+	}
 
     /* First, open the device and make sure it's accessible */
-    dev = open(argv[1], O_RDWR);
-    if(dev < 0) {
+	dev = open(argv[1], O_RDWR);
+	if(dev < 0) {
 	perror(argv[1]);
 	exit(0);
-    }
+	}
 
     /* Now open the lxboot and make sure it's reasonable */
-    fd = open(argv[2], O_RDONLY);
-    if(fd < 0) {
+	fd = open(argv[2], O_RDONLY);
+	if(fd < 0) {
 	perror(argv[2]);
 	close(dev);
 	exit(0);
-    }
+	}
 
     /* Read in the lxboot */
-    nread = read(fd, &bootloader_image, sizeof(bootblock));
-    if(nread != sizeof(bootblock)) {
+	nread = read(fd, &bootloader_image, sizeof(bootblock));
+	if(nread != sizeof(bootblock)) {
 	perror("lxboot read");
 	fprintf(stderr, "expected %zd, got %d\n", sizeof(bootblock), nread);
 	exit(0);
-    }
+	}
 
     /* Read in the bootblock from disk. */
-    nread = read(dev, &bootblock_from_disk, sizeof(bootblock));
-    if(nread != sizeof(bootblock)) {
+	nread = read(dev, &bootblock_from_disk, sizeof(bootblock));
+	if(nread != sizeof(bootblock)) {
 	perror("bootblock read");
 	fprintf(stderr, "expected %zd, got %d\n", sizeof(bootblock), nread);
 	exit(0);
-    }
+	}
 
     /* Swap the bootblock's disklabel into the bootloader */
-    bootloader_image.bootblock_label = bootblock_from_disk.bootblock_label;
+	bootloader_image.bootblock_label = bootblock_from_disk.bootblock_label;
 
     /* Calculate the bootblock checksum */
-    bootloader_image.bootblock_checksum = 0;
-    for(i = 0; i < 63; i++) {
-	bootloader_image.bootblock_checksum += 
+	bootloader_image.bootblock_checksum = 0;
+	for(i = 0; i < 63; i++) {
+	bootloader_image.bootblock_checksum +=
 			bootloader_image.bootblock_quadwords[i];
-    }
+	}
 
     /* Write the whole thing out! */
-    lseek(dev, 0L, SEEK_SET);
-    if(write(dev, &bootloader_image, sizeof(bootblock)) != sizeof(bootblock)) {
+	lseek(dev, 0L, SEEK_SET);
+	if(write(dev, &bootloader_image, sizeof(bootblock)) != sizeof(bootblock)) {
 	perror("bootblock write");
 	exit(0);
-    }
+	}
 
-    close(fd);
-    close(dev);
-    exit(0);
+	close(fd);
+	close(dev);
+	exit(0);
 }
-
-
-- 
1.9.1


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

* [PATCH 2/4] alpha: checkpatch errors in mkbb for if statements
  2014-07-16  4:13 [PATCH 1/4] alpha: Converse all space errors into tabs in mkbb.c Nicholas Krause
@ 2014-07-16  4:14 ` Nicholas Krause
  2014-07-16  4:14 ` [PATCH 3/4] alpha: Indent if Statements properly Nicholas Krause
  2014-07-16  4:14 ` [PATCH 4/4] alpha: Remove checkpatch error in mkk.b Nicholas Krause
  2 siblings, 0 replies; 4+ messages in thread
From: Nicholas Krause @ 2014-07-16  4:14 UTC (permalink / raw)
  To: rth; +Cc: ink, mattst88, linux-alpha, linux-kernel

This patch fixes all errors related to no space between if and parthesises
to remove this errors when running checkpatch against this file.
---
 arch/alpha/boot/tools/mkbb.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/arch/alpha/boot/tools/mkbb.c b/arch/alpha/boot/tools/mkbb.c
index 6f1b957..214b6c7 100644
--- a/arch/alpha/boot/tools/mkbb.c
+++ b/arch/alpha/boot/tools/mkbb.c
@@ -91,21 +91,21 @@ int main(int argc, char ** argv)
 	int			nread;
 
     /* Make sure of the arg count */
-	if(argc != 3) {
+	if (argc != 3) {
 	fprintf(stderr, "Usage: %s device lxboot\n", argv[0]);
 	exit(0);
 	}
 
     /* First, open the device and make sure it's accessible */
 	dev = open(argv[1], O_RDWR);
-	if(dev < 0) {
+	if (dev < 0) {
 	perror(argv[1]);
 	exit(0);
 	}
 
     /* Now open the lxboot and make sure it's reasonable */
 	fd = open(argv[2], O_RDONLY);
-	if(fd < 0) {
+	if (fd < 0) {
 	perror(argv[2]);
 	close(dev);
 	exit(0);
@@ -113,7 +113,7 @@ int main(int argc, char ** argv)
 
     /* Read in the lxboot */
 	nread = read(fd, &bootloader_image, sizeof(bootblock));
-	if(nread != sizeof(bootblock)) {
+	if (nread != sizeof(bootblock)) {
 	perror("lxboot read");
 	fprintf(stderr, "expected %zd, got %d\n", sizeof(bootblock), nread);
 	exit(0);
@@ -121,7 +121,7 @@ int main(int argc, char ** argv)
 
     /* Read in the bootblock from disk. */
 	nread = read(dev, &bootblock_from_disk, sizeof(bootblock));
-	if(nread != sizeof(bootblock)) {
+	if (nread != sizeof(bootblock)) {
 	perror("bootblock read");
 	fprintf(stderr, "expected %zd, got %d\n", sizeof(bootblock), nread);
 	exit(0);
@@ -132,14 +132,14 @@ int main(int argc, char ** argv)
 
     /* Calculate the bootblock checksum */
 	bootloader_image.bootblock_checksum = 0;
-	for(i = 0; i < 63; i++) {
+	for (i = 0; i < 63; i++) {
 	bootloader_image.bootblock_checksum +=
 			bootloader_image.bootblock_quadwords[i];
 	}
 
     /* Write the whole thing out! */
 	lseek(dev, 0L, SEEK_SET);
-	if(write(dev, &bootloader_image, sizeof(bootblock)) != sizeof(bootblock)) {
+	if (write(dev, &bootloader_image, sizeof(bootblock)) != sizeof(bootblock)) {
 	perror("bootblock write");
 	exit(0);
 	}
-- 
1.9.1


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

* [PATCH 3/4] alpha: Indent if Statements properly
  2014-07-16  4:13 [PATCH 1/4] alpha: Converse all space errors into tabs in mkbb.c Nicholas Krause
  2014-07-16  4:14 ` [PATCH 2/4] alpha: checkpatch errors in mkbb for if statements Nicholas Krause
@ 2014-07-16  4:14 ` Nicholas Krause
  2014-07-16  4:14 ` [PATCH 4/4] alpha: Remove checkpatch error in mkk.b Nicholas Krause
  2 siblings, 0 replies; 4+ messages in thread
From: Nicholas Krause @ 2014-07-16  4:14 UTC (permalink / raw)
  To: rth; +Cc: ink, mattst88, linux-alpha, linux-kernel

This patch removes the errors this file gets with checkpatch due
to errors relating to if statements and their enclosing statements
not being indented properly.
---
 arch/alpha/boot/tools/mkbb.c | 34 +++++++++++++++++-----------------
 1 file changed, 17 insertions(+), 17 deletions(-)

diff --git a/arch/alpha/boot/tools/mkbb.c b/arch/alpha/boot/tools/mkbb.c
index 214b6c7..69f347e 100644
--- a/arch/alpha/boot/tools/mkbb.c
+++ b/arch/alpha/boot/tools/mkbb.c
@@ -92,39 +92,39 @@ int main(int argc, char ** argv)
 
     /* Make sure of the arg count */
 	if (argc != 3) {
-	fprintf(stderr, "Usage: %s device lxboot\n", argv[0]);
-	exit(0);
+		fprintf(stderr, "Usage: %s device lxboot\n", argv[0]);
+		exit(0);
 	}
 
     /* First, open the device and make sure it's accessible */
 	dev = open(argv[1], O_RDWR);
-	if (dev < 0) {
-	perror(argv[1]);
-	exit(0);
+		if (dev < 0) {
+			perror(argv[1]);
+			exit(0);
 	}
 
     /* Now open the lxboot and make sure it's reasonable */
 	fd = open(argv[2], O_RDONLY);
 	if (fd < 0) {
-	perror(argv[2]);
-	close(dev);
-	exit(0);
+		perror(argv[2]);
+		close(dev);
+		exit(0);
 	}
 
     /* Read in the lxboot */
 	nread = read(fd, &bootloader_image, sizeof(bootblock));
 	if (nread != sizeof(bootblock)) {
-	perror("lxboot read");
-	fprintf(stderr, "expected %zd, got %d\n", sizeof(bootblock), nread);
-	exit(0);
+		perror("lxboot read");
+		fprintf(stderr, "expected %zd, got %d\n", sizeof(bootblock), nread);
+		exit(0);
 	}
 
     /* Read in the bootblock from disk. */
 	nread = read(dev, &bootblock_from_disk, sizeof(bootblock));
 	if (nread != sizeof(bootblock)) {
-	perror("bootblock read");
-	fprintf(stderr, "expected %zd, got %d\n", sizeof(bootblock), nread);
-	exit(0);
+		perror("bootblock read");
+		fprintf(stderr, "expected %zd, got %d\n", sizeof(bootblock), nread);
+		exit(0);
 	}
 
     /* Swap the bootblock's disklabel into the bootloader */
@@ -133,15 +133,15 @@ int main(int argc, char ** argv)
     /* Calculate the bootblock checksum */
 	bootloader_image.bootblock_checksum = 0;
 	for (i = 0; i < 63; i++) {
-	bootloader_image.bootblock_checksum +=
+		bootloader_image.bootblock_checksum +=
 			bootloader_image.bootblock_quadwords[i];
 	}
 
     /* Write the whole thing out! */
 	lseek(dev, 0L, SEEK_SET);
 	if (write(dev, &bootloader_image, sizeof(bootblock)) != sizeof(bootblock)) {
-	perror("bootblock write");
-	exit(0);
+		perror("bootblock write");
+		exit(0);
 	}
 
 	close(fd);
-- 
1.9.1

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

* [PATCH 4/4] alpha: Remove checkpatch error in mkk.b
  2014-07-16  4:13 [PATCH 1/4] alpha: Converse all space errors into tabs in mkbb.c Nicholas Krause
  2014-07-16  4:14 ` [PATCH 2/4] alpha: checkpatch errors in mkbb for if statements Nicholas Krause
  2014-07-16  4:14 ` [PATCH 3/4] alpha: Indent if Statements properly Nicholas Krause
@ 2014-07-16  4:14 ` Nicholas Krause
  2 siblings, 0 replies; 4+ messages in thread
From: Nicholas Krause @ 2014-07-16  4:14 UTC (permalink / raw)
  To: rth; +Cc: ink, mattst88, linux-alpha, linux-kernel

This removes a error I get when I run checkpatch on this file, relating
to the asterisk for declaritations needing to be next to the name of the
variable/structure and not the type of variable/structure.
---
 arch/alpha/boot/tools/mkbb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/alpha/boot/tools/mkbb.c b/arch/alpha/boot/tools/mkbb.c
index 69f347e..dac34c2 100644
--- a/arch/alpha/boot/tools/mkbb.c
+++ b/arch/alpha/boot/tools/mkbb.c
@@ -82,7 +82,7 @@ typedef union __bootblock {
 #define	bootblock_label		__u1.__label
 #define bootblock_checksum	__u2.__checksum
 
-int main(int argc, char ** argv)
+int main(int argc, char **argv)
 {
 	bootblock		bootblock_from_disk;
 	bootblock		bootloader_image;
-- 
1.9.1


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

end of thread, other threads:[~2014-07-16  4:14 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-16  4:13 [PATCH 1/4] alpha: Converse all space errors into tabs in mkbb.c Nicholas Krause
2014-07-16  4:14 ` [PATCH 2/4] alpha: checkpatch errors in mkbb for if statements Nicholas Krause
2014-07-16  4:14 ` [PATCH 3/4] alpha: Indent if Statements properly Nicholas Krause
2014-07-16  4:14 ` [PATCH 4/4] alpha: Remove checkpatch error in mkk.b Nicholas Krause

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).