[GIT PULL] hwspinlock fixes for 3.2

Ohad Ben-Cohen ohad at wizery.com
Tue Nov 8 02:44:45 EST 2011


Hi Linus,

Please pull:

  git://git.kernel.org/pub/scm/linux/kernel/git/ohad/hwspinlock.git fixes

To get two hwspinlock fixes from Axel.

There was a third, "module.h split merge"-related fix from Axel that I
dropped because I saw Paul picked it up in his pull request.

Thanks,
Ohad.

The following changes since commit 1ea6b8f48918282bdca0b32a34095504ee65bab5:

  Linux 3.2-rc1 (2011-11-07 16:16:02 -0800)

are available in the git repository at:
  git://git.kernel.org/pub/scm/linux/kernel/git/ohad/hwspinlock.git fixes

Axel Lin (2):
      hwspinlock: Don't return a value in __hwspin_unlock
      hwspinlock/u8500: fix build error due to undefined label

 drivers/hwspinlock/u8500_hsem.c |    6 ++----
 include/linux/hwspinlock.h      |    1 -
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/hwspinlock/u8500_hsem.c b/drivers/hwspinlock/u8500_hsem.c
index 143461a..a120f14 100644
--- a/drivers/hwspinlock/u8500_hsem.c
+++ b/drivers/hwspinlock/u8500_hsem.c
@@ -108,10 +108,8 @@ static int __devinit u8500_hsem_probe(struct
platform_device *pdev)
 		return -ENODEV;

 	io_base = ioremap(res->start, resource_size(res));
-	if (!io_base) {
-		ret = -ENOMEM;
-		goto free_state;
-	}
+	if (!io_base)
+		return -ENOMEM;

 	/* make sure protocol 1 is selected */
 	val = readl(io_base + HSEM_CTRL_REG);
diff --git a/include/linux/hwspinlock.h b/include/linux/hwspinlock.h
index 08a2fee..aad6bd4 100644
--- a/include/linux/hwspinlock.h
+++ b/include/linux/hwspinlock.h
@@ -118,7 +118,6 @@ int __hwspin_trylock(struct hwspinlock *hwlock,
int mode, unsigned long *flags)
 static inline
 void __hwspin_unlock(struct hwspinlock *hwlock, int mode, unsigned long *flags)
 {
-	return 0;
 }

 static inline int hwspin_lock_get_id(struct hwspinlock *hwlock)



More information about the linux-arm-kernel mailing list