On 2018-07-19 19:05, Mathias Kresin wrote:
> The early return on failed open of the mtd device allows to get rid of
> one level of indentation.
>
> Move the put_mtd_device() called on multiple error conditions to a goto
> label to use it later for more error conditions.
>
> While at it, make clear that we only check for the UBI magic if the read
> from flash was successful.
>
> Signed-off-by: Mathias Kresin <d...@kresin.me>
While you're touching this code, you could remove one more level of
indentation (and fix a refcount bug) by writing:
if (mtd->type != MTD_NANDFLASH &&
mtd->type != MTD_NORFLASH &&
mtd->type != MTD_DATAFLASH &&
mtd->type != MTD_MLCNANDFLASH)
goto cleanup;
- Felix
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel
