diff options
| author | Marek Vasut <marek.vasut@gmail.com> | 2010-09-03 16:35:46 -0400 |
|---|---|---|
| committer | Eric Miao <eric.y.miao@gmail.com> | 2010-10-08 04:21:18 -0400 |
| commit | a754aea25e804d0635d1871558bee5024605ff92 (patch) | |
| tree | 95664f969460f0c622b149a42b74c6963b67b427 | |
| parent | 884646a5721e547d302390d8a483e9b75a31b017 (diff) | |
ARM: pxa: fix pxa3xx-u2d crash when ULPI not used
In case the pxa3xx-u2d driver isn't used, probing of ohci-pxa27x will cause an
ugly kernel crash (NULL pointer dereference in pxa3xx_u2d_start_hc(), because
struct u2d is NULL and clk_enable() call will crash the kernel, trying to access
it).
Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
Signed-off-by: Eric Miao <eric.y.miao@gmail.com>
| -rw-r--r-- | arch/arm/mach-pxa/pxa3xx-ulpi.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/arch/arm/mach-pxa/pxa3xx-ulpi.c b/arch/arm/mach-pxa/pxa3xx-ulpi.c index e57439e01171..ce7168b233e2 100644 --- a/arch/arm/mach-pxa/pxa3xx-ulpi.c +++ b/arch/arm/mach-pxa/pxa3xx-ulpi.c | |||
| @@ -252,6 +252,10 @@ int pxa3xx_u2d_start_hc(struct usb_bus *host) | |||
| 252 | { | 252 | { |
| 253 | int err = 0; | 253 | int err = 0; |
| 254 | 254 | ||
| 255 | /* In case the PXA3xx ULPI isn't used, do nothing. */ | ||
| 256 | if (!u2d) | ||
| 257 | return 0; | ||
| 258 | |||
| 255 | clk_enable(u2d->clk); | 259 | clk_enable(u2d->clk); |
| 256 | 260 | ||
| 257 | if (cpu_is_pxa310()) { | 261 | if (cpu_is_pxa310()) { |
| @@ -264,6 +268,10 @@ int pxa3xx_u2d_start_hc(struct usb_bus *host) | |||
| 264 | 268 | ||
| 265 | void pxa3xx_u2d_stop_hc(struct usb_bus *host) | 269 | void pxa3xx_u2d_stop_hc(struct usb_bus *host) |
| 266 | { | 270 | { |
| 271 | /* In case the PXA3xx ULPI isn't used, do nothing. */ | ||
| 272 | if (!u2d) | ||
| 273 | return; | ||
| 274 | |||
| 267 | if (cpu_is_pxa310()) | 275 | if (cpu_is_pxa310()) |
| 268 | pxa310_stop_otg_hc(); | 276 | pxa310_stop_otg_hc(); |
| 269 | 277 | ||
