2020-12-14 21:05:04

by Aaro Koskinen

[permalink] [raw]
Subject: [PATCH] ARM: OMAP1: OSK: fix ohci-omap breakage

From: Linus Walleij <[email protected]>

Commit 45c5775460f3 ("usb: ohci-omap: Fix descriptor conversion") tried to
fix all issues related to ohci-omap descriptor conversion, but a wrong
patch was applied, and one needed change to the OSK board file is still
missing. Fix that.

Fixes: 45c5775460f3 ("usb: ohci-omap: Fix descriptor conversion")
Signed-off-by: Linus Walleij <[email protected]>
[[email protected]: rebased and updated the changelog]
Signed-off-by: Aaro Koskinen <[email protected]>
---

This is the patch that should have been merged:

https://marc.info/?l=linux-usb&m=160682513710441&w=2

arch/arm/mach-omap1/board-osk.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/arch/arm/mach-omap1/board-osk.c b/arch/arm/mach-omap1/board-osk.c
index a720259099ed..0a4c9b0b13b0 100644
--- a/arch/arm/mach-omap1/board-osk.c
+++ b/arch/arm/mach-omap1/board-osk.c
@@ -203,6 +203,8 @@ static int osk_tps_setup(struct i2c_client *client, void *context)
*/
gpio_request(OSK_TPS_GPIO_USB_PWR_EN, "n_vbus_en");
gpio_direction_output(OSK_TPS_GPIO_USB_PWR_EN, 1);
+ /* Free the GPIO again as the driver will request it */
+ gpio_free(OSK_TPS_GPIO_USB_PWR_EN);

/* Set GPIO 2 high so LED D3 is off by default */
tps65010_set_gpio_out_value(GPIO2, HIGH);
--
2.17.0


2020-12-18 07:32:47

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH] ARM: OMAP1: OSK: fix ohci-omap breakage

* Aaro Koskinen <[email protected]> [201214 23:01]:
> From: Linus Walleij <[email protected]>
>
> Commit 45c5775460f3 ("usb: ohci-omap: Fix descriptor conversion") tried to
> fix all issues related to ohci-omap descriptor conversion, but a wrong
> patch was applied, and one needed change to the OSK board file is still
> missing. Fix that.

Thanks applying into fixes.

Tony

2020-12-18 16:24:44

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH] ARM: OMAP1: OSK: fix ohci-omap breakage

On Mon, Dec 14, 2020 at 10:01 PM Aaro Koskinen <[email protected]> wrote:

> From: Linus Walleij <[email protected]>
>
> Commit 45c5775460f3 ("usb: ohci-omap: Fix descriptor conversion") tried to
> fix all issues related to ohci-omap descriptor conversion, but a wrong
> patch was applied, and one needed change to the OSK board file is still
> missing. Fix that.
>
> Fixes: 45c5775460f3 ("usb: ohci-omap: Fix descriptor conversion")
> Signed-off-by: Linus Walleij <[email protected]>
> [[email protected]: rebased and updated the changelog]
> Signed-off-by: Aaro Koskinen <[email protected]>

Reviewed-by: Linus Walleij <[email protected]>

Thanks for sweeping up the mistakes, much appreciated!

I am vaguely feeling it might be lower effort for me to convert
the OSK board to device tree than fixing all GPIO consumers
one by one, but I don't think I could procure the hardware :P

Yours,
Linus Walleij