From: Reinhard Meyer Date: Tue, 28 Oct 2014 11:35:03 +0100 Subject: [U-Boot] [PATCH] Disable FLASH_AMLV256U support for TOP860 target In-Reply-To: <20141028102309.E463A382FEC@gemini.denx.de> References: <1402929691-3972-1-git-send-email-vvv444@gmail.com> <20141028093348.43664382FEC@gemini.denx.de> <20141028094822.14406382FEC@gemini.denx.de> <544F6AD8.2070605@emk-elektronik.de> <20141028102309.E463A382FEC@gemini.denx.de> Message-ID: <544F7157.9040701@emk-elektronik.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 Wolfgang, > Dear Reinhard, > > In message <544F6AD8.2070605@emk-elektronik.de> you wrote: >> top860 can be removed (We already had that discussion a while ago.) >> >> top5200 is still active in several older projects, but there was no need >> to make changes to u-boot or to integrate new features of u-boot. >> Therefore I am not testing whether any changes to u-boot break the >> function of top5200. >> >> top9000 is dead. Thanks atmel :( >> However it might be left in u-boot as an example. > Hm... if two of the boards are dead, and you are not even testing the > third, can we not just remove all of them? That would also allow to > get rid of board/emk/common: Agreed. > > - am79c874.c is completely bogus anyway How so? > - flash.c should have been replaced by the CFI driver years ago I think there was/is an issue with the fact that top5200 uses GPIO for the uppermost (A16+) address lines AND that FLASH is connected 8 Bit wide. Maybe CFI code does support such by now, at that time it did not. > - vpd.c is causing efforts when I2C / EEPROM code gets changed ideally, the API should not change that often ;) > > What do you think? Since its quite unlikely I will make any changes to those boards in the future, they can be removed. Best Regards, Reinhard