ihex: request_ihex_firmware() function to load and validate firmware
authorDavid Woodhouse <dwmw2@infradead.org>
Sat, 31 May 2008 12:20:37 +0000 (15:20 +0300)
committerDavid Woodhouse <David.Woodhouse@intel.com>
Thu, 10 Jul 2008 13:47:38 +0000 (14:47 +0100)
Provide a helper to load the file and validate it in one call, to
simplify error handling in the drivers which are going to use it.

Signed-off-by: David Woodhouse <dwmw2@infradead.org>
include/linux/ihex.h

index df89edd890ae15f3fbfe722c0c93e456a2e9ed38..2baace2788a7d73cffcd12396ea2aa37dd117963 100644 (file)
@@ -9,6 +9,7 @@
 
 #include <linux/types.h>
 #include <linux/firmware.h>
+#include <linux/device.h>
 
 /* Intel HEX files actually limit the length to 256 bytes, but we have
    drivers which would benefit from using separate records which are
@@ -47,4 +48,27 @@ static inline int ihex_validate_fw(const struct firmware *fw)
        }
        return -EINVAL;
 }
+
+/* Request firmware and validate it so that we can trust we won't
+ * run off the end while reading records... */
+static inline int request_ihex_firmware(const struct firmware **fw,
+                                       const char *fw_name,
+                                       struct device *dev)
+{
+       const struct firmware *lfw;
+       int ret;
+
+       ret = request_firmware(&lfw, fw_name, dev);
+       if (ret)
+               return ret;
+       ret = ihex_validate_fw(lfw);
+       if (ret) {
+               dev_err(dev, "Firmware \"%s\" not valid IHEX records\n",
+                       fw_name);
+               release_firmware(lfw);
+               return ret;
+       }
+       *fw = lfw;
+       return 0;
+}
 #endif /* __LINUX_IHEX_H__ */