> What I'd like to do though is the following change. It changes the way > how the mdio bus / phy address is specified. With separate options for > specifying the mdio bus and address it's not clear that the -a, -b and > -r options only make sense when given together, whereas the -a and -b > options are ignored when information is printed. > While at it I added a check for the maximum phy address and added a > <varname> to BAREBOX_CMD_HELP_OPT when an option requires an argument. > Yeah, I agree, single "-r" option instead of what I originally had makes much cleaner UI. Feel free to squash the fixup into my patch or if it's easier I can make v3 of the patch set. Thanks, Andrey _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
