code.ossystems Code Review - openembedded-core.git/blob

   1 From 7ddd5eaa7bc345c3719d613a46a95b7e8052ad2c Mon Sep 17 00:00:00 2001
   2 From: Imre Deak <imre.deak@nokia.com>
   3 Date: Tue, 21 Apr 2009 15:18:36 +0200
   4 Subject: [PATCH] DSS2: fix return value for rotate_type sysfs function
   5
   6 Signed-off-by: Imre Deak <imre.deak@nokia.com>
   7 ---
   8  drivers/video/omap2/omapfb/omapfb-sysfs.c |    9 ++++-----
   9  1 files changed, 4 insertions(+), 5 deletions(-)
  10
  11 diff --git a/drivers/video/omap2/omapfb/omapfb-sysfs.c b/drivers/video/omap2/omapfb/omapfb-sysfs.c
  12 index 4e3da42..13028ae 100644
  13 --- a/drivers/video/omap2/omapfb/omapfb-sysfs.c
  14 +++ b/drivers/video/omap2/omapfb/omapfb-sysfs.c
  15 @@ -64,9 +64,10 @@ static ssize_t store_rotate_type(struct device *dev,
  16         if (rot_type == ofbi->rotation_type)
  17                 goto out;
  18
  19 -       r = -EBUSY;
  20 -       if (ofbi->region.size)
  21 +       if (ofbi->region.size) {
  22 +               r = -EBUSY;
  23                 goto out;
  24 +       }
  25
  26         ofbi->rotation_type = rot_type;
  27
  28 @@ -74,12 +75,10 @@ static ssize_t store_rotate_type(struct device *dev,
  29          * Since the VRAM for this FB is not allocated at the moment we don't need to
  30          * do any further parameter checking at this point.
  31          */
  32 -
  33 -       r = count;
  34  out:
  35         omapfb_unlock(fbdev);
  36
  37 -       return r;
  38 +       return r ? r : count;
  39  }
  40
  41
  42 --
  43 1.5.6.5
  44