Hi Omar and Hari,
From: "ext Ramirez Luna, Omar" <x00omar@xxxxxx>
Subject: [PATCH][omapzoom] BRIDGE Add preemption support to unprotected functions
Date: Wed, 8 Oct 2008 17:07:15 -0500
> From: Hari Kanigeri <h-kanigeri2@xxxxxx>
> Date: Fri, 3 Oct 2008 12:20:45 -0500
> Subject: [PATCH] BRIDGE Add preemption support to unprotected functions
>
> This patch protects the critical sections in bridge
> driver that otherwise might end in unexpected behavior.
Nice catch.
By the way, would it be possible to replace this "sync.c" with normal
in-kernel APIs like "mutex_lock()" and "wait_event()" families?
There's no point to have home brewed synchronization APIs, but it may
lose the readability of the code. If you need to add debug feature to
kernel synchronization mechanism, it should be done in generic APIs,
not in the driver specific code.
Hiroshi DOYU
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
