staging: most: hdm-usb: initialize variables by declaration
authorAndrey Shvetsov <andrey.shvetsov@k2l.de>
Fri, 9 Sep 2016 13:25:41 +0000 (15:25 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 12 Sep 2016 07:47:15 +0000 (09:47 +0200)
This patch moves initialization of the variables to the declaration
where possible.

Signed-off-by: Andrey Shvetsov <andrey.shvetsov@k2l.de>
Signed-off-by: Christian Gromm <christian.gromm@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/most/hdm-usb/hdm_usb.c

index 073a2aae4ef24dc68ac6af4afdb4b20c33559e3a..f3264f1e3aa3528d4cb12764b74f09d979594950 100644 (file)
@@ -326,10 +326,10 @@ static int hdm_poison_channel(struct most_interface *iface, int channel)
 static int hdm_add_padding(struct most_dev *mdev, int channel, struct mbo *mbo)
 {
        struct most_channel_config *conf = &mdev->conf[channel];
-       unsigned int j, num_frames, frame_size;
+       unsigned int frame_size = get_stream_frame_size(conf);
+       unsigned int j, num_frames;
        u16 rd_addr, wr_addr;
 
-       frame_size = get_stream_frame_size(conf);
        if (!frame_size)
                return -EIO;
        num_frames = mbo->buffer_length / frame_size;
@@ -363,10 +363,10 @@ static int hdm_add_padding(struct most_dev *mdev, int channel, struct mbo *mbo)
 static int hdm_remove_padding(struct most_dev *mdev, int channel,
                              struct mbo *mbo)
 {
-       unsigned int j, num_frames, frame_size;
        struct most_channel_config *const conf = &mdev->conf[channel];
+       unsigned int frame_size = get_stream_frame_size(conf);
+       unsigned int j, num_frames;
 
-       frame_size = get_stream_frame_size(conf);
        if (!frame_size)
                return -EIO;
        num_frames = mbo->processed_length / USB_MTU;
@@ -1098,9 +1098,8 @@ static ssize_t store_value(struct most_dci_obj *dci_obj,
 {
        u16 val;
        u16 reg_addr;
-       int err;
+       int err = kstrtou16(buf, 16, &val);
 
-       err = kstrtou16(buf, 16, &val);
        if (err)
                return err;