Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Dear Benoît Thébaudeau, In message <1879824314.332488.1342790324727.javamail.r...@advansee.com> you wrote: > Remove spaces before opening parentheses in function calls. > > Signed-off-by: Benoît Thébaudeau > Cc: Wolfgang Denk > --- > Changes for v2: > - New patch for this version of the series. > > .../fs/fat/fat.c | 44 ++--> > > 1 file changed, 21 insertions(+), 23 deletions(-) Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Dear Wolfgang Denk, > Dear =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?=, > > you wrote: > > > > > > > If it's ack'ed, why does nobody apply it? What is the normal > > > > > life cycle of > > > > > patches like these that don't have a custodian? > > And I explained to you: > > > They end on my table, and probably get merged when the next merge > > window opens. See http://www.denx.de/wiki/U-Boot/ReleaseCycle > > But you continue to ask again and again and again: > > > > > Can you apply it now that the merge window is open? > > ... > > > Can someone answer, please? > > > > > Do you have any idea why nobody applies this series and answers? > > Who is the fs > > custodian? Wolfgang since I can't find a custodian defined for fs, > > no? Is it > > because of vacations or something? I'd like if this series could be > > applied > > before the end of the current merge window. > > Didn't I tell you that these will be handled by me? The when depends > on when I find time to do this. I have a lot of patches on my stack, > and usually I process these sequentially, and not in the order of the > "priority" as the submitter wants to assign. It is perfectly OK to > send a reminder when there is nothing happening, or when the merge > window closes and I call for unprecessed patches to be flagged, but > frankly, you ask a bit too often. You may want to read > http://www.catb.org/~esr/faqs/smart-questions.html#urgent ... Understood. Sorry for the noise. I'm still new to U-Boot mailing list, so I'm learning the rules and timings progressively. I was not asking for priority over others. It's just that you said you handled such patches when the next merge window opens, but then nothing happened when the merge window opened, even after a few weeks, so I was wondering why. Now it's clear. Best regards, Benoît ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Dear =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?=, In message <1605401818.2306523.1344722631605.javamail.r...@advansee.com> you wrote: > > Oh, great! This is good news. Thanks for the information. The merge window > should be renamed to "submission window" or something since it is not really a > merge window like for Linux. I was scared my patches got lost somewhere. In theory the merge window is really a merge window. It's only due to lack of sufficient resources that we end up with completing the merges usually a bit later. Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Men don't talk peace unless they're ready to back it up with war. -- Col. Green, "The Savage Curtain", stardate 5906.4 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Dear =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?=, you wrote: > > > > > If it's ack'ed, why does nobody apply it? What is the normal life cycle > > > > of > > > > patches like these that don't have a custodian? And I explained to you: > They end on my table, and probably get merged when the next merge > window opens. See http://www.denx.de/wiki/U-Boot/ReleaseCycle But you continue to ask again and again and again: > > > Can you apply it now that the merge window is open? > ... > > Can someone answer, please? > > > Do you have any idea why nobody applies this series and answers? Who is the fs > custodian? Wolfgang since I can't find a custodian defined for fs, no? Is it > because of vacations or something? I'd like if this series could be applied > before the end of the current merge window. Didn't I tell you that these will be handled by me? The when depends on when I find time to do this. I have a lot of patches on my stack, and usually I process these sequentially, and not in the order of the "priority" as the submitter wants to assign. It is perfectly OK to send a reminder when there is nothing happening, or when the merge window closes and I call for unprecessed patches to be flagged, but frankly, you ask a bit too often. You may want to read http://www.catb.org/~esr/faqs/smart-questions.html#urgent ... Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Doubt isn't the opposite of faith; it is an element of faith. - Paul Tillich, German theologian and historian ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Hi Graeme, On Aug 11, 2012 11:43:39 PM, Graeme Russ wrote: > (added ML to Cc) > Hi Benoît, > On Aug 12, 2012 6:27 AM, "Benoît Thébaudeau" < > benoit.thebaud...@advansee.com > wrote: > > > > Hi Mike, > > > > On Fri, Aug 10, 2012 at 04:41:51 PM, Benoît Thébaudeau wrote: > > > On Sat, Aug 4, 2012 at 01:11:16 AM, Benoît Thébaudeau wrote: > > > > Dear Wolfgang Denk, > > > > > > > > On Fri, Jul 27, 2012 at 04:29:41 PM, Wolfgang Denk wrote: > > > > > Dear =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?=, > > > > > > > > > > In message > > > > > < 845266616.705298.1343398892482.javamail.r...@advansee.com > > > > > > you > > > > > wrote: > > > > > > > > > > > > On Sat, Jul 21, 2012 at 07:23:57 PM, Mike Frysinger wrote: > > > > > > > Acked-by: Mike Frysinger < vap...@gentoo.org > > > > > > > > > > > > > If it's ack'ed, why does nobody apply it? What is the > > > > > > normal > > > > > > life > > > > > > cycle of > > > > > > patches like these that don't have a custodian? > > > > > > > > > > They end on my table, and probably get merged when the next > > > > > merge > > > > > window opens. See http://www.denx.de/wiki/U-Boot/ReleaseCycle > > > > > > > > Can you apply it now that the merge window is open? > > > > > > > > Thanks in advance. > > > > > > Can someone answer, please? > > > > Do you have any idea why nobody applies this series and answers? > > Who is the fs > > custodian? Wolfgang since I can't find a custodian defined for fs, > > no? Is it > > because of vacations or something? I'd like if this series could be > > applied > > before the end of the current merge window. > > > It is probably somewhere in Wolfgang's todo list. The merge window is > not an indicator of when patches must be applied, it's a submission > cutoff date. As a general rule, as long as the patches are submitted > before the window closes, they will go in the next release (unless > they are nack'd or cause breakage). > The best time to ping patches is during the RC cycle just prior > to.the release. Oh, great! This is good news. Thanks for the information. The merge window should be renamed to "submission window" or something since it is not really a merge window like for Linux. I was scared my patches got lost somewhere. Best regards, Benoît ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Hi Mike, On Fri, Aug 10, 2012 at 04:41:51 PM, Benoît Thébaudeau wrote: > On Sat, Aug 4, 2012 at 01:11:16 AM, Benoît Thébaudeau wrote: > > Dear Wolfgang Denk, > > > > On Fri, Jul 27, 2012 at 04:29:41 PM, Wolfgang Denk wrote: > > > Dear =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?=, > > > > > > In message > > > <845266616.705298.1343398892482.javamail.r...@advansee.com> you > > > wrote: > > > > > > > > On Sat, Jul 21, 2012 at 07:23:57 PM, Mike Frysinger wrote: > > > > > Acked-by: Mike Frysinger > > > > > > > > If it's ack'ed, why does nobody apply it? What is the normal > > > > life > > > > cycle of > > > > patches like these that don't have a custodian? > > > > > > They end on my table, and probably get merged when the next merge > > > window opens. See http://www.denx.de/wiki/U-Boot/ReleaseCycle > > > > Can you apply it now that the merge window is open? > > > > Thanks in advance. > > Can someone answer, please? Do you have any idea why nobody applies this series and answers? Who is the fs custodian? Wolfgang since I can't find a custodian defined for fs, no? Is it because of vacations or something? I'd like if this series could be applied before the end of the current merge window. Best regards, Benoît ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
On Sat, Aug 4, 2012 at 01:11:16 AM, Benoît Thébaudeau wrote: > Dear Wolfgang Denk, > > On Fri, Jul 27, 2012 at 04:29:41 PM, Wolfgang Denk wrote: > > Dear =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?=, > > > > In message > > <845266616.705298.1343398892482.javamail.r...@advansee.com> you > > wrote: > > > > > > On Sat, Jul 21, 2012 at 07:23:57 PM, Mike Frysinger wrote: > > > > Acked-by: Mike Frysinger > > > > > > If it's ack'ed, why does nobody apply it? What is the normal life > > > cycle of > > > patches like these that don't have a custodian? > > > > They end on my table, and probably get merged when the next merge > > window opens. See http://www.denx.de/wiki/U-Boot/ReleaseCycle > > Can you apply it now that the merge window is open? > > Thanks in advance. Can someone answer, please? Best regards, Benoît ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Dear Wolfgang Denk, On Fri, Jul 27, 2012 at 04:29:41 PM, Wolfgang Denk wrote: > Dear =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?=, > > In message > <845266616.705298.1343398892482.javamail.r...@advansee.com> you > wrote: > > > > On Sat, Jul 21, 2012 at 07:23:57 PM, Mike Frysinger wrote: > > > Acked-by: Mike Frysinger > > > > If it's ack'ed, why does nobody apply it? What is the normal life > > cycle of > > patches like these that don't have a custodian? > > They end on my table, and probably get merged when the next merge > window opens. See http://www.denx.de/wiki/U-Boot/ReleaseCycle Can you apply it now that the merge window is open? Thanks in advance. Best regards, Benoît ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Dear =?utf-8?Q?Beno=C3=AEt_Th=C3=A9baudeau?=, In message <845266616.705298.1343398892482.javamail.r...@advansee.com> you wrote: > > On Sat, Jul 21, 2012 at 07:23:57 PM, Mike Frysinger wrote: > > Acked-by: Mike Frysinger > > If it's ack'ed, why does nobody apply it? What is the normal life cycle of > patches like these that don't have a custodian? They end on my table, and probably get merged when the next merge window opens. See http://www.denx.de/wiki/U-Boot/ReleaseCycle Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de What is wanted is not the will to believe, but the will to find out, which is the exact opposite. -- Bertrand Russell, "Skeptical Essays", 1928 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Dear Mike, On Sat, Jul 21, 2012 at 07:23:57 PM, Mike Frysinger wrote: > Acked-by: Mike Frysinger If it's ack'ed, why does nobody apply it? What is the normal life cycle of patches like these that don't have a custodian? Regards, Benoît ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Acked-by: Mike Frysinger -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Argh, it was 1/8 actually, not 1/9...
- Original Message -
> From: "Benoît Thébaudeau"
> To: u-boot@lists.denx.de
> Cc: "Wolfgang Denk" , "Mike Frysinger"
> Sent: Friday, July 20, 2012 3:18:44 PM
> Subject: [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
>
> Remove spaces before opening parentheses in function calls.
>
> Signed-off-by: Benoît Thébaudeau
> Cc: Wolfgang Denk
> ---
> Changes for v2:
> - New patch for this version of the series.
>
> .../fs/fat/fat.c | 44
> ++--
> 1 file changed, 21 insertions(+), 23 deletions(-)
>
> diff --git u-boot-66714b1.orig/fs/fat/fat.c
> u-boot-66714b1/fs/fat/fat.c
> index bc46cc5..9f83572 100644
> --- u-boot-66714b1.orig/fs/fat/fat.c
> +++ u-boot-66714b1/fs/fat/fat.c
> @@ -37,7 +37,7 @@
> /*
> * Convert a string to lowercase.
> */
> -static void downcase (char *str)
> +static void downcase(char *str)
> {
> while (*str != '\0') {
> TOLOWER(*str);
> @@ -62,7 +62,7 @@ static int disk_read(__u32 block, __u32 nr_blocks,
> void *buf)
> cur_part_info.start + block, nr_blocks, buf);
> }
>
> -int fat_register_device (block_dev_desc_t * dev_desc, int part_no)
> +int fat_register_device(block_dev_desc_t * dev_desc, int part_no)
> {
> ALLOC_CACHE_ALIGN_BUFFER(unsigned char, buffer, dev_desc->blksz);
>
> @@ -127,7 +127,7 @@ int fat_register_device (block_dev_desc_t *
> dev_desc, int part_no)
> * Get the first occurence of a directory delimiter ('/' or '\') in
> a string.
> * Return index into string if found, -1 otherwise.
> */
> -static int dirdelim (char *str)
> +static int dirdelim(char *str)
> {
> char *start = str;
>
> @@ -142,7 +142,7 @@ static int dirdelim (char *str)
> /*
> * Extract zero terminated short name from a directory entry.
> */
> -static void get_name (dir_entry *dirent, char *s_name)
> +static void get_name(dir_entry *dirent, char *s_name)
> {
> char *ptr;
>
> @@ -171,7 +171,7 @@ static void get_name (dir_entry *dirent, char
> *s_name)
> * Get the entry at index 'entry' in a FAT (12/16/32) table.
> * On failure 0x00 is returned.
> */
> -static __u32 get_fatent (fsdata *mydata, __u32 entry)
> +static __u32 get_fatent(fsdata *mydata, __u32 entry)
> {
> __u32 bufnum;
> __u32 off16, offset;
> @@ -270,8 +270,7 @@ static __u32 get_fatent (fsdata *mydata, __u32
> entry)
> * Return 0 on success, -1 otherwise.
> */
> static int
> -get_cluster (fsdata *mydata, __u32 clustnum, __u8 *buffer,
> - unsigned long size)
> +get_cluster(fsdata *mydata, __u32 clustnum, __u8 *buffer, unsigned
> long size)
> {
> __u32 idx = 0;
> __u32 startsect;
> @@ -317,8 +316,8 @@ get_cluster (fsdata *mydata, __u32 clustnum, __u8
> *buffer,
> * Return the number of bytes read or -1 on fatal errors.
> */
> static long
> -get_contents (fsdata *mydata, dir_entry *dentptr, __u8 *buffer,
> - unsigned long maxsize)
> +get_contents(fsdata *mydata, dir_entry *dentptr, __u8 *buffer,
> + unsigned long maxsize)
> {
> unsigned long filesize = FAT2CPU32(dentptr->size), gotsize = 0;
> unsigned int bytesperclust = mydata->clust_size *
> mydata->sect_size;
> @@ -397,7 +396,7 @@ getit:
> * starting at l_name[*idx].
> * Return 1 if terminator (zero byte) is found, 0 otherwise.
> */
> -static int slot2str (dir_slot *slotptr, char *l_name, int *idx)
> +static int slot2str(dir_slot *slotptr, char *l_name, int *idx)
> {
> int j;
>
> @@ -433,8 +432,8 @@ __u8 get_vfatname_block[MAX_CLUSTSIZE]
> __aligned(ARCH_DMA_MINALIGN);
>
> static int
> -get_vfatname (fsdata *mydata, int curclust, __u8 *cluster,
> - dir_entry *retdent, char *l_name)
> +get_vfatname(fsdata *mydata, int curclust, __u8 *cluster,
> + dir_entry *retdent, char *l_name)
> {
> dir_entry *realdent;
> dir_slot *slotptr = (dir_slot *)retdent;
> @@ -516,7 +515,7 @@ get_vfatname (fsdata *mydata, int curclust, __u8
> *cluster,
> }
>
> /* Calculate short name checksum */
> -static __u8 mkcksum (const char *str)
> +static __u8 mkcksum(const char *str)
> {
> int i;
>
> @@ -537,9 +536,9 @@ static __u8 mkcksum (const char *str)
> __u8 get_dentfromdir_block[MAX_CLUSTSIZE]
> __aligned(ARCH_DMA_MINALIGN);
>
> -static dir_entry *get_dentfromdir (fsdata *mydata, int startsect,
> -char *filename, dir_entry *retdent,
> -int dols)
> +static dir_entry *get_dentfromdir(fsdata *mydata, int startsect,
> + char *filename, dir_entry *retdent,
> + int dols)
> {
> __u16 prevcksum = 0x;
> __u32 curclust = START(retdent);
> @@ -699,7 +698,7 @@ static dir_entry *get_dentfromdir (fsdata
> *mydata, int startsect,
> * Read boot sector and volume info from a FAT filesystem
> */
> static int
> -read_bootsectandvi (boot_sec
[U-Boot] [PATCH v2 1/9] FAT: cosmetic: Remove extra spaces
Remove spaces before opening parentheses in function calls.
Signed-off-by: Benoît Thébaudeau
Cc: Wolfgang Denk
---
Changes for v2:
- New patch for this version of the series.
.../fs/fat/fat.c | 44 ++--
1 file changed, 21 insertions(+), 23 deletions(-)
diff --git u-boot-66714b1.orig/fs/fat/fat.c u-boot-66714b1/fs/fat/fat.c
index bc46cc5..9f83572 100644
--- u-boot-66714b1.orig/fs/fat/fat.c
+++ u-boot-66714b1/fs/fat/fat.c
@@ -37,7 +37,7 @@
/*
* Convert a string to lowercase.
*/
-static void downcase (char *str)
+static void downcase(char *str)
{
while (*str != '\0') {
TOLOWER(*str);
@@ -62,7 +62,7 @@ static int disk_read(__u32 block, __u32 nr_blocks, void *buf)
cur_part_info.start + block, nr_blocks, buf);
}
-int fat_register_device (block_dev_desc_t * dev_desc, int part_no)
+int fat_register_device(block_dev_desc_t * dev_desc, int part_no)
{
ALLOC_CACHE_ALIGN_BUFFER(unsigned char, buffer, dev_desc->blksz);
@@ -127,7 +127,7 @@ int fat_register_device (block_dev_desc_t * dev_desc, int
part_no)
* Get the first occurence of a directory delimiter ('/' or '\') in a string.
* Return index into string if found, -1 otherwise.
*/
-static int dirdelim (char *str)
+static int dirdelim(char *str)
{
char *start = str;
@@ -142,7 +142,7 @@ static int dirdelim (char *str)
/*
* Extract zero terminated short name from a directory entry.
*/
-static void get_name (dir_entry *dirent, char *s_name)
+static void get_name(dir_entry *dirent, char *s_name)
{
char *ptr;
@@ -171,7 +171,7 @@ static void get_name (dir_entry *dirent, char *s_name)
* Get the entry at index 'entry' in a FAT (12/16/32) table.
* On failure 0x00 is returned.
*/
-static __u32 get_fatent (fsdata *mydata, __u32 entry)
+static __u32 get_fatent(fsdata *mydata, __u32 entry)
{
__u32 bufnum;
__u32 off16, offset;
@@ -270,8 +270,7 @@ static __u32 get_fatent (fsdata *mydata, __u32 entry)
* Return 0 on success, -1 otherwise.
*/
static int
-get_cluster (fsdata *mydata, __u32 clustnum, __u8 *buffer,
-unsigned long size)
+get_cluster(fsdata *mydata, __u32 clustnum, __u8 *buffer, unsigned long size)
{
__u32 idx = 0;
__u32 startsect;
@@ -317,8 +316,8 @@ get_cluster (fsdata *mydata, __u32 clustnum, __u8 *buffer,
* Return the number of bytes read or -1 on fatal errors.
*/
static long
-get_contents (fsdata *mydata, dir_entry *dentptr, __u8 *buffer,
- unsigned long maxsize)
+get_contents(fsdata *mydata, dir_entry *dentptr, __u8 *buffer,
+unsigned long maxsize)
{
unsigned long filesize = FAT2CPU32(dentptr->size), gotsize = 0;
unsigned int bytesperclust = mydata->clust_size * mydata->sect_size;
@@ -397,7 +396,7 @@ getit:
* starting at l_name[*idx].
* Return 1 if terminator (zero byte) is found, 0 otherwise.
*/
-static int slot2str (dir_slot *slotptr, char *l_name, int *idx)
+static int slot2str(dir_slot *slotptr, char *l_name, int *idx)
{
int j;
@@ -433,8 +432,8 @@ __u8 get_vfatname_block[MAX_CLUSTSIZE]
__aligned(ARCH_DMA_MINALIGN);
static int
-get_vfatname (fsdata *mydata, int curclust, __u8 *cluster,
- dir_entry *retdent, char *l_name)
+get_vfatname(fsdata *mydata, int curclust, __u8 *cluster,
+dir_entry *retdent, char *l_name)
{
dir_entry *realdent;
dir_slot *slotptr = (dir_slot *)retdent;
@@ -516,7 +515,7 @@ get_vfatname (fsdata *mydata, int curclust, __u8 *cluster,
}
/* Calculate short name checksum */
-static __u8 mkcksum (const char *str)
+static __u8 mkcksum(const char *str)
{
int i;
@@ -537,9 +536,9 @@ static __u8 mkcksum (const char *str)
__u8 get_dentfromdir_block[MAX_CLUSTSIZE]
__aligned(ARCH_DMA_MINALIGN);
-static dir_entry *get_dentfromdir (fsdata *mydata, int startsect,
- char *filename, dir_entry *retdent,
- int dols)
+static dir_entry *get_dentfromdir(fsdata *mydata, int startsect,
+ char *filename, dir_entry *retdent,
+ int dols)
{
__u16 prevcksum = 0x;
__u32 curclust = START(retdent);
@@ -699,7 +698,7 @@ static dir_entry *get_dentfromdir (fsdata *mydata, int
startsect,
* Read boot sector and volume info from a FAT filesystem
*/
static int
-read_bootsectandvi (boot_sector *bs, volume_info *volinfo, int *fatsize)
+read_bootsectandvi(boot_sector *bs, volume_info *volinfo, int *fatsize)
{
__u8 *block;
volume_info *vistart;
@@ -716,7 +715,7 @@ read_bootsectandvi (boot_sector *bs, volume_info *volinfo,
int *fatsize)
return -1;
}
- if (disk_read (0, 1, block) < 0) {
+ if (disk_read(0, 1, block) < 0) {
debug("Error: reading block\n");
goto fail;
}
@@ -770,8 +769,7 @@ __u8
