Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1721734
| From | Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: Send a large patch right now or is it better to do it later? |
| Date | 2017-08-28 17:20 +0200 |
| Message-ID | <ujtZx-7cf-49@gated-at.bofh.it> (permalink) |
| References | <u8dv3-4hd-9@gated-at.bofh.it> <u8er8-4Rs-15@gated-at.bofh.it> <u8er8-4Rs-17@gated-at.bofh.it> <u8fdw-5s4-13@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Fri, Jul 28, 2017 at 05:16:56PM +0200, Marcus Wolf wrote: > Hi Greg, > > according to the proposals of Walter Harms, I revised the rf69.c: I replaced > some macros with inline functions and removed some obsolete ifdefs. According to > walter this will improve the resource situation. In addition the readybility is > enhanced, since lines got shorter. It's a quite big change, that touched nearly > every function in that file. > I was testing the new code for a while now and did not observer a problem so > far. But I don't have a kind of unit test, so my tests for sure didn't cover > everything. > > Is it a good time, to submit such a change in these days, or is it prefrable to > submit it later? > In adition, I am a bit afraid of my current mailtool doing something > unexpected... You can send patches anytime, don't worry about timing, I can handle putting them in the correct patch queue. > P.S. Can you process diffs fom SVN, too, or is it mandatory to create the diff > with git? As long as it is in normal patch format, I can apply it. Personally, I use quilt to generate patche, which uses 'diff', and that works just fine. thanks, greg k-h
Back to linux.kernel | Previous | Next | Find similar | Unroll thread
Re: Send a large patch right now or is it better to do it later? Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-08-28 17:20 +0200
csiph-web