From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfgang Denk Date: Thu, 29 Jan 2009 11:52:05 +0100 Subject: [U-Boot] [PATCH] Add support for the digsy MTC board. In-Reply-To: <49816F40.3000201@semihalf.com> References: <12331552753467-git-send-email-gjb@semihalf.com> <20090128210559.8CF2A832E416@gemini.denx.de> <49816F40.3000201@semihalf.com> Message-ID: <20090129105205.68BF1832E416@gemini.denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Grzegorz Bernacki, In message <49816F40.3000201@semihalf.com> you wrote: > > > General comment: there is a lot of code which could use a few comments > > so the reader has a chance to understand what exactly you are doing. > > > > Could you point me out which parts of code except eeprom defines needs > more comments? Hm... ther eare longish code sequences without a single line of comment what you are doing, or why. > >> * This program is distributed in the hope that it will be useful, > >> * but WITHOUT ANY WARRANTY; without even the implied warranty of > >> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > >> * GNU General Public License for more details. > >> * > >> * You should have received a copy of the GNU General Public License > > > > Please don't arbitrarily reformat the code. > > I think this tab was inserted by mistake. In u-boot code there are both > versions of license with and without this tab. Do you really want to > leave it? The original form of the code has 2 spaces there, as usual after a full stop. It seems this file has been run through "unexpand -a" (or an equivalent command), which inserted a TAB instead of the two spaces. This seems no problem to me, as the text still looks exactly the same (and we same one byte of disk storage :-). Your patch makes the text look different, and the change is to the worse. 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: wd at denx.de The light at the end of the tunnel is usually a "No Exit" sign.