linux-kernel - RE: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off"

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <17A5987AD917BF4DB82CC60AED33D5E107096E44@008-AM1MPN1-021.mgdnok.nokia.com>
Date:	Wed, 7 Dec 2011 07:14:45 +0000
From:	<kalle.jokiniemi@...ia.com>
To:	<felipe.contreras@...ia.com>, <linux-kernel@...r.kernel.org>
CC:	<heikki.krogerus@...ia.com>, <cbouatmailru@...il.com>,
	<tony@...mide.com>, <felipe.contreras@...il.com>
Subject: RE: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off"

Hi,

> -----Original Message-----
> From: Contreras Felipe (Nokia-SD/Helsinki)
> Sent: 5. joulukuuta 2011 19:24
> To: linux-main
> Cc: Jokiniemi Kalle (Nokia-MP/Tampere); Heikki Krogerus; Anton Vorontsov;
> Tony Lindgren; Felipe Contreras
> Subject: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off"
> 
> From: Felipe Contreras <felipe.contreras@...il.com>
> 
> This reverts commit 10299e2e4e3ed3b16503d4e04edd48b33083f4e2.
> 
> This seems to break USB networking stuff.

Reverting this patch will cause 14,7mA leak from the ISP1704 USB transceiver. I did test at the time that USB networking was ok on MeeGo 1.1 based (MeeGo community edition N900) release and this patch. Though can't help debugging this now, sorry. 

If you guys are not concerned with use time, no problem with this patch.

- Kalle

> 
> Cc: Kalle Jokiniemi <kalle.jokiniemi@...ia.com>
> Cc: Heikki Krogerus <heikki.krogerus@...ia.com>
> Cc: Anton Vorontsov <cbouatmailru@...il.com>
> Cc: Tony Lindgren <tony@...mide.com>
> Signed-off-by: Felipe Contreras <felipe.contreras@...il.com>
> ---
>  arch/arm/mach-omap2/board-rx51-peripherals.c |   27 +------------------------
>  1 files changed, 2 insertions(+), 25 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c b/arch/arm/mach-
> omap2/board-rx51-peripherals.c
> index ba1aa07..9f8e4d2 100644
> --- a/arch/arm/mach-omap2/board-rx51-peripherals.c
> +++ b/arch/arm/mach-omap2/board-rx51-peripherals.c
> @@ -23,7 +23,6 @@
>  #include <linux/gpio.h>
>  #include <linux/gpio_keys.h>
>  #include <linux/mmc/host.h>
> -#include <linux/power/isp1704_charger.h>
> 
>  #include <plat/mcspi.h>
>  #include <plat/board.h>
> @@ -56,8 +55,6 @@
>  #define RX51_FMTX_IRQ			53
>  #define RX51_LP5523_CHIP_EN_GPIO	41
> 
> -#define RX51_USB_TRANSCEIVER_RST_GPIO	67
> -
>  /* list all spi devices here */
>  enum {
>  	RX51_SPI_WL1251,
> @@ -174,30 +171,10 @@ static struct spi_board_info
> rx51_peripherals_spi_board_info[] __initdata = {
>  	},
>  };
> 
> -static void rx51_charger_set_power(bool on)
> -{
> -	gpio_set_value(RX51_USB_TRANSCEIVER_RST_GPIO, on);
> -}
> -
> -static struct isp1704_charger_data rx51_charger_data = {
> -	.set_power	= rx51_charger_set_power,
> -};
> -
>  static struct platform_device rx51_charger_device = {
> -	.name	= "isp1704_charger",
> -	.dev	= {
> -		.platform_data = &rx51_charger_data,
> -	},
> +	.name = "isp1704_charger",
>  };
> 
> -static void __init rx51_charger_init(void)
> -{
> -	WARN_ON(gpio_request_one(RX51_USB_TRANSCEIVER_RST_GPIO,
> -		GPIOF_OUT_INIT_LOW, "isp1704_reset"));
> -
> -	platform_device_register(&rx51_charger_device);
> -}
> -
>  #if defined(CONFIG_KEYBOARD_GPIO) ||
> defined(CONFIG_KEYBOARD_GPIO_MODULE)
> 
>  #define RX51_GPIO_CAMERA_LENS_COVER	110
> @@ -1102,6 +1079,6 @@ void __init rx51_peripherals_init(void)
>  	if (partition)
>  		omap2_hsmmc_init(mmc);
> 
> -	rx51_charger_init();
> +	platform_device_register(&rx51_charger_device);
>  }
> 
> --
> 1.7.8.rc3.17.gf56ef1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/