Re: Fix style in s3c-hsotg.c
Hi! > > Pavel, Felipe's remark was valid.. > > > > > > No, I'll not polish patch for hardware I don't have and have little > > > > interest in. wanted to help you, but according to your first reply, > > > > you do not really want help. > > > > > > that's your call. Now how about you stop being such a baby and go fix > > > your mistakes to start with ? Just because I'm the maintainer of the > > > gadget framework, doesn't mean I'm the maintainer of s3c-hsotc.c file. > > > Maintainer != author too, btw. > > > > > > Anyway, I got much better stuff to do than babysitting grown ups. > > > > ..even if it could have been communicated in a gentler way. > > > > Anyway s3c-hsotg.c is a driver for our hardware so I'm going to pick this > > patch up, polish it and then re-submit it later. Thanks. > > I tried a few times being nice and the response was always in the lines > of "you should go fix the patch yourself if you don't like it" (not > exactly these words, but same message). After some time of being nice > and get nowhere, I just got fed up. For the record, this is Felipe's idea of being nice: > > > > > this is not how you send a patch, please read > > > > > Documentation/SubmittingPatches (What an useless reply. Yes, I know how to submit patches. I also know that patches need to be tested, so I was not interested in "proper" submission. I tried to explain the difference between submit patch for inclusion and make someone look at the patch and fix the code...) > > > > Have you considered possibility that this is how you nudge maintainer > > > > into fix their coding style? (Felipe) > > > cute... > > > > > > Seriously though, read that file, you're commit log has garbage in it > > > which shouldn't go to git history. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Fix style in s3c-hsotg.c
Hi! > > > No, I'll not polish patch for hardware I don't have and have little > > > interest in. wanted to help you, but according to your first reply, > > > you do not really want help. > > > > that's your call. Now how about you stop being such a baby and go fix > > your mistakes to start with ? Just because I'm the maintainer of the > > gadget framework, doesn't mean I'm the maintainer of s3c-hsotc.c file. > > Maintainer != author too, btw. > > > > Anyway, I got much better stuff to do than babysitting grown ups. > > ..even if it could have been communicated in a gentler way. > > Anyway s3c-hsotg.c is a driver for our hardware so I'm going to pick this > patch up, polish it and then re-submit it later. Thanks. Here's version that should compile. Thanks, Pavel Signed-off-by: Pavel Machek diff --git a/drivers/usb/gadget/s3c-hsotg.c b/drivers/usb/gadget/s3c-hsotg.c index d69b36a..c8cdde0 100644 --- a/drivers/usb/gadget/s3c-hsotg.c +++ b/drivers/usb/gadget/s3c-hsotg.c @@ -2092,12 +2092,14 @@ static void kill_all_requests(struct s3c_hsotg *hsotg, } #define call_gadget(_hs, _entry) \ +do { \ if ((_hs)->gadget.speed != USB_SPEED_UNKNOWN && \ (_hs)->driver && (_hs)->driver->_entry) { \ spin_unlock(&_hs->lock); \ (_hs)->driver->_entry(&(_hs)->gadget); \ spin_lock(&_hs->lock); \ - } + } \ +} while (0) /** * s3c_hsotg_disconnect - disconnect service @@ -3205,7 +3207,7 @@ static int state_show(struct seq_file *seq, void *v) readl(regs + GNPTXSTS), readl(regs + GRXSTSR)); - seq_printf(seq, "\nEndpoint status:\n"); + seq_puts(seq, "\nEndpoint status:\n"); for (idx = 0; idx < 15; idx++) { u32 in, out; @@ -3222,7 +3224,7 @@ static int state_show(struct seq_file *seq, void *v) seq_printf(seq, ", DIEPTSIZ=0x%08x, DOEPTSIZ=0x%08x", in, out); - seq_printf(seq, "\n"); + seq_puts(seq, "\n"); } return 0; @@ -3256,7 +3258,7 @@ static int fifo_show(struct seq_file *seq, void *v) u32 val; int idx; - seq_printf(seq, "Non-periodic FIFOs:\n"); + seq_puts(seq, "Non-periodic FIFOs:\n"); seq_printf(seq, "RXFIFO: Size %d\n", readl(regs + GRXFSIZ)); val = readl(regs + GNPTXFSIZ); @@ -3264,7 +3266,7 @@ static int fifo_show(struct seq_file *seq, void *v) val >> GNPTXFSIZ_NPTxFDep_SHIFT, val & GNPTXFSIZ_NPTxFStAddr_MASK); - seq_printf(seq, "\nPeriodic TXFIFOs:\n"); + seq_puts(seq, "\nPeriodic TXFIFOs:\n"); for (idx = 1; idx <= 15; idx++) { val = readl(regs + DPTXFSIZn(idx)); @@ -3335,7 +3337,7 @@ static int ep_show(struct seq_file *seq, void *v) readl(regs + DIEPTSIZ(index)), readl(regs + DOEPTSIZ(index))); - seq_printf(seq, "\n"); + seq_puts(seq, "\n"); seq_printf(seq, "mps %d\n", ep->ep.maxpacket); seq_printf(seq, "total_data=%ld\n", ep->total_data); @@ -3346,7 +3348,7 @@ static int ep_show(struct seq_file *seq, void *v) list_for_each_entry(req, >queue, queue) { if (--show_limit < 0) { - seq_printf(seq, "not showing more requests...\n"); + seq_puts(seq, "not showing more requests...\n"); break; } -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Fix style in s3c-hsotg.c
Hi, On Wed, Sep 25, 2013 at 05:39:39PM +0200, Bartlomiej Zolnierkiewicz wrote: > Pavel, Felipe's remark was valid.. > > > > No, I'll not polish patch for hardware I don't have and have little > > > interest in. wanted to help you, but according to your first reply, > > > you do not really want help. > > > > that's your call. Now how about you stop being such a baby and go fix > > your mistakes to start with ? Just because I'm the maintainer of the > > gadget framework, doesn't mean I'm the maintainer of s3c-hsotc.c file. > > Maintainer != author too, btw. > > > > Anyway, I got much better stuff to do than babysitting grown ups. > > ..even if it could have been communicated in a gentler way. > > Anyway s3c-hsotg.c is a driver for our hardware so I'm going to pick this > patch up, polish it and then re-submit it later. Thanks. I tried a few times being nice and the response was always in the lines of "you should go fix the patch yourself if you don't like it" (not exactly these words, but same message). After some time of being nice and get nowhere, I just got fed up. cheers -- balbi signature.asc Description: Digital signature
Re: Fix style in s3c-hsotg.c
Hi, On Wednesday, September 18, 2013 09:11:06 AM Felipe Balbi wrote: > Hi, > > On Wed, Sep 18, 2013 at 11:20:27AM +0200, Pavel Machek wrote: > > On Tue 2013-09-17 20:45:39, Felipe Balbi wrote: > > > On Wed, Sep 18, 2013 at 12:04:27AM +0200, Pavel Machek wrote: > > > > On Tue 2013-09-17 10:42:30, Felipe Balbi wrote: > > > > > Hi, > > > > > > > > > > On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: > > > > > > Hi! > > > > > > > > > > > > checkpatch.pl has some valid complaints about style in s3c-hsotg.c : > > > > > > macro with if should be really enclosed in do {} while, and puts is > > > > > > going to be slightly faster. > > > > > > > > > > > > Here's suggested patch. I don't have the hardware, so it is > > > > > > completely > > > > > > untested. > > > > > > > > > > > > Signed-off-by: Pavel Machek, > > > > > > > > > > this is not how you send a patch, please read > > > > > Documentation/SubmittingPatches > > > > > > > > Have you considered possibility that this is how you nudge maintainer > > > > into fix their coding style? > > > > > > cute... > > > > > > Seriously though, read that file, you're commit log has garbage in it > > > which shouldn't go to git history. > > > > Run git log on SubmittingPatches. > > > > Then, instead of telling me what to read, run checkpatch on your > > files. You can either fix them yourself, or use my patch as a > > basis. Note there's missing } or something, so it probably will not > > compile, see the other mail. So you actually will have to modify that > > patch. Stripping "Hi!" from it should not be that hard, neither should > > be stripping note that patch is untested when you actually test > > it. And as you are the maintainer, it is your job. > > you misunderstand the work of maintainers. Our work is not to fix you > crappy patches. If we start allowing crappy patches, we'd be fixing your > nonsense forever and linux wouldn't move. Pavel, Felipe's remark was valid.. > > No, I'll not polish patch for hardware I don't have and have little > > interest in. wanted to help you, but according to your first reply, > > you do not really want help. > > that's your call. Now how about you stop being such a baby and go fix > your mistakes to start with ? Just because I'm the maintainer of the > gadget framework, doesn't mean I'm the maintainer of s3c-hsotc.c file. > Maintainer != author too, btw. > > Anyway, I got much better stuff to do than babysitting grown ups. ..even if it could have been communicated in a gentler way. Anyway s3c-hsotg.c is a driver for our hardware so I'm going to pick this patch up, polish it and then re-submit it later. Thanks. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Fix style in s3c-hsotg.c
Hi, On Wednesday, September 18, 2013 09:11:06 AM Felipe Balbi wrote: Hi, On Wed, Sep 18, 2013 at 11:20:27AM +0200, Pavel Machek wrote: On Tue 2013-09-17 20:45:39, Felipe Balbi wrote: On Wed, Sep 18, 2013 at 12:04:27AM +0200, Pavel Machek wrote: On Tue 2013-09-17 10:42:30, Felipe Balbi wrote: Hi, On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: Hi! checkpatch.pl has some valid complaints about style in s3c-hsotg.c : macro with if should be really enclosed in do {} while, and puts is going to be slightly faster. Here's suggested patch. I don't have the hardware, so it is completely untested. Signed-off-by: Pavel Machek, pa...@denx.de this is not how you send a patch, please read Documentation/SubmittingPatches Have you considered possibility that this is how you nudge maintainer into fix their coding style? cute... Seriously though, read that file, you're commit log has garbage in it which shouldn't go to git history. Run git log on SubmittingPatches. Then, instead of telling me what to read, run checkpatch on your files. You can either fix them yourself, or use my patch as a basis. Note there's missing } or something, so it probably will not compile, see the other mail. So you actually will have to modify that patch. Stripping Hi! from it should not be that hard, neither should be stripping note that patch is untested when you actually test it. And as you are the maintainer, it is your job. you misunderstand the work of maintainers. Our work is not to fix you crappy patches. If we start allowing crappy patches, we'd be fixing your nonsense forever and linux wouldn't move. Pavel, Felipe's remark was valid.. No, I'll not polish patch for hardware I don't have and have little interest in. wanted to help you, but according to your first reply, you do not really want help. that's your call. Now how about you stop being such a baby and go fix your mistakes to start with ? Just because I'm the maintainer of the gadget framework, doesn't mean I'm the maintainer of s3c-hsotc.c file. Maintainer != author too, btw. Anyway, I got much better stuff to do than babysitting grown ups. ..even if it could have been communicated in a gentler way. Anyway s3c-hsotg.c is a driver for our hardware so I'm going to pick this patch up, polish it and then re-submit it later. Thanks. Best regards, -- Bartlomiej Zolnierkiewicz Samsung RD Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Fix style in s3c-hsotg.c
Hi, On Wed, Sep 25, 2013 at 05:39:39PM +0200, Bartlomiej Zolnierkiewicz wrote: Pavel, Felipe's remark was valid.. No, I'll not polish patch for hardware I don't have and have little interest in. wanted to help you, but according to your first reply, you do not really want help. that's your call. Now how about you stop being such a baby and go fix your mistakes to start with ? Just because I'm the maintainer of the gadget framework, doesn't mean I'm the maintainer of s3c-hsotc.c file. Maintainer != author too, btw. Anyway, I got much better stuff to do than babysitting grown ups. ..even if it could have been communicated in a gentler way. Anyway s3c-hsotg.c is a driver for our hardware so I'm going to pick this patch up, polish it and then re-submit it later. Thanks. I tried a few times being nice and the response was always in the lines of you should go fix the patch yourself if you don't like it (not exactly these words, but same message). After some time of being nice and get nowhere, I just got fed up. cheers -- balbi signature.asc Description: Digital signature
Re: Fix style in s3c-hsotg.c
Hi! No, I'll not polish patch for hardware I don't have and have little interest in. wanted to help you, but according to your first reply, you do not really want help. that's your call. Now how about you stop being such a baby and go fix your mistakes to start with ? Just because I'm the maintainer of the gadget framework, doesn't mean I'm the maintainer of s3c-hsotc.c file. Maintainer != author too, btw. Anyway, I got much better stuff to do than babysitting grown ups. ..even if it could have been communicated in a gentler way. Anyway s3c-hsotg.c is a driver for our hardware so I'm going to pick this patch up, polish it and then re-submit it later. Thanks. Here's version that should compile. Thanks, Pavel Signed-off-by: Pavel Machek pa...@ucw.cz diff --git a/drivers/usb/gadget/s3c-hsotg.c b/drivers/usb/gadget/s3c-hsotg.c index d69b36a..c8cdde0 100644 --- a/drivers/usb/gadget/s3c-hsotg.c +++ b/drivers/usb/gadget/s3c-hsotg.c @@ -2092,12 +2092,14 @@ static void kill_all_requests(struct s3c_hsotg *hsotg, } #define call_gadget(_hs, _entry) \ +do { \ if ((_hs)-gadget.speed != USB_SPEED_UNKNOWN \ (_hs)-driver (_hs)-driver-_entry) { \ spin_unlock(_hs-lock); \ (_hs)-driver-_entry((_hs)-gadget); \ spin_lock(_hs-lock); \ - } + } \ +} while (0) /** * s3c_hsotg_disconnect - disconnect service @@ -3205,7 +3207,7 @@ static int state_show(struct seq_file *seq, void *v) readl(regs + GNPTXSTS), readl(regs + GRXSTSR)); - seq_printf(seq, \nEndpoint status:\n); + seq_puts(seq, \nEndpoint status:\n); for (idx = 0; idx 15; idx++) { u32 in, out; @@ -3222,7 +3224,7 @@ static int state_show(struct seq_file *seq, void *v) seq_printf(seq, , DIEPTSIZ=0x%08x, DOEPTSIZ=0x%08x, in, out); - seq_printf(seq, \n); + seq_puts(seq, \n); } return 0; @@ -3256,7 +3258,7 @@ static int fifo_show(struct seq_file *seq, void *v) u32 val; int idx; - seq_printf(seq, Non-periodic FIFOs:\n); + seq_puts(seq, Non-periodic FIFOs:\n); seq_printf(seq, RXFIFO: Size %d\n, readl(regs + GRXFSIZ)); val = readl(regs + GNPTXFSIZ); @@ -3264,7 +3266,7 @@ static int fifo_show(struct seq_file *seq, void *v) val GNPTXFSIZ_NPTxFDep_SHIFT, val GNPTXFSIZ_NPTxFStAddr_MASK); - seq_printf(seq, \nPeriodic TXFIFOs:\n); + seq_puts(seq, \nPeriodic TXFIFOs:\n); for (idx = 1; idx = 15; idx++) { val = readl(regs + DPTXFSIZn(idx)); @@ -3335,7 +3337,7 @@ static int ep_show(struct seq_file *seq, void *v) readl(regs + DIEPTSIZ(index)), readl(regs + DOEPTSIZ(index))); - seq_printf(seq, \n); + seq_puts(seq, \n); seq_printf(seq, mps %d\n, ep-ep.maxpacket); seq_printf(seq, total_data=%ld\n, ep-total_data); @@ -3346,7 +3348,7 @@ static int ep_show(struct seq_file *seq, void *v) list_for_each_entry(req, ep-queue, queue) { if (--show_limit 0) { - seq_printf(seq, not showing more requests...\n); + seq_puts(seq, not showing more requests...\n); break; } -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Fix style in s3c-hsotg.c
Hi! Pavel, Felipe's remark was valid.. No, I'll not polish patch for hardware I don't have and have little interest in. wanted to help you, but according to your first reply, you do not really want help. that's your call. Now how about you stop being such a baby and go fix your mistakes to start with ? Just because I'm the maintainer of the gadget framework, doesn't mean I'm the maintainer of s3c-hsotc.c file. Maintainer != author too, btw. Anyway, I got much better stuff to do than babysitting grown ups. ..even if it could have been communicated in a gentler way. Anyway s3c-hsotg.c is a driver for our hardware so I'm going to pick this patch up, polish it and then re-submit it later. Thanks. I tried a few times being nice and the response was always in the lines of you should go fix the patch yourself if you don't like it (not exactly these words, but same message). After some time of being nice and get nowhere, I just got fed up. For the record, this is Felipe's idea of being nice: this is not how you send a patch, please read Documentation/SubmittingPatches (What an useless reply. Yes, I know how to submit patches. I also know that patches need to be tested, so I was not interested in proper submission. I tried to explain the difference between submit patch for inclusion and make someone look at the patch and fix the code...) Have you considered possibility that this is how you nudge maintainer into fix their coding style? (Felipe) cute... Seriously though, read that file, you're commit log has garbage in it which shouldn't go to git history. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Fix style in s3c-hsotg.c
Hi, On Wed, Sep 18, 2013 at 11:20:27AM +0200, Pavel Machek wrote: > On Tue 2013-09-17 20:45:39, Felipe Balbi wrote: > > On Wed, Sep 18, 2013 at 12:04:27AM +0200, Pavel Machek wrote: > > > On Tue 2013-09-17 10:42:30, Felipe Balbi wrote: > > > > Hi, > > > > > > > > On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: > > > > > Hi! > > > > > > > > > > checkpatch.pl has some valid complaints about style in s3c-hsotg.c : > > > > > macro with if should be really enclosed in do {} while, and puts is > > > > > going to be slightly faster. > > > > > > > > > > Here's suggested patch. I don't have the hardware, so it is completely > > > > > untested. > > > > > > > > > > Signed-off-by: Pavel Machek, > > > > > > > > this is not how you send a patch, please read > > > > Documentation/SubmittingPatches > > > > > > Have you considered possibility that this is how you nudge maintainer > > > into fix their coding style? > > > > cute... > > > > Seriously though, read that file, you're commit log has garbage in it > > which shouldn't go to git history. > > Run git log on SubmittingPatches. > > Then, instead of telling me what to read, run checkpatch on your > files. You can either fix them yourself, or use my patch as a > basis. Note there's missing } or something, so it probably will not > compile, see the other mail. So you actually will have to modify that > patch. Stripping "Hi!" from it should not be that hard, neither should > be stripping note that patch is untested when you actually test > it. And as you are the maintainer, it is your job. you misunderstand the work of maintainers. Our work is not to fix you crappy patches. If we start allowing crappy patches, we'd be fixing your nonsense forever and linux wouldn't move. > No, I'll not polish patch for hardware I don't have and have little > interest in. wanted to help you, but according to your first reply, > you do not really want help. that's your call. Now how about you stop being such a baby and go fix your mistakes to start with ? Just because I'm the maintainer of the gadget framework, doesn't mean I'm the maintainer of s3c-hsotc.c file. Maintainer != author too, btw. Anyway, I got much better stuff to do than babysitting grown ups. cheers -- balbi signature.asc Description: Digital signature
Re: Fix style in s3c-hsotg.c
On Tue 2013-09-17 20:45:39, Felipe Balbi wrote: > On Wed, Sep 18, 2013 at 12:04:27AM +0200, Pavel Machek wrote: > > On Tue 2013-09-17 10:42:30, Felipe Balbi wrote: > > > Hi, > > > > > > On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: > > > > Hi! > > > > > > > > checkpatch.pl has some valid complaints about style in s3c-hsotg.c : > > > > macro with if should be really enclosed in do {} while, and puts is > > > > going to be slightly faster. > > > > > > > > Here's suggested patch. I don't have the hardware, so it is completely > > > > untested. > > > > > > > > Signed-off-by: Pavel Machek, > > > > > > this is not how you send a patch, please read > > > Documentation/SubmittingPatches > > > > Have you considered possibility that this is how you nudge maintainer > > into fix their coding style? > > cute... > > Seriously though, read that file, you're commit log has garbage in it > which shouldn't go to git history. Run git log on SubmittingPatches. Then, instead of telling me what to read, run checkpatch on your files. You can either fix them yourself, or use my patch as a basis. Note there's missing } or something, so it probably will not compile, see the other mail. So you actually will have to modify that patch. Stripping "Hi!" from it should not be that hard, neither should be stripping note that patch is untested when you actually test it. And as you are the maintainer, it is your job. No, I'll not polish patch for hardware I don't have and have little interest in. wanted to help you, but according to your first reply, you do not really want help. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Fix style in s3c-hsotg.c
On Tue 2013-09-17 20:45:39, Felipe Balbi wrote: On Wed, Sep 18, 2013 at 12:04:27AM +0200, Pavel Machek wrote: On Tue 2013-09-17 10:42:30, Felipe Balbi wrote: Hi, On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: Hi! checkpatch.pl has some valid complaints about style in s3c-hsotg.c : macro with if should be really enclosed in do {} while, and puts is going to be slightly faster. Here's suggested patch. I don't have the hardware, so it is completely untested. Signed-off-by: Pavel Machek, pa...@denx.de this is not how you send a patch, please read Documentation/SubmittingPatches Have you considered possibility that this is how you nudge maintainer into fix their coding style? cute... Seriously though, read that file, you're commit log has garbage in it which shouldn't go to git history. Run git log on SubmittingPatches. Then, instead of telling me what to read, run checkpatch on your files. You can either fix them yourself, or use my patch as a basis. Note there's missing } or something, so it probably will not compile, see the other mail. So you actually will have to modify that patch. Stripping Hi! from it should not be that hard, neither should be stripping note that patch is untested when you actually test it. And as you are the maintainer, it is your job. No, I'll not polish patch for hardware I don't have and have little interest in. wanted to help you, but according to your first reply, you do not really want help. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Fix style in s3c-hsotg.c
Hi, On Wed, Sep 18, 2013 at 11:20:27AM +0200, Pavel Machek wrote: On Tue 2013-09-17 20:45:39, Felipe Balbi wrote: On Wed, Sep 18, 2013 at 12:04:27AM +0200, Pavel Machek wrote: On Tue 2013-09-17 10:42:30, Felipe Balbi wrote: Hi, On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: Hi! checkpatch.pl has some valid complaints about style in s3c-hsotg.c : macro with if should be really enclosed in do {} while, and puts is going to be slightly faster. Here's suggested patch. I don't have the hardware, so it is completely untested. Signed-off-by: Pavel Machek, pa...@denx.de this is not how you send a patch, please read Documentation/SubmittingPatches Have you considered possibility that this is how you nudge maintainer into fix their coding style? cute... Seriously though, read that file, you're commit log has garbage in it which shouldn't go to git history. Run git log on SubmittingPatches. Then, instead of telling me what to read, run checkpatch on your files. You can either fix them yourself, or use my patch as a basis. Note there's missing } or something, so it probably will not compile, see the other mail. So you actually will have to modify that patch. Stripping Hi! from it should not be that hard, neither should be stripping note that patch is untested when you actually test it. And as you are the maintainer, it is your job. you misunderstand the work of maintainers. Our work is not to fix you crappy patches. If we start allowing crappy patches, we'd be fixing your nonsense forever and linux wouldn't move. No, I'll not polish patch for hardware I don't have and have little interest in. wanted to help you, but according to your first reply, you do not really want help. that's your call. Now how about you stop being such a baby and go fix your mistakes to start with ? Just because I'm the maintainer of the gadget framework, doesn't mean I'm the maintainer of s3c-hsotc.c file. Maintainer != author too, btw. Anyway, I got much better stuff to do than babysitting grown ups. cheers -- balbi signature.asc Description: Digital signature
Re: Fix style in s3c-hsotg.c
On Wed, Sep 18, 2013 at 12:04:27AM +0200, Pavel Machek wrote: > On Tue 2013-09-17 10:42:30, Felipe Balbi wrote: > > Hi, > > > > On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: > > > Hi! > > > > > > checkpatch.pl has some valid complaints about style in s3c-hsotg.c : > > > macro with if should be really enclosed in do {} while, and puts is > > > going to be slightly faster. > > > > > > Here's suggested patch. I don't have the hardware, so it is completely > > > untested. > > > > > > Signed-off-by: Pavel Machek, > > > > this is not how you send a patch, please read > > Documentation/SubmittingPatches > > Have you considered possibility that this is how you nudge maintainer > into fix their coding style? cute... Seriously though, read that file, you're commit log has garbage in it which shouldn't go to git history. -- balbi signature.asc Description: Digital signature
Re: Fix style in s3c-hsotg.c
On Tue 2013-09-17 10:42:30, Felipe Balbi wrote: > Hi, > > On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: > > Hi! > > > > checkpatch.pl has some valid complaints about style in s3c-hsotg.c : > > macro with if should be really enclosed in do {} while, and puts is > > going to be slightly faster. > > > > Here's suggested patch. I don't have the hardware, so it is completely > > untested. > > > > Signed-off-by: Pavel Machek, > > this is not how you send a patch, please read > Documentation/SubmittingPatches Have you considered possibility that this is how you nudge maintainer into fix their coding style? Thanks, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Fix style in s3c-hsotg.c
Hi, On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: > Hi! > > checkpatch.pl has some valid complaints about style in s3c-hsotg.c : > macro with if should be really enclosed in do {} while, and puts is > going to be slightly faster. > > Here's suggested patch. I don't have the hardware, so it is completely > untested. > > Signed-off-by: Pavel Machek, this is not how you send a patch, please read Documentation/SubmittingPatches -- balbi signature.asc Description: Digital signature
Re: Fix style in s3c-hsotg.c
Hi, On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: Hi! checkpatch.pl has some valid complaints about style in s3c-hsotg.c : macro with if should be really enclosed in do {} while, and puts is going to be slightly faster. Here's suggested patch. I don't have the hardware, so it is completely untested. Signed-off-by: Pavel Machek, pa...@denx.de this is not how you send a patch, please read Documentation/SubmittingPatches -- balbi signature.asc Description: Digital signature
Re: Fix style in s3c-hsotg.c
On Tue 2013-09-17 10:42:30, Felipe Balbi wrote: Hi, On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: Hi! checkpatch.pl has some valid complaints about style in s3c-hsotg.c : macro with if should be really enclosed in do {} while, and puts is going to be slightly faster. Here's suggested patch. I don't have the hardware, so it is completely untested. Signed-off-by: Pavel Machek, pa...@denx.de this is not how you send a patch, please read Documentation/SubmittingPatches Have you considered possibility that this is how you nudge maintainer into fix their coding style? Thanks, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Fix style in s3c-hsotg.c
On Wed, Sep 18, 2013 at 12:04:27AM +0200, Pavel Machek wrote: On Tue 2013-09-17 10:42:30, Felipe Balbi wrote: Hi, On Mon, Sep 02, 2013 at 03:58:32PM +0200, Pavel Machek wrote: Hi! checkpatch.pl has some valid complaints about style in s3c-hsotg.c : macro with if should be really enclosed in do {} while, and puts is going to be slightly faster. Here's suggested patch. I don't have the hardware, so it is completely untested. Signed-off-by: Pavel Machek, pa...@denx.de this is not how you send a patch, please read Documentation/SubmittingPatches Have you considered possibility that this is how you nudge maintainer into fix their coding style? cute... Seriously though, read that file, you're commit log has garbage in it which shouldn't go to git history. -- balbi signature.asc Description: Digital signature
Re: Fix style in s3c-hsotg.c
Hello. On 09/02/2013 05:58 PM, Pavel Machek wrote: checkpatch.pl has some valid complaints about style in s3c-hsotg.c : macro with if should be really enclosed in do {} while, and puts is going to be slightly faster. Here's suggested patch. I don't have the hardware, so it is completely untested. Signed-off-by: Pavel Machek, diff --git a/drivers/usb/gadget/s3c-hsotg.c b/drivers/usb/gadget/s3c-hsotg.c index af22f24..f8e762a 100644 --- a/drivers/usb/gadget/s3c-hsotg.c +++ b/drivers/usb/gadget/s3c-hsotg.c @@ -2091,12 +2091,14 @@ static void kill_all_requests(struct s3c_hsotg *hsotg, } #define call_gadget(_hs, _entry) \ +do { \ if ((_hs)->gadget.speed != USB_SPEED_UNKNOWN && \ (_hs)->driver && (_hs)->driver->_entry) { \ spin_unlock(&_hs->lock); \ (_hs)->driver->_entry(&(_hs)->gadget); \ spin_lock(&_hs->lock); \ } Indentation also needs fixing on this line, maybe in another patch... +} while (0) WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: Fix style in s3c-hsotg.c
Hello. On 09/02/2013 05:58 PM, Pavel Machek wrote: checkpatch.pl has some valid complaints about style in s3c-hsotg.c : macro with if should be really enclosed in do {} while, and puts is going to be slightly faster. Here's suggested patch. I don't have the hardware, so it is completely untested. Signed-off-by: Pavel Machek, pa...@denx.de diff --git a/drivers/usb/gadget/s3c-hsotg.c b/drivers/usb/gadget/s3c-hsotg.c index af22f24..f8e762a 100644 --- a/drivers/usb/gadget/s3c-hsotg.c +++ b/drivers/usb/gadget/s3c-hsotg.c @@ -2091,12 +2091,14 @@ static void kill_all_requests(struct s3c_hsotg *hsotg, } #define call_gadget(_hs, _entry) \ +do { \ if ((_hs)-gadget.speed != USB_SPEED_UNKNOWN \ (_hs)-driver (_hs)-driver-_entry) { \ spin_unlock(_hs-lock); \ (_hs)-driver-_entry((_hs)-gadget); \ spin_lock(_hs-lock); \ } Indentation also needs fixing on this line, maybe in another patch... +} while (0) WBR, Sergei -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/