From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Date: Wed, 29 Feb 2012 08:15:05 -0600 Subject: [U-Boot] [PATCH 0/2] Highbank fixes for 2012.03 In-Reply-To: <4F4D59BD.9020606@aribaud.net> References: <1329864747-2914-1-git-send-email-robherring2@gmail.com> <4F4D59BD.9020606@aribaud.net> Message-ID: <4F4E32E9.1060405@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Albert, On 02/28/2012 04:48 PM, Albert ARIBAUD wrote: > Hi Rob, > > Le 21/02/2012 23:52, Rob Herring a ?crit : >> From: Rob Herring >> >> This is a couple of fixes for a build error and warning. >> >> Rob >> >> Rob Herring (2): >> ARM: highbank: fix warning for calxedaxgmac_initialize >> ARM: highbank: add missing get_tbclk >> >> arch/arm/cpu/armv7/highbank/timer.c | 5 +++++ >> board/highbank/highbank.c | 1 + >> 2 files changed, 6 insertions(+), 0 deletions(-) > > Weird: I get a persistent error with ELDK42 (4.2.2) and Ubuntu (4.6.1) > ARM toolchains: > > calxedaxgmac.c:317: error: expected declaration specifiers or '...' > before numeric constant > calxedaxgmac.c:317: warning: function declaration isn't a prototype > > I would not mention it if you did not seem to ba able to build on your > side, so I must have something wrong in my setup. Can you tell me on > which commit ID you applied your patch set? You also need the previous fix that I sent: http://patchwork.ozlabs.org/patch/139020/ Here's a pull request for all 4 fixes if you prefer: The following changes since commit 2acca35ce4604dcef933f07d90aa9c9c930e1049: Merge branch 'master' of git://git.denx.de/u-boot-mmc (2012-02-17 23:54:46 +0100) are available in the git repository at: git://sources.calxeda.com/u-boot.git fixes Rob Herring (4): net: calxedaxgmac: fix build due to missing __aligned definition ARM: highbank: fix warning for calxedaxgmac_initialize ARM: highbank: add missing get_tbclk ARM: highbank: fix us_to_tick calculation arch/arm/cpu/armv7/highbank/timer.c | 9 +++++++-- board/highbank/highbank.c | 1 + drivers/net/calxedaxgmac.c | 1 + 3 files changed, 9 insertions(+), 2 deletions(-) Thanks, Rob