On 07/18/2011 05:18 PM, Franky Lin wrote:
No, i didn't. Thank you for pointing out this. I missed it when I rebased my local branch. It didn't break any thing because brcmf_mmc_suspend is a global variable. Is it ok to keep the patch like this? I will prepare another patch to put it back and put the variable in a structure instead of using a global.
Broken commits are bad for bisecting. "git bisect" is used to search for a revision that introduces a bug. "git bisect" selects a commit between the known working and the known broken revision. If the commit selected by "git commit" suffers from additional breakage, this could lead to wasted time and possibly to a failure to find the commit responsible for the original problem.
Please don't submit known broken commits. You can always send another version of the series. If you are using stgit, see "stg mail --help".
-- Regards, Pavel Roskin -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
