On Thu, Jun 20, 2019 at 04:29:19AM -0400, Ethan Sommer wrote: > Ah sorry about that, I accidentally replied to Kieran only instead of to > all, my response was "I will upload a patch with those issues fixed > shortly, in terms of the dependency as far as I know commands only required > for running tests don't count as kernel compilation dependencies, and I > don't see any other uses of bc except for Documentation/EDID/Makefile, so I > believe that bc can be removed from the kernel compilation section of the > process document and will include that change with the updated patch that > fixes the 2 issues you pointed out." Sounds like parts of it should be in your commit message as a justification *why* you're doing it. You can do that for your next revision once you've waited a couple of days to gather feedback. Also, please do not top-post. Thx. -- Regards/Gruss, Boris. SUSE Linux GmbH, GF: Felix Imendörffer, Mary Higgins, Sri Rasiah, HRB 21284 (AG Nürnberg)