Re: [U-Boot] Make preparatory patches that initially have no effect?
Dear Reinhard Meyer, In message <4c66cccf.9080...@emk-elektronik.de> you wrote: > > None for me. Only for people that want to try out a single (driver) > patch. They can apply the patch series anyway (at least to the patch they are interested in). > For example at91_gpbr.h is required by two independent patches. > Of course, I could base both patches such that each one introduces > that file. Have the first add that file, and the second assume it comes later in the sequence. > > Yes,m that's a bad idea. Please re-read the "patches" wiki page. > > Commits shall be atomic, and complete. Splitting stuff that > > belongstogether is a bad idea, and your first patch that adds unused > > stuff will be rejected because of that reason: adding unused stuff. > > I know that, however it could be argued that adding header files to > describe an architectures' hardware is not exactly specific to a driver. > Thats why I asked The wiki page does not talk about drivers... It's a general rule and applies to all sorts of code. Only add what is really used (this also refers, for example, to struct definitions for register blocks etc. - don't try to provide a complete description of your SoC; add only stuff that is actually used by the code). Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de "I like your game but we have to change the rules." ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Make preparatory patches that initially have no effect?
Dear Wolfgang Denk, >> If I add those discrete changes to each driver patch (where it >> might actually belong), the incremental changes to some of >> those files would require all those driver patches to be applied >> in the right order to avoid conflicts. > > Yes, and? What's the problem with that? None for me. Only for people that want to try out a single (driver) patch. For example at91_gpbr.h is required by two independent patches. Of course, I could base both patches such that each one introduces that file. > Yes,m that's a bad idea. Please re-read the "patches" wiki page. > Commits shall be atomic, and complete. Splitting stuff that > belongstogether is a bad idea, and your first patch that adds unused > stuff will be rejected because of that reason: adding unused stuff. I know that, however it could be argued that adding header files to describe an architectures' hardware is not exactly specific to a driver. Thats why I asked Best Regards, Reinhard ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Make preparatory patches that initially have no effect?
Dear Reinhard Meyer, In message <4c665cb9.2040...@emk-elektronik.de> you wrote: > > If I add those discrete changes to each driver patch (where it > might actually belong), the incremental changes to some of > those files would require all those driver patches to be applied > in the right order to avoid conflicts. Yes, and? What's the problem with that? > Therefore I would like to put all new header files and all > changes to header files in one patch which would need to be > applied before the driver patches. > > That patch would essentially cause no change to existing code. > > Anyone find this idea bad? Yes,m that's a bad idea. Please re-read the "patches" wiki page. Commits shall be atomic, and complete. Splitting stuff that belongstogether is a bad idea, and your first patch that adds unused stuff will be rejected because of that reason: adding unused stuff. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Perl already has an IDE. It's called Unix. -- Tom Christiansen in 375bd...@cs.colorado.edu ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot