Re: [PATCH] usb: phy: Quiet unable to find transceiver message
On Mon, 2014-02-03 at 11:04 -0500, Alan Stern wrote: > On Mon, 3 Feb 2014, Paul Bolle wrote: > > This message cab still be seen when booting v3.14-rc1. Is a patch to > > downgrade this message to dev_dbg() - from Josh, Felipe or someone else > > - queued somewhere? > > http://marc.info/?l=linux-usb=139092084714232=2 > > As far as I know, this hasn't been merged into anybody's tree yet. And > Felipe is on vacation for a few weeks. Still, it should get in before > 3.14 is released. Thanks. I'll add it to my local stack of patches, and if nothing is merged in a few weeks I might send a short reminder. Paul Bolle -- 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: [PATCH] usb: phy: Quiet unable to find transceiver message
On Mon, 3 Feb 2014, Paul Bolle wrote: > > > >> --- a/drivers/usb/phy/phy.c > > > >> +++ b/drivers/usb/phy/phy.c > > > >> @@ -228,7 +228,7 @@ struct usb_phy *usb_get_phy_dev(struct device > > > >> *dev, u8 index) > > > >> > > > >> phy = __usb_find_phy_dev(dev, _bind_list, index); > > > >> if (IS_ERR(phy) || !try_module_get(phy->dev->driver->owner)) { > > > >> - pr_err("unable to find transceiver\n"); > > > >> + pr_debug("unable to find transceiver\n"); > > > >> goto err0; > > > >> } > > > > > > > > Wouldn't it make more sense to change this to dev_debug? As it stands, > > > > the user has no idea which device is lacking a transceiver. > > > > > > Quite possibly, yes. I'm not overly familiar with the subsystem and > > > was just writing up what Felipe suggested. > > > > > > > (The same is probably true for other log messages in this source file.) > > > > > > I don't disagree, but I'd rather someone with more experience in the > > > USB subsystem do that kind of broader audit/change. I'd be happy to > > > test. > > > > yeah, I just sent a patch where I forgot to switch over to dev_dbg(), if > > you can do that for both messages and remove the out of memory message, > > I'd be glad to take your patch instead of mine. > > This message cab still be seen when booting v3.14-rc1. Is a patch to > downgrade this message to dev_dbg() - from Josh, Felipe or someone else > - queued somewhere? http://marc.info/?l=linux-usb=139092084714232=2 As far as I know, this hasn't been merged into anybody's tree yet. And Felipe is on vacation for a few weeks. Still, it should get in before 3.14 is released. Alan Stern -- 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: [PATCH] usb: phy: Quiet unable to find transceiver message
Felipe Balbi schreef op ma 27-01-2014 om 09:30 [-0600]: > On Sat, Jan 25, 2014 at 03:24:55PM -0500, Josh Boyer wrote: > > On Sat, Jan 25, 2014 at 10:37 AM, Alan Stern > > wrote: > > > On Sat, 25 Jan 2014, Josh Boyer wrote: > > > > > >> commit 1ae5799ef6317 ("usb: hcd: Initialize USB phy if needed") allows > > >> the USB layer to initialize external PHYs if needed. However, a PHY is > > >> not needed in all cases. The usb_get_phy_device function will print (Minor nit: that should have been "usb_get_phy_dev".) > > >> an error message, "unable to find transceiver" but everything still > > >> functions normally. > > >> > > >> Drop the severity of this message to pr_debug. > > >> > > >> Signed-off-by: Josh Boyer > > >> --- > > >> drivers/usb/phy/phy.c | 2 +- > > >> 1 file changed, 1 insertion(+), 1 deletion(-) > > >> > > >> diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c > > >> index e6f61e4..c7fe880 100644 > > >> --- a/drivers/usb/phy/phy.c > > >> +++ b/drivers/usb/phy/phy.c > > >> @@ -228,7 +228,7 @@ struct usb_phy *usb_get_phy_dev(struct device *dev, > > >> u8 index) > > >> > > >> phy = __usb_find_phy_dev(dev, _bind_list, index); > > >> if (IS_ERR(phy) || !try_module_get(phy->dev->driver->owner)) { > > >> - pr_err("unable to find transceiver\n"); > > >> + pr_debug("unable to find transceiver\n"); > > >> goto err0; > > >> } > > > > > > Wouldn't it make more sense to change this to dev_debug? As it stands, > > > the user has no idea which device is lacking a transceiver. > > > > Quite possibly, yes. I'm not overly familiar with the subsystem and > > was just writing up what Felipe suggested. > > > > > (The same is probably true for other log messages in this source file.) > > > > I don't disagree, but I'd rather someone with more experience in the > > USB subsystem do that kind of broader audit/change. I'd be happy to > > test. > > yeah, I just sent a patch where I forgot to switch over to dev_dbg(), if > you can do that for both messages and remove the out of memory message, > I'd be glad to take your patch instead of mine. This message cab still be seen when booting v3.14-rc1. Is a patch to downgrade this message to dev_dbg() - from Josh, Felipe or someone else - queued somewhere? Paul Bolle -- 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: [PATCH] usb: phy: Quiet unable to find transceiver message
Felipe Balbi schreef op ma 27-01-2014 om 09:30 [-0600]: On Sat, Jan 25, 2014 at 03:24:55PM -0500, Josh Boyer wrote: On Sat, Jan 25, 2014 at 10:37 AM, Alan Stern st...@rowland.harvard.edu wrote: On Sat, 25 Jan 2014, Josh Boyer wrote: commit 1ae5799ef6317 (usb: hcd: Initialize USB phy if needed) allows the USB layer to initialize external PHYs if needed. However, a PHY is not needed in all cases. The usb_get_phy_device function will print (Minor nit: that should have been usb_get_phy_dev.) an error message, unable to find transceiver but everything still functions normally. Drop the severity of this message to pr_debug. Signed-off-by: Josh Boyer jwbo...@fedoraproject.org --- drivers/usb/phy/phy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c index e6f61e4..c7fe880 100644 --- a/drivers/usb/phy/phy.c +++ b/drivers/usb/phy/phy.c @@ -228,7 +228,7 @@ struct usb_phy *usb_get_phy_dev(struct device *dev, u8 index) phy = __usb_find_phy_dev(dev, phy_bind_list, index); if (IS_ERR(phy) || !try_module_get(phy-dev-driver-owner)) { - pr_err(unable to find transceiver\n); + pr_debug(unable to find transceiver\n); goto err0; } Wouldn't it make more sense to change this to dev_debug? As it stands, the user has no idea which device is lacking a transceiver. Quite possibly, yes. I'm not overly familiar with the subsystem and was just writing up what Felipe suggested. (The same is probably true for other log messages in this source file.) I don't disagree, but I'd rather someone with more experience in the USB subsystem do that kind of broader audit/change. I'd be happy to test. yeah, I just sent a patch where I forgot to switch over to dev_dbg(), if you can do that for both messages and remove the out of memory message, I'd be glad to take your patch instead of mine. This message cab still be seen when booting v3.14-rc1. Is a patch to downgrade this message to dev_dbg() - from Josh, Felipe or someone else - queued somewhere? Paul Bolle -- 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: [PATCH] usb: phy: Quiet unable to find transceiver message
On Mon, 3 Feb 2014, Paul Bolle wrote: --- a/drivers/usb/phy/phy.c +++ b/drivers/usb/phy/phy.c @@ -228,7 +228,7 @@ struct usb_phy *usb_get_phy_dev(struct device *dev, u8 index) phy = __usb_find_phy_dev(dev, phy_bind_list, index); if (IS_ERR(phy) || !try_module_get(phy-dev-driver-owner)) { - pr_err(unable to find transceiver\n); + pr_debug(unable to find transceiver\n); goto err0; } Wouldn't it make more sense to change this to dev_debug? As it stands, the user has no idea which device is lacking a transceiver. Quite possibly, yes. I'm not overly familiar with the subsystem and was just writing up what Felipe suggested. (The same is probably true for other log messages in this source file.) I don't disagree, but I'd rather someone with more experience in the USB subsystem do that kind of broader audit/change. I'd be happy to test. yeah, I just sent a patch where I forgot to switch over to dev_dbg(), if you can do that for both messages and remove the out of memory message, I'd be glad to take your patch instead of mine. This message cab still be seen when booting v3.14-rc1. Is a patch to downgrade this message to dev_dbg() - from Josh, Felipe or someone else - queued somewhere? http://marc.info/?l=linux-usbm=139092084714232w=2 As far as I know, this hasn't been merged into anybody's tree yet. And Felipe is on vacation for a few weeks. Still, it should get in before 3.14 is released. Alan Stern -- 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: [PATCH] usb: phy: Quiet unable to find transceiver message
On Mon, 2014-02-03 at 11:04 -0500, Alan Stern wrote: On Mon, 3 Feb 2014, Paul Bolle wrote: This message cab still be seen when booting v3.14-rc1. Is a patch to downgrade this message to dev_dbg() - from Josh, Felipe or someone else - queued somewhere? http://marc.info/?l=linux-usbm=139092084714232w=2 As far as I know, this hasn't been merged into anybody's tree yet. And Felipe is on vacation for a few weeks. Still, it should get in before 3.14 is released. Thanks. I'll add it to my local stack of patches, and if nothing is merged in a few weeks I might send a short reminder. Paul Bolle -- 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: [PATCH] usb: phy: Quiet unable to find transceiver message
On Sat, Jan 25, 2014 at 03:24:55PM -0500, Josh Boyer wrote: > On Sat, Jan 25, 2014 at 10:37 AM, Alan Stern > wrote: > > On Sat, 25 Jan 2014, Josh Boyer wrote: > > > >> commit 1ae5799ef6317 ("usb: hcd: Initialize USB phy if needed") allows > >> the USB layer to initialize external PHYs if needed. However, a PHY is > >> not needed in all cases. The usb_get_phy_device function will print > >> an error message, "unable to find transceiver" but everything still > >> functions normally. > >> > >> Drop the severity of this message to pr_debug. > >> > >> Signed-off-by: Josh Boyer > >> --- > >> drivers/usb/phy/phy.c | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c > >> index e6f61e4..c7fe880 100644 > >> --- a/drivers/usb/phy/phy.c > >> +++ b/drivers/usb/phy/phy.c > >> @@ -228,7 +228,7 @@ struct usb_phy *usb_get_phy_dev(struct device *dev, u8 > >> index) > >> > >> phy = __usb_find_phy_dev(dev, _bind_list, index); > >> if (IS_ERR(phy) || !try_module_get(phy->dev->driver->owner)) { > >> - pr_err("unable to find transceiver\n"); > >> + pr_debug("unable to find transceiver\n"); > >> goto err0; > >> } > > > > Wouldn't it make more sense to change this to dev_debug? As it stands, > > the user has no idea which device is lacking a transceiver. > > Quite possibly, yes. I'm not overly familiar with the subsystem and > was just writing up what Felipe suggested. > > > (The same is probably true for other log messages in this source file.) > > I don't disagree, but I'd rather someone with more experience in the > USB subsystem do that kind of broader audit/change. I'd be happy to > test. yeah, I just sent a patch where I forgot to switch over to dev_dbg(), if you can do that for both messages and remove the out of memory message, I'd be glad to take your patch instead of mine. cheers -- balbi signature.asc Description: Digital signature
Re: [PATCH] usb: phy: Quiet unable to find transceiver message
On Sat, Jan 25, 2014 at 03:24:55PM -0500, Josh Boyer wrote: On Sat, Jan 25, 2014 at 10:37 AM, Alan Stern st...@rowland.harvard.edu wrote: On Sat, 25 Jan 2014, Josh Boyer wrote: commit 1ae5799ef6317 (usb: hcd: Initialize USB phy if needed) allows the USB layer to initialize external PHYs if needed. However, a PHY is not needed in all cases. The usb_get_phy_device function will print an error message, unable to find transceiver but everything still functions normally. Drop the severity of this message to pr_debug. Signed-off-by: Josh Boyer jwbo...@fedoraproject.org --- drivers/usb/phy/phy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c index e6f61e4..c7fe880 100644 --- a/drivers/usb/phy/phy.c +++ b/drivers/usb/phy/phy.c @@ -228,7 +228,7 @@ struct usb_phy *usb_get_phy_dev(struct device *dev, u8 index) phy = __usb_find_phy_dev(dev, phy_bind_list, index); if (IS_ERR(phy) || !try_module_get(phy-dev-driver-owner)) { - pr_err(unable to find transceiver\n); + pr_debug(unable to find transceiver\n); goto err0; } Wouldn't it make more sense to change this to dev_debug? As it stands, the user has no idea which device is lacking a transceiver. Quite possibly, yes. I'm not overly familiar with the subsystem and was just writing up what Felipe suggested. (The same is probably true for other log messages in this source file.) I don't disagree, but I'd rather someone with more experience in the USB subsystem do that kind of broader audit/change. I'd be happy to test. yeah, I just sent a patch where I forgot to switch over to dev_dbg(), if you can do that for both messages and remove the out of memory message, I'd be glad to take your patch instead of mine. cheers -- balbi signature.asc Description: Digital signature
Re: [PATCH] usb: phy: Quiet unable to find transceiver message
On Sat, Jan 25, 2014 at 10:37 AM, Alan Stern wrote: > On Sat, 25 Jan 2014, Josh Boyer wrote: > >> commit 1ae5799ef6317 ("usb: hcd: Initialize USB phy if needed") allows >> the USB layer to initialize external PHYs if needed. However, a PHY is >> not needed in all cases. The usb_get_phy_device function will print >> an error message, "unable to find transceiver" but everything still >> functions normally. >> >> Drop the severity of this message to pr_debug. >> >> Signed-off-by: Josh Boyer >> --- >> drivers/usb/phy/phy.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c >> index e6f61e4..c7fe880 100644 >> --- a/drivers/usb/phy/phy.c >> +++ b/drivers/usb/phy/phy.c >> @@ -228,7 +228,7 @@ struct usb_phy *usb_get_phy_dev(struct device *dev, u8 >> index) >> >> phy = __usb_find_phy_dev(dev, _bind_list, index); >> if (IS_ERR(phy) || !try_module_get(phy->dev->driver->owner)) { >> - pr_err("unable to find transceiver\n"); >> + pr_debug("unable to find transceiver\n"); >> goto err0; >> } > > Wouldn't it make more sense to change this to dev_debug? As it stands, > the user has no idea which device is lacking a transceiver. Quite possibly, yes. I'm not overly familiar with the subsystem and was just writing up what Felipe suggested. > (The same is probably true for other log messages in this source file.) I don't disagree, but I'd rather someone with more experience in the USB subsystem do that kind of broader audit/change. I'd be happy to test. josh -- 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: [PATCH] usb: phy: Quiet unable to find transceiver message
On Sat, 25 Jan 2014, Josh Boyer wrote: > commit 1ae5799ef6317 ("usb: hcd: Initialize USB phy if needed") allows > the USB layer to initialize external PHYs if needed. However, a PHY is > not needed in all cases. The usb_get_phy_device function will print > an error message, "unable to find transceiver" but everything still > functions normally. > > Drop the severity of this message to pr_debug. > > Signed-off-by: Josh Boyer > --- > drivers/usb/phy/phy.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c > index e6f61e4..c7fe880 100644 > --- a/drivers/usb/phy/phy.c > +++ b/drivers/usb/phy/phy.c > @@ -228,7 +228,7 @@ struct usb_phy *usb_get_phy_dev(struct device *dev, u8 > index) > > phy = __usb_find_phy_dev(dev, _bind_list, index); > if (IS_ERR(phy) || !try_module_get(phy->dev->driver->owner)) { > - pr_err("unable to find transceiver\n"); > + pr_debug("unable to find transceiver\n"); > goto err0; > } Wouldn't it make more sense to change this to dev_debug? As it stands, the user has no idea which device is lacking a transceiver. (The same is probably true for other log messages in this source file.) Alan Stern -- 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: [PATCH] usb: phy: Quiet unable to find transceiver message
On Sat, 25 Jan 2014, Josh Boyer wrote: commit 1ae5799ef6317 (usb: hcd: Initialize USB phy if needed) allows the USB layer to initialize external PHYs if needed. However, a PHY is not needed in all cases. The usb_get_phy_device function will print an error message, unable to find transceiver but everything still functions normally. Drop the severity of this message to pr_debug. Signed-off-by: Josh Boyer jwbo...@fedoraproject.org --- drivers/usb/phy/phy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c index e6f61e4..c7fe880 100644 --- a/drivers/usb/phy/phy.c +++ b/drivers/usb/phy/phy.c @@ -228,7 +228,7 @@ struct usb_phy *usb_get_phy_dev(struct device *dev, u8 index) phy = __usb_find_phy_dev(dev, phy_bind_list, index); if (IS_ERR(phy) || !try_module_get(phy-dev-driver-owner)) { - pr_err(unable to find transceiver\n); + pr_debug(unable to find transceiver\n); goto err0; } Wouldn't it make more sense to change this to dev_debug? As it stands, the user has no idea which device is lacking a transceiver. (The same is probably true for other log messages in this source file.) Alan Stern -- 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: [PATCH] usb: phy: Quiet unable to find transceiver message
On Sat, Jan 25, 2014 at 10:37 AM, Alan Stern st...@rowland.harvard.edu wrote: On Sat, 25 Jan 2014, Josh Boyer wrote: commit 1ae5799ef6317 (usb: hcd: Initialize USB phy if needed) allows the USB layer to initialize external PHYs if needed. However, a PHY is not needed in all cases. The usb_get_phy_device function will print an error message, unable to find transceiver but everything still functions normally. Drop the severity of this message to pr_debug. Signed-off-by: Josh Boyer jwbo...@fedoraproject.org --- drivers/usb/phy/phy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c index e6f61e4..c7fe880 100644 --- a/drivers/usb/phy/phy.c +++ b/drivers/usb/phy/phy.c @@ -228,7 +228,7 @@ struct usb_phy *usb_get_phy_dev(struct device *dev, u8 index) phy = __usb_find_phy_dev(dev, phy_bind_list, index); if (IS_ERR(phy) || !try_module_get(phy-dev-driver-owner)) { - pr_err(unable to find transceiver\n); + pr_debug(unable to find transceiver\n); goto err0; } Wouldn't it make more sense to change this to dev_debug? As it stands, the user has no idea which device is lacking a transceiver. Quite possibly, yes. I'm not overly familiar with the subsystem and was just writing up what Felipe suggested. (The same is probably true for other log messages in this source file.) I don't disagree, but I'd rather someone with more experience in the USB subsystem do that kind of broader audit/change. I'd be happy to test. josh -- 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/