Re: [U-Boot] [PATCH] board/bsc913x: Add config flag for bootdelay
Hi Albert, (weird way of quoting messages; makes it really hard to see who said what. Can't your mail client use the standard line prefix?) Fixed As for the commit summary, it is fine by me. Scott has commented on the same. Can you please have a look and suggest? Are you not mixing up BSC9131RDB and BSC9132QDS maintainership with MPC85xx tree custodianship? 913x fall under 85xx platforms only (though their naming convention is quite different) for which Andy has the custodianship. I will anyways include both Poonam and Naveen in the next rev of the patch (if any) Harry++ ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] board/bsc913x: Add config flag for bootdelay
Hello Albert, Plase fix commit message: you're not simply adding a boot delay flag, you're in fact disabling autoboot, which is quite different. [Rai Harninder-B01044] Disable autoboot through bootdelay config flag. Is this ok? Also, when touching on a board's code, you should Cc: the board's maintainer(s). Please do so for V2 of your patch -- and if you get a non-delivery status (e.g. a maintainer's mail address does not exist any more), please follow up on your patch submission to let us know. [Rai Harninder-B01044] The board's maintainer is Andy Fleming who is already included in cc Harry++ ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot