On Mon, Mar 21, 2011 at 09:21:48AM -0700, Greg KH wrote:
> On Mon, Mar 21, 2011 at 04:40:26PM +0200, Aaro Koskinen wrote:
> > Hi,
> >
> > On Mon, 21 Mar 2011, Sergei Shtylyov wrote:
> > >>This patch allows the ISP1707 USB tranceiver on Nokia
> > >>N900 to be disabled when usb cable is disconnected.
> > >>This saves approximately 14mA of battery current.
> > >
> > >>Patch based on work done by Heikki Krogerus on N900
> > >>maemo kernel.
> > >
> > >>Signed-off-by: Kalle Jokiniemi <kalle.jokiniemi@xxxxxxxxx>
> > >>Cc: Heikki Krogerus <heikki.krogerus@xxxxxxxxx>
> > >>---
> > >> arch/arm/mach-omap2/board-rx51-peripherals.c | 32
> > >>++++++++++++++++++++++++++
> > >> 1 files changed, 32 insertions(+), 0 deletions(-)
> > >
> > >>diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c
> > >>b/arch/arm/mach-omap2/board-rx51-peripherals.c
> > >>index e75e240..9dd22bc 100644
> > >>--- a/arch/arm/mach-omap2/board-rx51-peripherals.c
> > >>+++ b/arch/arm/mach-omap2/board-rx51-peripherals.c
> > >[...]
> > >>@@ -534,8 +537,36 @@ static struct twl4030_gpio_platform_data
> > >>rx51_gpio_data = {
> > >> .setup = rx51_twlgpio_setup,
> > >> };
> > >> +static void __init rx51_xceiv_init(void)
> > >>+{
> > >>+ if (gpio_request(RX51_USB_TRANSCEIVER_RST_GPIO, NULL) < 0)
> > >>+ BUG();
> > >
> > > Why not:
> > >
> > > BUG_ON(gpio_request(RX51_USB_TRANSCEIVER_RST_GPIO, NULL) < 0);
> >
> > Well, there should not be no BUG() there in the first place. If the GPIO
> > cannot be requested there should be an error log and phy_power should
> > be set to NULL.
>
> Yes, NEVER add a BUG() call in a driver, you just crashed the whole
> machine, which is not nice at all to anyone involved.
Yeah, add a WARN(), though to make it a big fat error report.
--
balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
