On Mon, Jun 7, 2021 at 11:56 AM Leizhen (ThunderTown) <thunder.leizhen@xxxxxxxxxx> wrote: > On 2021/6/7 16:52, Leizhen (ThunderTown) wrote: > > On 2021/6/7 16:39, Andy Shevchenko wrote: > >> On Mon, Jun 7, 2021 at 6:21 AM Zhen Lei <thunder.leizhen@xxxxxxxxxx> wrote: > >> > >>> Fix some spelling mistakes in comments: > >>> thats ==> that's > >>> unitialized ==> uninitialized > >>> panicing ==> panicking > >>> sucess ==> success > >>> possitive ==> positive > >>> intepreted ==> interpreted > >> > >> Thanks for the fix! Is it done with the help of the codespell tool? If > >> not, can you run it and check if it suggests more fixes? > > > > Yes, it's detected by codespell tool. But to avoid too many changes in one patch, I tried > > breaking it down into smaller patches(If it can be classified) to make it easier to review. > > In fact, the other patch I just posted included the rest. > > https://lkml.org/lkml/2021/6/7/151 > > All the remaining spelling mistakes are fixed by the patch above. I can combine the two of > them into one patch if you think it's necessary. No, it's good to keep them split. What I meant is to use the tool against the same subset of the files you have done your patch for. But please mention in the commit message that you have used that tool, so reviewers will not waste time on the comments like mine. -- With Best Regards, Andy Shevchenko
