Re: [PATCH] Removes FIXME message in usb.c

2014-07-09 Thread Sergei Shtylyov 

Hello.

On 07/09/2014 05:58 AM, Nicholas Krause wrote:


This patch removes a fixme message in this file:wq for setting the usb 2


   The vim's commands interspersed with text? :-)


speed on the board to the correct level. We need to depend on the
bootloader for doing this as the wires may be shared for the other
things on the board with the usb chipset.



Signed-off-by: Nicholas Krause xerofo...@gmail.com

[...]

WBR, Sergei

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH] Removes FIXME message in usb.c

2014-07-09 Thread Nicholas Krause 
This patch removes a fixme message in this file for setting the usb 2
speed on the board to the correct level. We need to depend on the
bootloader for doing this as the wires may be shared for the other
things on the board with the usb chipset.

Signed-off-by: Nicholas Krause xerofo...@gmail.com
---
 arch/arm/mach-omap1/usb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-omap1/usb.c b/arch/arm/mach-omap1/usb.c
index 4118db5..172245a 100644
--- a/arch/arm/mach-omap1/usb.c
+++ b/arch/arm/mach-omap1/usb.c
@@ -505,7 +505,7 @@ static u32 __init omap1_usb2_init(unsigned nwires, unsigned 
alt_pingroup)
omap_cfg_reg(W5_USB2_SE0);
if (nwires != 3)
omap_cfg_reg(Y5_USB2_RCV);
-   // FIXME omap_cfg_reg(USB2_SPEED);
+   /* Depend on boatloader for USB speed to be stated for board */
} else {
pr_debug(usb%d cpu unrecognized\n, 1);
return 0;
-- 
1.9.1

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Removes FIXME message in usb.c

2014-07-09 Thread Greg KH 
Ok, this has been fun to watch on lkml for a while now, but really,
please, just stop doing this.  Randomly searching the kernel source for
FIXME lines and just commenting them out, isn't ok.  Almost always,
those lines are there because the original developer really doesn't know
how else to resolve the issue.

So, if the domain-specific-author doesn't have an idea of what to do,
how does someone who is brand-new to the code know better?

If you are looking for a task to do in the kernel, try
drivers/staging/*/TODO for a list.  Or look at the kernel janitor's list
on kernelnewbies.org.  Or try running the kernel and finding something
that is broken for you and fixing that.  Any of those would be better
than randomly deleting FIXME lines.  By doing that, you are just wasting
maintainer's time.  Which is the resource we have the least of at the
moment.

thanks,

greg k-h
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Removes FIXME message in usb.c

2014-07-09 Thread Nick Krause 
Greg KH wrote on July 9th at 1:19 P.M.
Ok, this has been fun to watch on lkml for a while now, but really,
please, just stop doing this.  Randomly searching the kernel source for
FIXME lines and just commenting them out, isn't ok.  Almost always,
those lines are there because the original developer really doesn't know
how else to resolve the issue.

So, if the domain-specific-author doesn't have an idea of what to do,
how does someone who is brand-new to the code know better?

If you are looking for a task to do in the kernel, try
drivers/staging/*/TODO for a list.  Or look at the kernel janitor's list
on kernelnewbies.org.  Or try running the kernel and finding something
that is broken for you and fixing that.  Any of those would be better
than randomly deleting FIXME lines.  By doing that, you are just wasting
maintainer's time.  Which is the resource we have the least of at the
moment.

thanks,

greg k-h



Greg ,

I sent in another patch which enables omap_cfg_reg(USB2_SPEED) as
stated in the fix me.
The maintainers of this file stated to me this was dangerous due to
the wires perhaps being
shared with other parts of the board and we should depend on the boot
loader to set this.
The maintainers decided to remove this fix me not me based on their
better understanding
of this file.

Cheers Nick

On Wed, Jul 9, 2014 at 1:19 PM, Greg KH g...@kroah.com wrote:
 Ok, this has been fun to watch on lkml for a while now, but really,
 please, just stop doing this.  Randomly searching the kernel source for
 FIXME lines and just commenting them out, isn't ok.  Almost always,
 those lines are there because the original developer really doesn't know
 how else to resolve the issue.

 So, if the domain-specific-author doesn't have an idea of what to do,
 how does someone who is brand-new to the code know better?

 If you are looking for a task to do in the kernel, try
 drivers/staging/*/TODO for a list.  Or look at the kernel janitor's list
 on kernelnewbies.org.  Or try running the kernel and finding something
 that is broken for you and fixing that.  Any of those would be better
 than randomly deleting FIXME lines.  By doing that, you are just wasting
 maintainer's time.  Which is the resource we have the least of at the
 moment.

 thanks,

 greg k-h
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH] Removes FIXME message in usb.c

2014-07-08 Thread Nicholas Krause 
This patch removes a fixme message in this file:wq for setting the usb 2
speed on the board to the correct level. We need to depend on the
bootloader for doing this as the wires may be shared for the other
things on the board with the usb chipset.

Signed-off-by: Nicholas Krause xerofo...@gmail.com
---
 arch/arm/mach-omap1/usb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-omap1/usb.c b/arch/arm/mach-omap1/usb.c
index 4118db5..172245a 100644
--- a/arch/arm/mach-omap1/usb.c
+++ b/arch/arm/mach-omap1/usb.c
@@ -505,7 +505,7 @@ static u32 __init omap1_usb2_init(unsigned nwires, unsigned 
alt_pingroup)
omap_cfg_reg(W5_USB2_SE0);
if (nwires != 3)
omap_cfg_reg(Y5_USB2_RCV);
-   // FIXME omap_cfg_reg(USB2_SPEED);
+   /* Depend on boatloader for USB speed to be stated for board */
} else {
pr_debug(usb%d cpu unrecognized\n, 1);
return 0;
-- 
1.9.1

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Removes FIXME message in usb.c

2014-07-08 Thread Greg KH 
On Tue, Jul 08, 2014 at 09:58:39PM -0400, Nicholas Krause wrote:
 This patch removes a fixme message in this file:wq for setting the usb 2

I don't think you did this correctly :)

 speed on the board to the correct level. We need to depend on the
 bootloader for doing this as the wires may be shared for the other
 things on the board with the usb chipset.
 
 Signed-off-by: Nicholas Krause xerofo...@gmail.com
 ---
  arch/arm/mach-omap1/usb.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/arch/arm/mach-omap1/usb.c b/arch/arm/mach-omap1/usb.c
 index 4118db5..172245a 100644
 --- a/arch/arm/mach-omap1/usb.c
 +++ b/arch/arm/mach-omap1/usb.c
 @@ -505,7 +505,7 @@ static u32 __init omap1_usb2_init(unsigned nwires, 
 unsigned alt_pingroup)
   omap_cfg_reg(W5_USB2_SE0);
   if (nwires != 3)
   omap_cfg_reg(Y5_USB2_RCV);
 - // FIXME omap_cfg_reg(USB2_SPEED);
 + /* Depend on boatloader for USB speed to be stated for board */

Really?  Is that correct?  WHy?

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html