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 /arch | |
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>
Diffstat (limited to 'arch')
-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 | ||