From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Thu, 23 Nov 2006 11:35:37 +0100 (MET) Subject: [U-Boot-Users] MC68360 port In-Reply-To: <8496f91a0611230006r5a722052u2e8a8d72773d699b@mail.gmail.com> References: <8496f91a0611202248h1e43d4b4sc2961f67dba6c6fc@mail.gmail.com> <200611221333.11432.sr@denx.de> <8496f91a0611230006r5a722052u2e8a8d72773d699b@mail.gmail.com> Message-ID: <200611231135.41505.sr@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 On Thursday 23 November 2006 09:06, Matvejchikov Ilya wrote: > 2006/11/22, Stefan Roese : > > Some comments: > > - Are you sure that you can't use the common CFI FLASH driver? > > No, I'am not sure about it. Is it so important? Yes. Most of the board specific FLASH driver are error prone, since the code and it's error are copied over and over again. This is not maintainable. The CFI driver has a very broad test basis and is maintained quite good. And a new board port with a board specific flash driver that is not needed, meaning when CFI compliant FLASH chips are used, will not be accepted anymore. What FLASH chips are you using? > > - Please don't use C++ style comments. > > - Use tabs for indentation (for example in board/mc68360/mc68360.c). > > - Please rework your board config file. There is no comment at all > > so this is not readable. > > ok Thanks. Best regards, Stefan