* [PATCH 0/2] omap: rx51: fix USB networking @ 2011-12-05 17:23 Felipe Contreras 2011-12-05 17:23 ` [PATCH 1/2] isp1704_charger: fix missing check Felipe Contreras 2011-12-05 17:23 ` [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" Felipe Contreras 0 siblings, 2 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-05 17:23 UTC (permalink / raw) To: linux-main Cc: Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, Felipe Contreras Hi, Here are a couple of fixes in order to get USB networking on the Nokia N900. Looks like requesting the reset GPIO breakes the usb0 link presence detection. Felipe Contreras (2): isp1704_charger: fix missing check Revert "ARM: RX-51: Enable isp1704 power on/off" arch/arm/mach-omap2/board-rx51-peripherals.c | 27 +------------------------ drivers/power/isp1704_charger.c | 2 +- 2 files changed, 3 insertions(+), 26 deletions(-) -- 1.7.8.rc3.17.gf56ef1 ^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 1/2] isp1704_charger: fix missing check 2011-12-05 17:23 [PATCH 0/2] omap: rx51: fix USB networking Felipe Contreras @ 2011-12-05 17:23 ` Felipe Contreras 2011-12-08 14:42 ` Heikki Krogerus 2011-12-05 17:23 ` [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" Felipe Contreras 1 sibling, 1 reply; 38+ messages in thread From: Felipe Contreras @ 2011-12-05 17:23 UTC (permalink / raw) To: linux-main Cc: Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, Felipe Contreras From: Felipe Contreras <felipe.contreras@gmail.com> A segfault happens if there's no board information. Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> --- drivers/power/isp1704_charger.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/power/isp1704_charger.c b/drivers/power/isp1704_charger.c index f6d72b4..768cf5e 100644 --- a/drivers/power/isp1704_charger.c +++ b/drivers/power/isp1704_charger.c @@ -79,7 +79,7 @@ static void isp1704_charger_set_power(struct isp1704_charger *isp, bool on) { struct isp1704_charger_data *board = isp->dev->platform_data; - if (board->set_power) + if (board && board->set_power) board->set_power(on); } -- 1.7.8.rc3.17.gf56ef1 ^ permalink raw reply related [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] isp1704_charger: fix missing check 2011-12-05 17:23 ` [PATCH 1/2] isp1704_charger: fix missing check Felipe Contreras @ 2011-12-08 14:42 ` Heikki Krogerus 2012-01-06 2:02 ` Anton Vorontsov 0 siblings, 1 reply; 38+ messages in thread From: Heikki Krogerus @ 2011-12-08 14:42 UTC (permalink / raw) To: Felipe Contreras Cc: linux-kernel, Kalle Jokiniemi, Anton Vorontsov, Tony Lindgren, Felipe Contreras On Mon, Dec 05, 2011 at 07:23:39PM +0200, Felipe Contreras wrote: > From: Felipe Contreras <felipe.contreras@gmail.com> > > A segfault happens if there's no board information. > > Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> Acked-by: Heikki Krogerus <krohei@gmail.com> -- heikki ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 1/2] isp1704_charger: fix missing check 2011-12-08 14:42 ` Heikki Krogerus @ 2012-01-06 2:02 ` Anton Vorontsov 0 siblings, 0 replies; 38+ messages in thread From: Anton Vorontsov @ 2012-01-06 2:02 UTC (permalink / raw) To: Heikki Krogerus Cc: Felipe Contreras, linux-kernel, Kalle Jokiniemi, Tony Lindgren, Felipe Contreras On Thu, Dec 08, 2011 at 04:42:21PM +0200, Heikki Krogerus wrote: > On Mon, Dec 05, 2011 at 07:23:39PM +0200, Felipe Contreras wrote: > > From: Felipe Contreras <felipe.contreras@gmail.com> > > > > A segfault happens if there's no board information. > > > > Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> > > Acked-by: Heikki Krogerus <krohei@gmail.com> Applied, thanks! -- Anton Vorontsov Email: cbouatmailru@gmail.com ^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-05 17:23 [PATCH 0/2] omap: rx51: fix USB networking Felipe Contreras 2011-12-05 17:23 ` [PATCH 1/2] isp1704_charger: fix missing check Felipe Contreras @ 2011-12-05 17:23 ` Felipe Contreras 2011-12-05 17:31 ` Felipe Contreras ` (2 more replies) 1 sibling, 3 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-05 17:23 UTC (permalink / raw) To: linux-main Cc: Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, Felipe Contreras From: Felipe Contreras <felipe.contreras@gmail.com> This reverts commit 10299e2e4e3ed3b16503d4e04edd48b33083f4e2. This seems to break USB networking stuff. Cc: Kalle Jokiniemi <kalle.jokiniemi@nokia.com> Cc: Heikki Krogerus <heikki.krogerus@nokia.com> Cc: Anton Vorontsov <cbouatmailru@gmail.com> Cc: Tony Lindgren <tony@atomide.com> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.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 ^ permalink raw reply related [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-05 17:23 ` [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" Felipe Contreras @ 2011-12-05 17:31 ` Felipe Contreras 2011-12-07 11:46 ` Jarkko Nikula 2011-12-07 7:14 ` kalle.jokiniemi 2011-12-07 13:05 ` Heikki Krogerus 2 siblings, 1 reply; 38+ messages in thread From: Felipe Contreras @ 2011-12-05 17:31 UTC (permalink / raw) To: Felipe Contreras Cc: linux-main, Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, linux-omap Should probably have CC'ed linux-omap. On Mon, Dec 5, 2011 at 7:23 PM, Felipe Contreras <felipe.contreras@nokia.com> wrote: > From: Felipe Contreras <felipe.contreras@gmail.com> > > This reverts commit 10299e2e4e3ed3b16503d4e04edd48b33083f4e2. > > This seems to break USB networking stuff. > > Cc: Kalle Jokiniemi <kalle.jokiniemi@nokia.com> > Cc: Heikki Krogerus <heikki.krogerus@nokia.com> > Cc: Anton Vorontsov <cbouatmailru@gmail.com> > Cc: Tony Lindgren <tony@atomide.com> > Signed-off-by: Felipe Contreras <felipe.contreras@gmail.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); > } -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-05 17:31 ` Felipe Contreras @ 2011-12-07 11:46 ` Jarkko Nikula 2011-12-07 18:42 ` Felipe Contreras 0 siblings, 1 reply; 38+ messages in thread From: Jarkko Nikula @ 2011-12-07 11:46 UTC (permalink / raw) To: Felipe Contreras Cc: Felipe Contreras, linux-main, Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, linux-omap On 12/05/2011 07:31 PM, Felipe Contreras wrote: > Should probably have CC'ed linux-omap. > > On Mon, Dec 5, 2011 at 7:23 PM, Felipe Contreras > <felipe.contreras@nokia.com> wrote: >> From: Felipe Contreras<felipe.contreras@gmail.com> >> >> This reverts commit 10299e2e4e3ed3b16503d4e04edd48b33083f4e2. >> >> This seems to break USB networking stuff. >> I don't think revert is needed since CONFIG_CHARGER_ISP1704=y should make it working. Although I don't know do we really need to drive the ISP1704 into reset in board-rx51-peripherals.c? Would it be better to leave gpio state as it was set by the bootloader and let the driver to do reset sequence if needed? http://marc.info/?l=linux-omap&m=130795363204884&w=2 -- Jarkko ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-07 11:46 ` Jarkko Nikula @ 2011-12-07 18:42 ` Felipe Contreras 2011-12-12 7:09 ` Jarkko Nikula 0 siblings, 1 reply; 38+ messages in thread From: Felipe Contreras @ 2011-12-07 18:42 UTC (permalink / raw) To: Jarkko Nikula Cc: Felipe Contreras, linux-main, Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, linux-omap On Wed, Dec 7, 2011 at 1:46 PM, Jarkko Nikula <jarkko.nikula@bitmer.com> wrote: > On 12/05/2011 07:31 PM, Felipe Contreras wrote: >> >> Should probably have CC'ed linux-omap. >> >> On Mon, Dec 5, 2011 at 7:23 PM, Felipe Contreras >> <felipe.contreras@nokia.com> wrote: >>> >>> From: Felipe Contreras<felipe.contreras@gmail.com> >>> >>> This reverts commit 10299e2e4e3ed3b16503d4e04edd48b33083f4e2. >>> >>> This seems to break USB networking stuff. >>> > I don't think revert is needed since CONFIG_CHARGER_ISP1704=y should make it > working. Although I don't know do we really need to drive the ISP1704 into > reset in board-rx51-peripherals.c? Would it be better to leave gpio state as > it was set by the bootloader and let the driver to do reset sequence if > needed? > > http://marc.info/?l=linux-omap&m=130795363204884&w=2 Nope, that's actually worst: Unhandled fault: external abort on non-linefetch (0x1028) at 0xfa0ab001 Internal error: : 1028 [#1] Modules linked in: CPU: 0 Tainted: G W (3.2.0-rc4-n900+ #5) PC is at musb_ulpi_write+0x18/0x70 LR is at isp1704_charger_probe+0x7c/0x354 pc : [<c0186188>] lr : [<c022e720>] psr: a0000013 sp : ce81df08 ip : ce843f80 fp : 00000000 r10: 00000000 r9 : 00000000 r8 : 00000000 r7 : c0345984 r6 : c032d988 r5 : c032d9c4 r4 : fa0ab076 r3 : fa0ab000 r2 : 000000aa r1 : 00000016 r0 : 00000000 Flags: NzCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment kernel Control: 10c5387d Table: 80004019 DAC: 00000015 Process swapper (pid: 1, stack limit = 0xce81c2e8) Stack: (0xce81df08 to 0xce81e000) df00: cea76840 c022e720 c032d990 c032d9c4 c0345984 c0345984 df20: 00000000 c0168d48 c0168d30 c0167ba4 00000000 c032d990 c032d9c4 c0345984 df40: 00000000 c0167cc0 c0345984 00000000 c0167c60 c016741c ce806ef8 ce85fa70 df60: c0345984 cea768c0 c0343e10 c0166d88 c02a1768 c02a1768 00000074 c0345984 df80: c030ef3c c000e874 00000000 00000000 00000000 c01682c4 c031b15c c030ef3c dfa0: c000e874 00000000 00000000 c0008560 0000005f 0000000d c031ae90 00003539 dfc0: 00000000 c031b15c c031ae90 c000e874 00000013 00000000 00000000 00000000 dfe0: 00000000 c02f9208 00000000 00000000 c02f9198 c000e874 00000000 00000000 [<c0186188>] (musb_ulpi_write+0x18/0x70) from [<c022e720>] (isp1704_charger_probe+0x7c/0x354) [<c022e720>] (isp1704_charger_probe+0x7c/0x354) from [<c0168d48>] (platform_drv_probe+0x18/0x1c) [<c0168d48>] (platform_drv_probe+0x18/0x1c) from [<c0167ba4>] (driver_probe_device+0xc8/0x184) [<c0167ba4>] (driver_probe_device+0xc8/0x184) from [<c0167cc0>] (__driver_attach+0x60/0x84) [<c0167cc0>] (__driver_attach+0x60/0x84) from [<c016741c>] (bus_for_each_dev+0x44/0x74) [<c016741c>] (bus_for_each_dev+0x44/0x74) from [<c0166d88>] (bus_add_driver+0x9c/0x20c) [<c0166d88>] (bus_add_driver+0x9c/0x20c) from [<c01682c4>] (driver_register+0xa4/0x130) [<c01682c4>] (driver_register+0xa4/0x130) from [<c0008560>] (do_one_initcall+0x94/0x164) [<c0008560>] (do_one_initcall+0x94/0x164) from [<c02f9208>] (kernel_init+0x70/0x118) [<c02f9208>] (kernel_init+0x70/0x118) from [<c000e874>] (kernel_thread_exit+0x0/0x8) Code: e6ef2072 e6ef1071 e2834076 e3a00000 (e5d3c001) ------------[ cut here ]------------ WARNING: at arch/arm/mach-omap2/omap_l3_smx.c:161 omap3_l3_app_irq+0xe8/0x12c() In-band Error seen by MPU at address 0 Modules linked in: [<c0012aa4>] (unwind_backtrace+0x0/0xec) from [<c002f97c>] (warn_slowpath_common+0x4c/0x64) [<c002f97c>] (warn_slowpath_common+0x4c/0x64) from [<c002fa14>] (warn_slowpath_fmt+0x2c/0x3c) [<c002fa14>] (warn_slowpath_fmt+0x2c/0x3c) from [<c0021d84>] (omap3_l3_app_irq+0xe8/0x12c) [<c0021d84>] (omap3_l3_app_irq+0xe8/0x12c) from [<c005a338>] (handle_irq_event_percpu+0x28/0x174) [<c005a338>] (handle_irq_event_percpu+0x28/0x174) from [<c005a4ac>] (handle_irq_event+0x28/0x38) [<c005a4ac>] (handle_irq_event+0x28/0x38) from [<c005c8c8>] (handle_level_irq+0xb8/0xe0) [<c005c8c8>] (handle_level_irq+0xb8/0xe0) from [<c0059fe8>] (generic_handle_irq+0x28/0x30) [<c0059fe8>] (generic_handle_irq+0x28/0x30) from [<c000e80c>] (handle_IRQ+0x60/0x84) [<c000e80c>] (handle_IRQ+0x60/0x84) from [<c000d4f4>] (__irq_svc+0x34/0x80) [<c000d4f4>] (__irq_svc+0x34/0x80) from [<c002f8b0>] (oops_exit+0x0/0x18) [<c002f8b0>] (oops_exit+0x0/0x18) from [<0000000b>] (0xb) ---[ end trace 1b75b31a2719ed1e ]--- ---[ end trace 1b75b31a2719ed1f ]--- Kernel panic - not syncing: Attempted to kill init! [<c0012aa4>] (unwind_backtrace+0x0/0xec) from [<c022ee84>] (panic+0x54/0x184) [<c022ee84>] (panic+0x54/0x184) from [<c00327a4>] (do_exit+0x98/0x5bc) [<c00327a4>] (do_exit+0x98/0x5bc) from [<c0011010>] (die+0x27c/0x2b8) [<c0011010>] (die+0x27c/0x2b8) from [<c0008444>] (do_DataAbort+0x80/0x94) [<c0008444>] (do_DataAbort+0x80/0x94) from [<c000d498>] (__dabt_svc+0x38/0x60) Exception stack(0xce81dec0 to 0xce81df08) dec0: 00000000 00000016 000000aa fa0ab000 fa0ab076 c032d9c4 c032d988 c0345984 dee0: 00000000 00000000 00000000 00000000 ce843f80 ce81df08 c022e720 c0186188 df00: a0000013 ffffffff [<c000d498>] (__dabt_svc+0x38/0x60) from [<c0186188>] (musb_ulpi_write+0x18/0x70) [<c0186188>] (musb_ulpi_write+0x18/0x70) from [<c022e720>] (isp1704_charger_probe+0x7c/0x354) [<c022e720>] (isp1704_charger_probe+0x7c/0x354) from [<c0168d48>] (platform_drv_probe+0x18/0x1c) [<c0168d48>] (platform_drv_probe+0x18/0x1c) from [<c0167ba4>] (driver_probe_device+0xc8/0x184) [<c0167ba4>] (driver_probe_device+0xc8/0x184) from [<c0167cc0>] (__driver_attach+0x60/0x84) [<c0167cc0>] (__driver_attach+0x60/0x84) from [<c016741c>] (bus_for_each_dev+0x44/0x74) [<c016741c>] (bus_for_each_dev+0x44/0x74) from [<c0166d88>] (bus_add_driver+0x9c/0x20c) [<c0166d88>] (bus_add_driver+0x9c/0x20c) from [<c01682c4>] (driver_register+0xa4/0x130) [<c01682c4>] (driver_register+0xa4/0x130) from [<c0008560>] (do_one_initcall+0x94/0x164) [<c0008560>] (do_one_initcall+0x94/0x164) from [<c02f9208>] (kernel_init+0x70/0x118) [<c02f9208>] (kernel_init+0x70/0x118) from [<c000e874>] (kernel_thread_exit+0x0/0x8) -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-07 18:42 ` Felipe Contreras @ 2011-12-12 7:09 ` Jarkko Nikula 2011-12-13 21:19 ` Felipe Contreras 0 siblings, 1 reply; 38+ messages in thread From: Jarkko Nikula @ 2011-12-12 7:09 UTC (permalink / raw) To: Felipe Contreras Cc: Felipe Contreras, linux-main, Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, linux-omap On 12/07/2011 08:42 PM, Felipe Contreras wrote: > On Wed, Dec 7, 2011 at 1:46 PM, Jarkko Nikula <jarkko.nikula@bitmer.com> wrote: >> On 12/05/2011 07:31 PM, Felipe Contreras wrote: >>> Should probably have CC'ed linux-omap. >>> >>> On Mon, Dec 5, 2011 at 7:23 PM, Felipe Contreras >>> <felipe.contreras@nokia.com> wrote: >>>> From: Felipe Contreras<felipe.contreras@gmail.com> >>>> >>>> This reverts commit 10299e2e4e3ed3b16503d4e04edd48b33083f4e2. >>>> >>>> This seems to break USB networking stuff. >>>> >> I don't think revert is needed since CONFIG_CHARGER_ISP1704=y should make it >> working. Although I don't know do we really need to drive the ISP1704 into >> reset in board-rx51-peripherals.c? Would it be better to leave gpio state as >> it was set by the bootloader and let the driver to do reset sequence if >> needed? >> >> http://marc.info/?l=linux-omap&m=130795363204884&w=2 > Nope, that's actually worst: > > Unhandled fault: external abort on non-linefetch (0x1028) at 0xfa0ab001 > Internal error: : 1028 [#1] > Modules linked in: Indeed yes. I checked that in 3.0 it still works but not in 3.1 so some non isp1704_charger change has broke it as there hasn't been changes on it. -- Jarkko ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-12 7:09 ` Jarkko Nikula @ 2011-12-13 21:19 ` Felipe Contreras 2011-12-13 22:53 ` Felipe Balbi 0 siblings, 1 reply; 38+ messages in thread From: Felipe Contreras @ 2011-12-13 21:19 UTC (permalink / raw) To: Jarkko Nikula Cc: Felipe Contreras, linux-main, Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, linux-omap, Felipe Balbi On Mon, Dec 12, 2011 at 9:09 AM, Jarkko Nikula <jarkko.nikula@bitmer.com> wrote: > Indeed yes. I checked that in 3.0 it still works but not in 3.1 so some > non isp1704_charger change has broke it as there hasn't been changes on it. Actually it's broken in 3.0 as well, try this configuration: # CONFIG_USB_MUSB_HOST is not set # CONFIG_USB_MUSB_PERIPHERAL is not set CONFIG_USB_MUSB_OTG=y CONFIG_USB_GADGET_MUSB_HDRC=y CONFIG_USB_MUSB_HDRC_HCD=y CONFIG_USB_GADGET_SELECTED=y # CONFIG_USB_GADGET_FUSB300 is not set # CONFIG_USB_GADGET_OMAP is not set # CONFIG_USB_GADGET_R8A66597 is not set # CONFIG_USB_GADGET_PXA_U2O is not set # CONFIG_USB_GADGET_M66592 is not set # CONFIG_USB_GADGET_DUMMY_HCD is not set I will try to find where the issues started to happen, but it's a bit difficult because all this USB Kconfig stuff is completely messed up. *Sigh* -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-13 21:19 ` Felipe Contreras @ 2011-12-13 22:53 ` Felipe Balbi 2011-12-13 23:02 ` Felipe Contreras 0 siblings, 1 reply; 38+ messages in thread From: Felipe Balbi @ 2011-12-13 22:53 UTC (permalink / raw) To: Felipe Contreras Cc: Jarkko Nikula, Felipe Contreras, linux-main, Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, linux-omap, Felipe Balbi [-- Attachment #1: Type: text/plain, Size: 1049 bytes --] On Tue, Dec 13, 2011 at 11:19:52PM +0200, Felipe Contreras wrote: > On Mon, Dec 12, 2011 at 9:09 AM, Jarkko Nikula <jarkko.nikula@bitmer.com> wrote: > > Indeed yes. I checked that in 3.0 it still works but not in 3.1 so some > > non isp1704_charger change has broke it as there hasn't been changes on it. > > Actually it's broken in 3.0 as well, try this configuration: > > # CONFIG_USB_MUSB_HOST is not set > # CONFIG_USB_MUSB_PERIPHERAL is not set > CONFIG_USB_MUSB_OTG=y > CONFIG_USB_GADGET_MUSB_HDRC=y > CONFIG_USB_MUSB_HDRC_HCD=y > > CONFIG_USB_GADGET_SELECTED=y > # CONFIG_USB_GADGET_FUSB300 is not set > # CONFIG_USB_GADGET_OMAP is not set > # CONFIG_USB_GADGET_R8A66597 is not set > # CONFIG_USB_GADGET_PXA_U2O is not set > # CONFIG_USB_GADGET_M66592 is not set > # CONFIG_USB_GADGET_DUMMY_HCD is not set > > I will try to find where the issues started to happen, but it's a bit > difficult because all this USB Kconfig stuff is completely messed up. > > *Sigh* patches are welcome -- balbi [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-13 22:53 ` Felipe Balbi @ 2011-12-13 23:02 ` Felipe Contreras 2011-12-14 7:30 ` Felipe Balbi 0 siblings, 1 reply; 38+ messages in thread From: Felipe Contreras @ 2011-12-13 23:02 UTC (permalink / raw) To: balbi Cc: Jarkko Nikula, Felipe Contreras, linux-main, Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, linux-omap On Wed, Dec 14, 2011 at 12:53 AM, Felipe Balbi <balbi@ti.com> wrote: > On Tue, Dec 13, 2011 at 11:19:52PM +0200, Felipe Contreras wrote: >> On Mon, Dec 12, 2011 at 9:09 AM, Jarkko Nikula <jarkko.nikula@bitmer.com> wrote: >> > Indeed yes. I checked that in 3.0 it still works but not in 3.1 so some >> > non isp1704_charger change has broke it as there hasn't been changes on it. >> >> Actually it's broken in 3.0 as well, try this configuration: >> >> # CONFIG_USB_MUSB_HOST is not set >> # CONFIG_USB_MUSB_PERIPHERAL is not set >> CONFIG_USB_MUSB_OTG=y >> CONFIG_USB_GADGET_MUSB_HDRC=y >> CONFIG_USB_MUSB_HDRC_HCD=y >> >> CONFIG_USB_GADGET_SELECTED=y >> # CONFIG_USB_GADGET_FUSB300 is not set >> # CONFIG_USB_GADGET_OMAP is not set >> # CONFIG_USB_GADGET_R8A66597 is not set >> # CONFIG_USB_GADGET_PXA_U2O is not set >> # CONFIG_USB_GADGET_M66592 is not set >> # CONFIG_USB_GADGET_DUMMY_HCD is not set >> >> I will try to find where the issues started to happen, but it's a bit >> difficult because all this USB Kconfig stuff is completely messed up. >> >> *Sigh* > > patches are welcome The were not last times: http://mid.gmane.org/1288656853-4625-1-git-send-email-felipe.contreras@gmail.com http://mid.gmane.org/1287482608-11320-1-git-send-email-felipe.contreras@gmail.com But I'll try again... Once I'm done with this endless bisect =/ Cheers. -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-13 23:02 ` Felipe Contreras @ 2011-12-14 7:30 ` Felipe Balbi 2011-12-14 19:31 ` Felipe Contreras 0 siblings, 1 reply; 38+ messages in thread From: Felipe Balbi @ 2011-12-14 7:30 UTC (permalink / raw) To: Felipe Contreras Cc: balbi, Jarkko Nikula, Felipe Contreras, linux-main, Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, linux-omap [-- Attachment #1: Type: text/plain, Size: 3227 bytes --] hi, On Wed, Dec 14, 2011 at 01:02:09AM +0200, Felipe Contreras wrote: > On Wed, Dec 14, 2011 at 12:53 AM, Felipe Balbi <balbi@ti.com> wrote: > > On Tue, Dec 13, 2011 at 11:19:52PM +0200, Felipe Contreras wrote: > >> On Mon, Dec 12, 2011 at 9:09 AM, Jarkko Nikula <jarkko.nikula@bitmer.com> wrote: > >> > Indeed yes. I checked that in 3.0 it still works but not in 3.1 so some > >> > non isp1704_charger change has broke it as there hasn't been changes on it. > >> > >> Actually it's broken in 3.0 as well, try this configuration: > >> > >> # CONFIG_USB_MUSB_HOST is not set > >> # CONFIG_USB_MUSB_PERIPHERAL is not set > >> CONFIG_USB_MUSB_OTG=y > >> CONFIG_USB_GADGET_MUSB_HDRC=y > >> CONFIG_USB_MUSB_HDRC_HCD=y > >> > >> CONFIG_USB_GADGET_SELECTED=y > >> # CONFIG_USB_GADGET_FUSB300 is not set > >> # CONFIG_USB_GADGET_OMAP is not set > >> # CONFIG_USB_GADGET_R8A66597 is not set > >> # CONFIG_USB_GADGET_PXA_U2O is not set > >> # CONFIG_USB_GADGET_M66592 is not set > >> # CONFIG_USB_GADGET_DUMMY_HCD is not set > >> > >> I will try to find where the issues started to happen, but it's a bit > >> difficult because all this USB Kconfig stuff is completely messed up. > >> > >> *Sigh* > > > > patches are welcome > > The were not last times: > http://mid.gmane.org/1288656853-4625-1-git-send-email-felipe.contreras@gmail.com At least [1] will not apply anymore. Most of that stuff has been cleaned up after I introduced the UDC class/core driver. There are still lots of things to fix up, but it won't happen overnight. Specially if people are more willing to complain than to patch. [2] Makes no sense whatsoever. I have been fighting a lot against these ARCH dependencies. It's generally used to hide some moronic constructs relying on <plat/*> or <mach/*> headers. Most of the time, it's just to be able to access e.g. machine_is_*, cpu_is_* or omap_ctrl_read* and the like. Also, the first step for having a single ARM zImage is to get rid of those ARCH dependencies; we need to be able to compile modules on all ARCHes (even x86 for that matter) because: a) it makes maintainer's lives easier (make allmodconfig/allyesconfig really help when applying patches) b) it helps using linux-next for compile tests better (similar as above) c) it makes it simpler to have a single zImage (from that particular driver's point of view, you can delete arch/arm/plat-omap/include/plat/*.h and nothing will change). So, if those are the only patches you can provide, please refrain from doing so as it will only take time reviewing. Instead, help really cleaning up the problems, make sure drivers compile on other ARCHes, make sure you don't include <plat/*> or <mach/*> on drivers, make sure you help reviewing patches which are coming in. > http://mid.gmane.org/1287482608-11320-1-git-send-email-felipe.contreras@gmail.com "No such article" > But I'll try again... Once I'm done with this endless bisect =/ please do, but follow the above. [1] http://article.gmane.org/gmane.linux.kernel/1056944 [2] http://article.gmane.org/gmane.linux.ports.arm.omap/45713 -- balbi [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-14 7:30 ` Felipe Balbi @ 2011-12-14 19:31 ` Felipe Contreras 0 siblings, 0 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-14 19:31 UTC (permalink / raw) To: balbi Cc: Jarkko Nikula, Felipe Contreras, linux-main, Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, linux-omap On Wed, Dec 14, 2011 at 9:30 AM, Felipe Balbi <balbi@ti.com> wrote: > On Wed, Dec 14, 2011 at 01:02:09AM +0200, Felipe Contreras wrote: >> On Wed, Dec 14, 2011 at 12:53 AM, Felipe Balbi <balbi@ti.com> wrote: >> > patches are welcome >> >> The were not last times: >> http://mid.gmane.org/1288656853-4625-1-git-send-email-felipe.contreras@gmail.com > > At least [1] will not apply anymore. Most of that stuff has been cleaned > up after I introduced the UDC class/core driver. There are still lots of > things to fix up, but it won't happen overnight. Specially if people are > more willing to complain than to patch. Yes, the situation is much better, but these patches still apply on usb core, or at least the same concept. Anyway, I have the new trivial patches, I'll send them soon. > [2] Makes no sense whatsoever. I have been fighting a lot against these > ARCH dependencies. It's generally used to hide some moronic constructs > relying on <plat/*> or <mach/*> headers. Most of the time, it's just to > be able to access e.g. machine_is_*, cpu_is_* or omap_ctrl_read* and the > like. Also, the first step for having a single ARM zImage is to get rid > of those ARCH dependencies; we need to be able to compile modules on all > ARCHes (even x86 for that matter) because: Read the patch carefully, it doesn't add any new dependencies, it just shuffles them around, I already explained that. And the rest of the stuff adds *defaults*, not dependencies, so you can still build in x86. > So, if those are the only patches you can provide, please refrain from > doing so as it will only take time reviewing. Instead, help really > cleaning up the problems, make sure drivers compile on other ARCHes, > make sure you don't include <plat/*> or <mach/*> on drivers, make sure > you help reviewing patches which are coming in. Some of these changes have already been ack'ed, I just need to split them. The rest, I guess I have to push it upstairs to Linus so he can say if it does make sense to make builds more difficult for the users, and keep the size of the current defconfigs from decreasing. >> http://mid.gmane.org/1287482608-11320-1-git-send-email-felipe.contreras@gmail.com > > "No such article" It's there. Here's the direct link: http://article.gmane.org/gmane.linux.kernel/1050608 -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
* RE: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-05 17:23 ` [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" Felipe Contreras 2011-12-05 17:31 ` Felipe Contreras @ 2011-12-07 7:14 ` kalle.jokiniemi 2011-12-07 10:57 ` Felipe Contreras 2011-12-07 13:05 ` Heikki Krogerus 2 siblings, 1 reply; 38+ messages in thread From: kalle.jokiniemi @ 2011-12-07 7:14 UTC (permalink / raw) To: felipe.contreras, linux-kernel Cc: heikki.krogerus, cbouatmailru, tony, felipe.contreras 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@gmail.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@nokia.com> > Cc: Heikki Krogerus <heikki.krogerus@nokia.com> > Cc: Anton Vorontsov <cbouatmailru@gmail.com> > Cc: Tony Lindgren <tony@atomide.com> > Signed-off-by: Felipe Contreras <felipe.contreras@gmail.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 ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-07 7:14 ` kalle.jokiniemi @ 2011-12-07 10:57 ` Felipe Contreras 0 siblings, 0 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-07 10:57 UTC (permalink / raw) To: kalle.jokiniemi Cc: felipe.contreras, linux-kernel, heikki.krogerus, cbouatmailru, tony On Wed, Dec 7, 2011 at 9:14 AM, <kalle.jokiniemi@nokia.com> wrote: > 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. I think USB networking is more important that 14.7mA leak. So, my setup is very simple, I build a basic user-space system with buildroot that basically just has a shell (busybox), I configure usb0 as usual, and it doesn't work from the PC side. Also, I never see the message saying that the link was detected. I tried a few things, perhaps I could try building with ISP1704 built-in, and try g_nokia instead of g_ether, but regardless of the results, I think the patch should be reverted because it doesn't work on some setups (mine). -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-05 17:23 ` [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" Felipe Contreras 2011-12-05 17:31 ` Felipe Contreras 2011-12-07 7:14 ` kalle.jokiniemi @ 2011-12-07 13:05 ` Heikki Krogerus 2011-12-07 18:43 ` Felipe Contreras 2 siblings, 1 reply; 38+ messages in thread From: Heikki Krogerus @ 2011-12-07 13:05 UTC (permalink / raw) To: Felipe Contreras Cc: linux-main, Kalle Jokiniemi, Heikki Krogerus, Anton Vorontsov, Tony Lindgren, Felipe Contreras Hi, On Mon, Dec 05, 2011 at 07:23:40PM +0200, Felipe Contreras wrote: > From: Felipe Contreras <felipe.contreras@gmail.com> > > This reverts commit 10299e2e4e3ed3b16503d4e04edd48b33083f4e2. > > This seems to break USB networking stuff. > > Cc: Kalle Jokiniemi <kalle.jokiniemi@nokia.com> > Cc: Heikki Krogerus <heikki.krogerus@nokia.com> > Cc: Anton Vorontsov <cbouatmailru@gmail.com> > Cc: Tony Lindgren <tony@atomide.com> > Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> > --- > arch/arm/mach-omap2/board-rx51-peripherals.c | 27 +------------------------ > 1 files changed, 2 insertions(+), 25 deletions(-) <snip> > -static void __init rx51_charger_init(void) > -{ > - WARN_ON(gpio_request_one(RX51_USB_TRANSCEIVER_RST_GPIO, > - GPIOF_OUT_INIT_LOW, "isp1704_reset")); How about simply GPIOF_OUT_INIT_HIGH here. Is that not enough to fix your problem? Revering the whole thing does not make any sense to me. -- heikki ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-07 13:05 ` Heikki Krogerus @ 2011-12-07 18:43 ` Felipe Contreras 2011-12-07 19:31 ` Tony Lindgren 0 siblings, 1 reply; 38+ messages in thread From: Felipe Contreras @ 2011-12-07 18:43 UTC (permalink / raw) To: Heikki Krogerus Cc: Felipe Contreras, linux-main, Kalle Jokiniemi, Anton Vorontsov, Tony Lindgren On Wed, Dec 7, 2011 at 3:05 PM, Heikki Krogerus <krohei@gmail.com> wrote: > On Mon, Dec 05, 2011 at 07:23:40PM +0200, Felipe Contreras wrote: >> From: Felipe Contreras <felipe.contreras@gmail.com> >> >> This reverts commit 10299e2e4e3ed3b16503d4e04edd48b33083f4e2. >> >> This seems to break USB networking stuff. >> >> Cc: Kalle Jokiniemi <kalle.jokiniemi@nokia.com> >> Cc: Heikki Krogerus <heikki.krogerus@nokia.com> >> Cc: Anton Vorontsov <cbouatmailru@gmail.com> >> Cc: Tony Lindgren <tony@atomide.com> >> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> >> --- >> arch/arm/mach-omap2/board-rx51-peripherals.c | 27 +------------------------ >> 1 files changed, 2 insertions(+), 25 deletions(-) > > <snip> > >> -static void __init rx51_charger_init(void) >> -{ >> - WARN_ON(gpio_request_one(RX51_USB_TRANSCEIVER_RST_GPIO, >> - GPIOF_OUT_INIT_LOW, "isp1704_reset")); > > How about simply GPIOF_OUT_INIT_HIGH here. Is that not enough to fix > your problem? Revering the whole thing does not make any sense to me. Yes, that fixes it :) -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-07 18:43 ` Felipe Contreras @ 2011-12-07 19:31 ` Tony Lindgren 2011-12-07 19:56 ` Felipe Contreras 2011-12-08 14:13 ` [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" Felipe Contreras 0 siblings, 2 replies; 38+ messages in thread From: Tony Lindgren @ 2011-12-07 19:31 UTC (permalink / raw) To: Felipe Contreras Cc: Heikki Krogerus, Felipe Contreras, linux-main, Kalle Jokiniemi, Anton Vorontsov * Felipe Contreras <felipe.contreras@gmail.com> [111207 10:12]: > On Wed, Dec 7, 2011 at 3:05 PM, Heikki Krogerus <krohei@gmail.com> wrote: > > On Mon, Dec 05, 2011 at 07:23:40PM +0200, Felipe Contreras wrote: > >> From: Felipe Contreras <felipe.contreras@gmail.com> > >> > >> This reverts commit 10299e2e4e3ed3b16503d4e04edd48b33083f4e2. > >> > >> This seems to break USB networking stuff. > >> > >> Cc: Kalle Jokiniemi <kalle.jokiniemi@nokia.com> > >> Cc: Heikki Krogerus <heikki.krogerus@nokia.com> > >> Cc: Anton Vorontsov <cbouatmailru@gmail.com> > >> Cc: Tony Lindgren <tony@atomide.com> > >> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> > >> --- > >> arch/arm/mach-omap2/board-rx51-peripherals.c | 27 +------------------------ > >> 1 files changed, 2 insertions(+), 25 deletions(-) > > > > <snip> > > > >> -static void __init rx51_charger_init(void) > >> -{ > >> - WARN_ON(gpio_request_one(RX51_USB_TRANSCEIVER_RST_GPIO, > >> - GPIOF_OUT_INIT_LOW, "isp1704_reset")); > > > > How about simply GPIOF_OUT_INIT_HIGH here. Is that not enough to fix > > your problem? Revering the whole thing does not make any sense to me. > > Yes, that fixes it :) Got an updated patch for this one? Please also Cc linux-omap and linux-arm-kernel lists for arch/arm/*omap*/ touching patches. Regards, Tony ^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] omap: rx51: initialize isp1704 properly 2011-12-07 19:31 ` Tony Lindgren @ 2011-12-07 19:56 ` Felipe Contreras 2011-12-08 14:13 ` [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" Felipe Contreras 1 sibling, 0 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-07 19:56 UTC (permalink / raw) To: linux-omap Cc: linux-arm-kernel, linux-kernel, Tony Lindgren, Russell King, Kalle Jokiniemi, Anton Vorontsov, Heikki Krogerus, Felipe Contreras From: Heikki Krogerus <krohei@gmail.com> Without this USB networking doesn't work as the link is never detected. Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> --- arch/arm/mach-omap2/board-rx51-peripherals.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c b/arch/arm/mach-omap2/board-rx51-peripherals.c index a1d0257..faac1fc 100644 --- a/arch/arm/mach-omap2/board-rx51-peripherals.c +++ b/arch/arm/mach-omap2/board-rx51-peripherals.c @@ -193,7 +193,7 @@ static struct platform_device rx51_charger_device = { static void __init rx51_charger_init(void) { WARN_ON(gpio_request_one(RX51_USB_TRANSCEIVER_RST_GPIO, - GPIOF_OUT_INIT_LOW, "isp1704_reset")); + GPIOF_OUT_INIT_HIGH, "isp1704_reset")); platform_device_register(&rx51_charger_device); } -- 1.7.8.rc1.14.g248db ^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH] omap: rx51: initialize isp1704 properly @ 2011-12-07 19:56 ` Felipe Contreras 0 siblings, 0 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-07 19:56 UTC (permalink / raw) To: linux-arm-kernel From: Heikki Krogerus <krohei@gmail.com> Without this USB networking doesn't work as the link is never detected. Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> --- arch/arm/mach-omap2/board-rx51-peripherals.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c b/arch/arm/mach-omap2/board-rx51-peripherals.c index a1d0257..faac1fc 100644 --- a/arch/arm/mach-omap2/board-rx51-peripherals.c +++ b/arch/arm/mach-omap2/board-rx51-peripherals.c @@ -193,7 +193,7 @@ static struct platform_device rx51_charger_device = { static void __init rx51_charger_init(void) { WARN_ON(gpio_request_one(RX51_USB_TRANSCEIVER_RST_GPIO, - GPIOF_OUT_INIT_LOW, "isp1704_reset")); + GPIOF_OUT_INIT_HIGH, "isp1704_reset")); platform_device_register(&rx51_charger_device); } -- 1.7.8.rc1.14.g248db ^ permalink raw reply related [flat|nested] 38+ messages in thread
* Re: [PATCH] omap: rx51: initialize isp1704 properly 2011-12-07 19:56 ` Felipe Contreras @ 2011-12-08 14:40 ` Heikki Krogerus -1 siblings, 0 replies; 38+ messages in thread From: Heikki Krogerus @ 2011-12-08 14:40 UTC (permalink / raw) To: Felipe Contreras Cc: linux-omap, linux-arm-kernel, linux-kernel, Tony Lindgren, Russell King, Kalle Jokiniemi, Anton Vorontsov Hi Felipe, On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: > From: Heikki Krogerus <krohei@gmail.com> I would prefer that you change this so this is from you, like it actually is. You can mention me in the commit message if you like. > Without this USB networking doesn't work as the link is never detected. What this changes is, it leaves the transceiver powered by default instead of setting it into power off mode. So not only USB networking is affected by this. If isp1704_charger is not enabled with RX51, USB does not work at all. Please change the comment. Thanks, -- heikki ^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] omap: rx51: initialize isp1704 properly @ 2011-12-08 14:40 ` Heikki Krogerus 0 siblings, 0 replies; 38+ messages in thread From: Heikki Krogerus @ 2011-12-08 14:40 UTC (permalink / raw) To: linux-arm-kernel Hi Felipe, On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: > From: Heikki Krogerus <krohei@gmail.com> I would prefer that you change this so this is from you, like it actually is. You can mention me in the commit message if you like. > Without this USB networking doesn't work as the link is never detected. What this changes is, it leaves the transceiver powered by default instead of setting it into power off mode. So not only USB networking is affected by this. If isp1704_charger is not enabled with RX51, USB does not work at all. Please change the comment. Thanks, -- heikki ^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH v2] omap: rx51: fix USB 2011-12-08 14:40 ` Heikki Krogerus @ 2011-12-08 20:23 ` Felipe Contreras -1 siblings, 0 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-08 20:23 UTC (permalink / raw) To: linux-omap Cc: linux-arm-kernel, linux-kernel, Tony Lindgren, Kalle Jokiniemi, Heikki Krogerus, Russell King, Anton Vorontsov, Felipe Contreras, stable From: Felipe Contreras <felipe.contreras@gmail.com> The transceiver should be powered on by default, otherwise USB doesn't work at all. All kernels after v3.0 are affected. Cc: stable@kernel.org Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> --- arch/arm/mach-omap2/board-rx51-peripherals.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c b/arch/arm/mach-omap2/board-rx51-peripherals.c index ba1aa07..c15c5c9 100644 --- a/arch/arm/mach-omap2/board-rx51-peripherals.c +++ b/arch/arm/mach-omap2/board-rx51-peripherals.c @@ -193,7 +193,7 @@ static struct platform_device rx51_charger_device = { static void __init rx51_charger_init(void) { WARN_ON(gpio_request_one(RX51_USB_TRANSCEIVER_RST_GPIO, - GPIOF_OUT_INIT_LOW, "isp1704_reset")); + GPIOF_OUT_INIT_HIGH, "isp1704_reset")); platform_device_register(&rx51_charger_device); } -- 1.7.8.rc3.17.gf56ef1 ^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH v2] omap: rx51: fix USB @ 2011-12-08 20:23 ` Felipe Contreras 0 siblings, 0 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-08 20:23 UTC (permalink / raw) To: linux-arm-kernel From: Felipe Contreras <felipe.contreras@gmail.com> The transceiver should be powered on by default, otherwise USB doesn't work at all. All kernels after v3.0 are affected. Cc: stable at kernel.org Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> --- arch/arm/mach-omap2/board-rx51-peripherals.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c b/arch/arm/mach-omap2/board-rx51-peripherals.c index ba1aa07..c15c5c9 100644 --- a/arch/arm/mach-omap2/board-rx51-peripherals.c +++ b/arch/arm/mach-omap2/board-rx51-peripherals.c @@ -193,7 +193,7 @@ static struct platform_device rx51_charger_device = { static void __init rx51_charger_init(void) { WARN_ON(gpio_request_one(RX51_USB_TRANSCEIVER_RST_GPIO, - GPIOF_OUT_INIT_LOW, "isp1704_reset")); + GPIOF_OUT_INIT_HIGH, "isp1704_reset")); platform_device_register(&rx51_charger_device); } -- 1.7.8.rc3.17.gf56ef1 ^ permalink raw reply related [flat|nested] 38+ messages in thread
* Re: [PATCH v2] omap: rx51: fix USB 2011-12-08 20:23 ` Felipe Contreras @ 2011-12-10 8:37 ` Sebastian Reichel -1 siblings, 0 replies; 38+ messages in thread From: Sebastian Reichel @ 2011-12-10 8:37 UTC (permalink / raw) To: Felipe Contreras Cc: linux-omap, linux-arm-kernel, linux-kernel, Tony Lindgren, Kalle Jokiniemi, Heikki Krogerus, Russell King, Anton Vorontsov, Felipe Contreras, stable [-- Attachment #1: Type: text/plain, Size: 359 bytes --] On Thu, Dec 08, 2011 at 10:23:00PM +0200, Felipe Contreras wrote: > The transceiver should be powered on by default, otherwise USB doesn't > work at all. > > All kernels after v3.0 are affected. I tested this patch successfully on 3.1 mainline kernel. You can add this if you want: Reviewed-by: Sebastian Reichel <sre@debian.org> -- Sebastian [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH v2] omap: rx51: fix USB @ 2011-12-10 8:37 ` Sebastian Reichel 0 siblings, 0 replies; 38+ messages in thread From: Sebastian Reichel @ 2011-12-10 8:37 UTC (permalink / raw) To: linux-arm-kernel On Thu, Dec 08, 2011 at 10:23:00PM +0200, Felipe Contreras wrote: > The transceiver should be powered on by default, otherwise USB doesn't > work at all. > > All kernels after v3.0 are affected. I tested this patch successfully on 3.1 mainline kernel. You can add this if you want: Reviewed-by: Sebastian Reichel <sre@debian.org> -- Sebastian -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20111210/b1262b1d/attachment.sig> ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH] omap: rx51: initialize isp1704 properly 2011-12-08 14:40 ` Heikki Krogerus @ 2011-12-08 20:23 ` Felipe Contreras -1 siblings, 0 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-08 20:23 UTC (permalink / raw) To: Heikki Krogerus Cc: linux-omap, linux-arm-kernel, linux-kernel, Tony Lindgren, Russell King, Kalle Jokiniemi, Anton Vorontsov On Thu, Dec 8, 2011 at 4:40 PM, Heikki Krogerus <krohei@gmail.com> wrote: > On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: >> From: Heikki Krogerus <krohei@gmail.com> > > I would prefer that you change this so this is from you, like it > actually is. You can mention me in the commit message if you like. Well, you came with the fix, I just typed it down, but as you wish :) >> Without this USB networking doesn't work as the link is never detected. > > What this changes is, it leaves the transceiver powered by default > instead of setting it into power off mode. So not only USB networking > is affected by this. If isp1704_charger is not enabled with RX51, USB > does not work at all. Please change the comment. I've updated the patch. Cheers. -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] omap: rx51: initialize isp1704 properly @ 2011-12-08 20:23 ` Felipe Contreras 0 siblings, 0 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-08 20:23 UTC (permalink / raw) To: linux-arm-kernel On Thu, Dec 8, 2011 at 4:40 PM, Heikki Krogerus <krohei@gmail.com> wrote: > On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: >> From: Heikki Krogerus <krohei@gmail.com> > > I would prefer that you change this so this is from you, like it > actually is. You can mention me in the commit message if you like. Well, you came with the fix, I just typed it down, but as you wish :) >> Without this USB networking doesn't work as the link is never detected. > > What this changes is, it leaves the transceiver powered by default > instead of setting it into power off mode. So not only USB networking > is affected by this. If isp1704_charger is not enabled with RX51, USB > does not work at all. Please change the comment. I've updated the patch. Cheers. -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH] omap: rx51: initialize isp1704 properly 2011-12-08 20:23 ` Felipe Contreras @ 2011-12-12 19:44 ` Tony Lindgren -1 siblings, 0 replies; 38+ messages in thread From: Tony Lindgren @ 2011-12-12 19:44 UTC (permalink / raw) To: Felipe Contreras Cc: Heikki Krogerus, linux-omap, linux-arm-kernel, linux-kernel, Russell King, Kalle Jokiniemi, Anton Vorontsov * Felipe Contreras <felipe.contreras@gmail.com> [111208 11:52]: > On Thu, Dec 8, 2011 at 4:40 PM, Heikki Krogerus <krohei@gmail.com> wrote: > > On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: > >> From: Heikki Krogerus <krohei@gmail.com> > > > > I would prefer that you change this so this is from you, like it > > actually is. You can mention me in the commit message if you like. > > Well, you came with the fix, I just typed it down, but as you wish :) > > >> Without this USB networking doesn't work as the link is never detected. > > > > What this changes is, it leaves the transceiver powered by default > > instead of setting it into power off mode. So not only USB networking > > is affected by this. If isp1704_charger is not enabled with RX51, USB > > does not work at all. Please change the comment. > > I've updated the patch. Sorry what's the deal with this one? Is this the one to use or is there some other patch? Tony ^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] omap: rx51: initialize isp1704 properly @ 2011-12-12 19:44 ` Tony Lindgren 0 siblings, 0 replies; 38+ messages in thread From: Tony Lindgren @ 2011-12-12 19:44 UTC (permalink / raw) To: linux-arm-kernel * Felipe Contreras <felipe.contreras@gmail.com> [111208 11:52]: > On Thu, Dec 8, 2011 at 4:40 PM, Heikki Krogerus <krohei@gmail.com> wrote: > > On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: > >> From: Heikki Krogerus <krohei@gmail.com> > > > > I would prefer that you change this so this is from you, like it > > actually is. You can mention me in the commit message if you like. > > Well, you came with the fix, I just typed it down, but as you wish :) > > >> Without this USB networking doesn't work as the link is never detected. > > > > What this changes is, it leaves the transceiver powered by default > > instead of setting it into power off mode. So not only USB networking > > is affected by this. If isp1704_charger is not enabled with RX51, USB > > does not work at all. Please change the comment. > > I've updated the patch. Sorry what's the deal with this one? Is this the one to use or is there some other patch? Tony ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH] omap: rx51: initialize isp1704 properly 2011-12-12 19:44 ` Tony Lindgren @ 2011-12-13 20:00 ` Sebastian Reichel -1 siblings, 0 replies; 38+ messages in thread From: Sebastian Reichel @ 2011-12-13 20:00 UTC (permalink / raw) To: Tony Lindgren Cc: Felipe Contreras, Heikki Krogerus, linux-omap, linux-arm-kernel, linux-kernel, Russell King, Kalle Jokiniemi, Anton Vorontsov [-- Attachment #1: Type: text/plain, Size: 1330 bytes --] On Mon, Dec 12, 2011 at 11:44:45AM -0800, Tony Lindgren wrote: > * Felipe Contreras <felipe.contreras@gmail.com> [111208 11:52]: > > On Thu, Dec 8, 2011 at 4:40 PM, Heikki Krogerus <krohei@gmail.com> wrote: > > > On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: > > >> From: Heikki Krogerus <krohei@gmail.com> > > > > > > I would prefer that you change this so this is from you, like it > > > actually is. You can mention me in the commit message if you like. > > > > Well, you came with the fix, I just typed it down, but as you wish :) > > > > >> Without this USB networking doesn't work as the link is never detected. > > > > > > What this changes is, it leaves the transceiver powered by default > > > instead of setting it into power off mode. So not only USB networking > > > is affected by this. If isp1704_charger is not enabled with RX51, USB > > > does not work at all. Please change the comment. > > > > I've updated the patch. > > Sorry what's the deal with this one? Is this the one to use or is there > some other patch? I can confirm, that USB on RX51 does not work with 3.1 mainline kernel and applying this patch (There's a v2 with fixed Author under [0]) results in working USB support. [0] http://www.spinics.net/lists/linux-omap/msg61217.html -- Sebastian [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] omap: rx51: initialize isp1704 properly @ 2011-12-13 20:00 ` Sebastian Reichel 0 siblings, 0 replies; 38+ messages in thread From: Sebastian Reichel @ 2011-12-13 20:00 UTC (permalink / raw) To: linux-arm-kernel On Mon, Dec 12, 2011 at 11:44:45AM -0800, Tony Lindgren wrote: > * Felipe Contreras <felipe.contreras@gmail.com> [111208 11:52]: > > On Thu, Dec 8, 2011 at 4:40 PM, Heikki Krogerus <krohei@gmail.com> wrote: > > > On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: > > >> From: Heikki Krogerus <krohei@gmail.com> > > > > > > I would prefer that you change this so this is from you, like it > > > actually is. You can mention me in the commit message if you like. > > > > Well, you came with the fix, I just typed it down, but as you wish :) > > > > >> Without this USB networking doesn't work as the link is never detected. > > > > > > What this changes is, it leaves the transceiver powered by default > > > instead of setting it into power off mode. So not only USB networking > > > is affected by this. If isp1704_charger is not enabled with RX51, USB > > > does not work at all. Please change the comment. > > > > I've updated the patch. > > Sorry what's the deal with this one? Is this the one to use or is there > some other patch? I can confirm, that USB on RX51 does not work with 3.1 mainline kernel and applying this patch (There's a v2 with fixed Author under [0]) results in working USB support. [0] http://www.spinics.net/lists/linux-omap/msg61217.html -- Sebastian -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20111213/0eea6246/attachment-0001.sig> ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH] omap: rx51: initialize isp1704 properly 2011-12-12 19:44 ` Tony Lindgren @ 2011-12-13 20:05 ` Felipe Contreras -1 siblings, 0 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-13 20:05 UTC (permalink / raw) To: Tony Lindgren Cc: Heikki Krogerus, linux-omap, linux-arm-kernel, linux-kernel, Russell King, Kalle Jokiniemi, Anton Vorontsov On Mon, Dec 12, 2011 at 9:44 PM, Tony Lindgren <tony@atomide.com> wrote: > * Felipe Contreras <felipe.contreras@gmail.com> [111208 11:52]: >> On Thu, Dec 8, 2011 at 4:40 PM, Heikki Krogerus <krohei@gmail.com> wrote: >> > On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: >> >> From: Heikki Krogerus <krohei@gmail.com> >> > >> > I would prefer that you change this so this is from you, like it >> > actually is. You can mention me in the commit message if you like. >> >> Well, you came with the fix, I just typed it down, but as you wish :) >> >> >> Without this USB networking doesn't work as the link is never detected. >> > >> > What this changes is, it leaves the transceiver powered by default >> > instead of setting it into power off mode. So not only USB networking >> > is affected by this. If isp1704_charger is not enabled with RX51, USB >> > does not work at all. Please change the comment. >> >> I've updated the patch. > > Sorry what's the deal with this one? Is this the one to use or is there > some other patch? Yeap, there's v2 of this one: 1323375780-13190-1-git-send-email-felipe.contreras@nokia.com Cheers. -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] omap: rx51: initialize isp1704 properly @ 2011-12-13 20:05 ` Felipe Contreras 0 siblings, 0 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-13 20:05 UTC (permalink / raw) To: linux-arm-kernel On Mon, Dec 12, 2011 at 9:44 PM, Tony Lindgren <tony@atomide.com> wrote: > * Felipe Contreras <felipe.contreras@gmail.com> [111208 11:52]: >> On Thu, Dec 8, 2011 at 4:40 PM, Heikki Krogerus <krohei@gmail.com> wrote: >> > On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: >> >> From: Heikki Krogerus <krohei@gmail.com> >> > >> > I would prefer that you change this so this is from you, like it >> > actually is. You can mention me in the commit message if you like. >> >> Well, you came with the fix, I just typed it down, but as you wish :) >> >> >> Without this USB networking doesn't work as the link is never detected. >> > >> > What this changes is, it leaves the transceiver powered by default >> > instead of setting it into power off mode. So not only USB networking >> > is affected by this. If isp1704_charger is not enabled with RX51, USB >> > does not work at all. Please change the comment. >> >> I've updated the patch. > > Sorry what's the deal with this one? Is this the one to use or is there > some other patch? Yeap, there's v2 of this one: 1323375780-13190-1-git-send-email-felipe.contreras at nokia.com Cheers. -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH] omap: rx51: initialize isp1704 properly 2011-12-13 20:05 ` Felipe Contreras @ 2011-12-13 21:55 ` Tony Lindgren -1 siblings, 0 replies; 38+ messages in thread From: Tony Lindgren @ 2011-12-13 21:55 UTC (permalink / raw) To: Felipe Contreras Cc: Heikki Krogerus, linux-omap, linux-arm-kernel, linux-kernel, Russell King, Kalle Jokiniemi, Anton Vorontsov * Felipe Contreras <felipe.contreras@gmail.com> [111213 11:33]: > On Mon, Dec 12, 2011 at 9:44 PM, Tony Lindgren <tony@atomide.com> wrote: > > * Felipe Contreras <felipe.contreras@gmail.com> [111208 11:52]: > >> On Thu, Dec 8, 2011 at 4:40 PM, Heikki Krogerus <krohei@gmail.com> wrote: > >> > On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: > >> >> From: Heikki Krogerus <krohei@gmail.com> > >> > > >> > I would prefer that you change this so this is from you, like it > >> > actually is. You can mention me in the commit message if you like. > >> > >> Well, you came with the fix, I just typed it down, but as you wish :) > >> > >> >> Without this USB networking doesn't work as the link is never detected. > >> > > >> > What this changes is, it leaves the transceiver powered by default > >> > instead of setting it into power off mode. So not only USB networking > >> > is affected by this. If isp1704_charger is not enabled with RX51, USB > >> > does not work at all. Please change the comment. > >> > >> I've updated the patch. > > > > Sorry what's the deal with this one? Is this the one to use or is there > > some other patch? > > Yeap, there's v2 of this one: > 1323375780-13190-1-git-send-email-felipe.contreras@nokia.com OK, thanks applying into fixes. Tony ^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH] omap: rx51: initialize isp1704 properly @ 2011-12-13 21:55 ` Tony Lindgren 0 siblings, 0 replies; 38+ messages in thread From: Tony Lindgren @ 2011-12-13 21:55 UTC (permalink / raw) To: linux-arm-kernel * Felipe Contreras <felipe.contreras@gmail.com> [111213 11:33]: > On Mon, Dec 12, 2011 at 9:44 PM, Tony Lindgren <tony@atomide.com> wrote: > > * Felipe Contreras <felipe.contreras@gmail.com> [111208 11:52]: > >> On Thu, Dec 8, 2011 at 4:40 PM, Heikki Krogerus <krohei@gmail.com> wrote: > >> > On Wed, Dec 07, 2011 at 09:56:56PM +0200, Felipe Contreras wrote: > >> >> From: Heikki Krogerus <krohei@gmail.com> > >> > > >> > I would prefer that you change this so this is from you, like it > >> > actually is. You can mention me in the commit message if you like. > >> > >> Well, you came with the fix, I just typed it down, but as you wish :) > >> > >> >> Without this USB networking doesn't work as the link is never detected. > >> > > >> > What this changes is, it leaves the transceiver powered by default > >> > instead of setting it into power off mode. So not only USB networking > >> > is affected by this. If isp1704_charger is not enabled with RX51, USB > >> > does not work at all. Please change the comment. > >> > >> I've updated the patch. > > > > Sorry what's the deal with this one? Is this the one to use or is there > > some other patch? > > Yeap, there's v2 of this one: > 1323375780-13190-1-git-send-email-felipe.contreras at nokia.com OK, thanks applying into fixes. Tony ^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" 2011-12-07 19:31 ` Tony Lindgren 2011-12-07 19:56 ` Felipe Contreras @ 2011-12-08 14:13 ` Felipe Contreras 1 sibling, 0 replies; 38+ messages in thread From: Felipe Contreras @ 2011-12-08 14:13 UTC (permalink / raw) To: Tony Lindgren Cc: Heikki Krogerus, Felipe Contreras, linux-main, Kalle Jokiniemi, Anton Vorontsov On Wed, Dec 7, 2011 at 9:31 PM, Tony Lindgren <tony@atomide.com> wrote: > Got an updated patch for this one? Please also Cc linux-omap and > linux-arm-kernel lists for arch/arm/*omap*/ touching patches. You got the updated one... Right? -- Felipe Contreras ^ permalink raw reply [flat|nested] 38+ messages in thread
end of thread, other threads:[~2012-01-06 2:03 UTC | newest] Thread overview: 38+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2011-12-05 17:23 [PATCH 0/2] omap: rx51: fix USB networking Felipe Contreras 2011-12-05 17:23 ` [PATCH 1/2] isp1704_charger: fix missing check Felipe Contreras 2011-12-08 14:42 ` Heikki Krogerus 2012-01-06 2:02 ` Anton Vorontsov 2011-12-05 17:23 ` [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" Felipe Contreras 2011-12-05 17:31 ` Felipe Contreras 2011-12-07 11:46 ` Jarkko Nikula 2011-12-07 18:42 ` Felipe Contreras 2011-12-12 7:09 ` Jarkko Nikula 2011-12-13 21:19 ` Felipe Contreras 2011-12-13 22:53 ` Felipe Balbi 2011-12-13 23:02 ` Felipe Contreras 2011-12-14 7:30 ` Felipe Balbi 2011-12-14 19:31 ` Felipe Contreras 2011-12-07 7:14 ` kalle.jokiniemi 2011-12-07 10:57 ` Felipe Contreras 2011-12-07 13:05 ` Heikki Krogerus 2011-12-07 18:43 ` Felipe Contreras 2011-12-07 19:31 ` Tony Lindgren 2011-12-07 19:56 ` [PATCH] omap: rx51: initialize isp1704 properly Felipe Contreras 2011-12-07 19:56 ` Felipe Contreras 2011-12-08 14:40 ` Heikki Krogerus 2011-12-08 14:40 ` Heikki Krogerus 2011-12-08 20:23 ` [PATCH v2] omap: rx51: fix USB Felipe Contreras 2011-12-08 20:23 ` Felipe Contreras 2011-12-10 8:37 ` Sebastian Reichel 2011-12-10 8:37 ` Sebastian Reichel 2011-12-08 20:23 ` [PATCH] omap: rx51: initialize isp1704 properly Felipe Contreras 2011-12-08 20:23 ` Felipe Contreras 2011-12-12 19:44 ` Tony Lindgren 2011-12-12 19:44 ` Tony Lindgren 2011-12-13 20:00 ` Sebastian Reichel 2011-12-13 20:00 ` Sebastian Reichel 2011-12-13 20:05 ` Felipe Contreras 2011-12-13 20:05 ` Felipe Contreras 2011-12-13 21:55 ` Tony Lindgren 2011-12-13 21:55 ` Tony Lindgren 2011-12-08 14:13 ` [PATCH 2/2] Revert "ARM: RX-51: Enable isp1704 power on/off" Felipe Contreras
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.