Re: [PATCH] Change struct flchip_shared spinlock locking into mutex

[Artem Bityutskiy]

On Thu, 2010-08-05 at 07:58 +0300, Artem Bityutskiy wrote:
> On Mon, 2010-08-02 at 21:40 +0200, stefani@xxxxxxxxxxx wrote:
> > From: Stefani Seibold <stefani@xxxxxxxxxxx>
> > 
> > This patch prevent to schedule while atomic by changing the
> > flchip_shared spinlock into a mutex. This should be save since no atomic
> > path will use this lock.
> > 
> > This patch is based on linux kernel 2.6.35. Please apply.
> > 
> > It was requested by Arnd Bergmann and Vasiliy Kulikov.
> 
> Taken to my l2-mtd-2.6.git / master

This patch causes the following compilation error:

drivers/mtd/lpddr/lpddr_cmds.c: In function âlpddr_cmdsetâ:
drivers/mtd/lpddr/lpddr_cmds.c:101: warning: passing argument 1 of âspinlock_checkâ from incompatible pointer type
include/linux/spinlock.h:271: note: expected âstruct spinlock_t *â but argument is of type âstruct mutex *â
drivers/mtd/lpddr/lpddr_cmds.c:101: error: âstruct mutexâ has no member named ârlockâ
drivers/mtd/lpddr/lpddr_cmds.c: In function âget_chipâ:
drivers/mtd/lpddr/lpddr_cmds.c:220: warning: passing argument 1 of âspin_lockâ from incompatible pointer type
include/linux/spinlock.h:282: note: expected âstruct spinlock_t *â but argument is of type âstruct mutex *â
drivers/mtd/lpddr/lpddr_cmds.c:233: warning: passing argument 1 of âspin_unlockâ from incompatible pointer type
include/linux/spinlock.h:322: note: expected âstruct spinlock_t *â but argument is of type âstruct mutex *â
drivers/mtd/lpddr/lpddr_cmds.c:248: warning: passing argument 1 of âspin_lockâ from incompatible pointer type
include/linux/spinlock.h:282: note: expected âstruct spinlock_t *â but argument is of type âstruct mutex *â
drivers/mtd/lpddr/lpddr_cmds.c:264: warning: passing argument 1 of âspin_unlockâ from incompatible pointer type
include/linux/spinlock.h:322: note: expected âstruct spinlock_t *â but argument is of type âstruct mutex *â
drivers/mtd/lpddr/lpddr_cmds.c:278: warning: passing argument 1 of âspin_unlockâ from incompatible pointer type
include/linux/spinlock.h:322: note: expected âstruct spinlock_t *â but argument is of type âstruct mutex *â
drivers/mtd/lpddr/lpddr_cmds.c: In function âput_chipâ:
drivers/mtd/lpddr/lpddr_cmds.c:351: warning: passing argument 1 of âspin_lockâ from incompatible pointer type
include/linux/spinlock.h:282: note: expected âstruct spinlock_t *â but argument is of type âstruct mutex *â
drivers/mtd/lpddr/lpddr_cmds.c:359: warning: passing argument 1 of âspin_unlockâ from incompatible pointer type
include/linux/spinlock.h:322: note: expected âstruct spinlock_t *â but argument is of type âstruct mutex *â
drivers/mtd/lpddr/lpddr_cmds.c:377: warning: passing argument 1 of âspin_unlockâ from incompatible pointer type
include/linux/spinlock.h:322: note: expected âstruct spinlock_t *â but argument is of type âstruct mutex *â
drivers/mtd/lpddr/lpddr_cmds.c:381: warning: passing argument 1 of âspin_unlockâ from incompatible pointer type
include/linux/spinlock.h:322: note: expected âstruct spinlock_t *â but argument is of type âstruct mutex *â

Removed from my tree.

-- 
Best Regards,
Artem Bityutskiy (ÐÑÑÑÐ ÐÐÑÑÑÐÐÐ)

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/