Hi Simon, On Sun, 22 Jun 2014 22:54:43 -0600 Simon Glass <s...@chromium.org> wrote:
> Hi, > > On Jun 22, 2014 10:23 PM, "Fabio Estevam" <feste...@gmail.com> wrote: > > > > On Mon, Jun 23, 2014 at 12:39 AM, Simon Glass <s...@chromium.org> wrote: > > > > > The whitespace problems are not common but they do happen sometimes - > > > or at least I have seen it at times. Unless perhaps checkpatch has got > > > smarter? > > > > Couldn't ./scripts/cleanpatch be used to fix the whitespace issues? > > Maybe. If you know how to test this please give it a try. Does Patman fix the whitespace issues? I'm getting confused. To make things clearer, please let me ask you some questions. [1] What does "whitespace errors" mean here? - Trailing whitespaces ? - Spaces before TAB indent ? - Or what else ? [2] What do you expect Patman to do when it finds those errors? - Just display an error message ? or - Fix the issues automatically ? [3] Which features were missing from scripts/checkpatch.pl when you wrote Patman first ? Best Regards Masahiro Yamada _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot