On 02/14/2016 07:16 PM, Simon Glass wrote:
The current name is too generic. The function returns a block device based on a provided string. Rename it to aid searching and make its purpose clearer. Also add a few comments.
+int blk_get_device_str(const char *ifname, const char *dev_str, + struct blk_desc **dev_desc);
Bikeshed: s/_str/_by_str/? Otherwise it somewhat sounds like it returns a "device_str" rather than returns a "device" and searches for it using a "str".
Of course, this is a really annoying comment that would cause a large amount of work, so feel free to ignore it.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot