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