Re: [U-Boot] [PATCH 14/28] drivers/fsl-mc: Changed MC firmware loading for new boot architecture

2015-03-27 Thread Kim Phillips
On Thu, 26 Mar 2015 18:57:02 -0500 Rivera Jose-B46482 german.riv...@freescale.com wrote: From: Kim Phillips [mailto:kim.phill...@freescale.com] Sent: Wednesday, March 25, 2015 4:13 PM On Tue, 24 Mar 2015 21:32:56 -0500 Stuart Yoder b08...@gmail.com wrote: On a system with the MC

Re: [U-Boot] [PATCH 14/28] drivers/fsl-mc: Changed MC firmware loading for new boot architecture

2015-03-25 Thread Kim Phillips
interfaces, so MC init - with or without DPL processing - is just a waste of our time! Thanks, Kim Thanks, Stuart On Tue, Mar 24, 2015 at 10:35 AM, Kim Phillips kim.phill...@freescale.com wrote: On Tue, 24 Mar 2015 10:14:39 -0500 Rivera Jose-B46482 german.riv...@freescale.com wrote

Re: [U-Boot] [PATCH 14/28] drivers/fsl-mc: Changed MC firmware loading for new boot architecture

2015-03-24 Thread Kim Phillips
On Tue, 24 Mar 2015 10:14:39 -0500 Rivera Jose-B46482 german.riv...@freescale.com wrote: From: Kim Phillips [mailto:kim.phill...@freescale.com] Sent: Monday, March 23, 2015 5:06 PM On Mon, 23 Mar 2015 16:15:56 -0500 Rivera Jose-B46482 german.riv...@freescale.com wrote: From: Kim

Re: [U-Boot] [PATCH 14/28] drivers/fsl-mc: Changed MC firmware loading for new boot architecture

2015-03-23 Thread Kim Phillips
On Mon, 23 Mar 2015 15:06:11 -0500 Rivera Jose-B46482 german.riv...@freescale.com wrote: From: Kim Phillips [mailto:kim.phill...@freescale.com] Sent: Thursday, March 19, 2015 12:53 PM On Thu, 19 Mar 2015 09:45:45 -0700 York Sun york...@freescale.com wrote: From: J. German Rivera

Re: [U-Boot] [PATCH 14/28] drivers/fsl-mc: Changed MC firmware loading for new boot architecture

2015-03-23 Thread Kim Phillips
On Mon, 23 Mar 2015 16:15:56 -0500 Rivera Jose-B46482 german.riv...@freescale.com wrote: -Original Message- From: Kim Phillips [mailto:kim.phill...@freescale.com] Sent: Monday, March 23, 2015 3:34 PM To: Rivera Jose-B46482 Cc: Sun York-R58495; u-boot@lists.denx.de Subject: Re

Re: [U-Boot] [PATCH v5 14/28] drivers/fsl-mc: Changed MC firmware loading for new boot architecture

2015-03-23 Thread Kim Phillips
On Fri, 20 Mar 2015 19:28:18 -0700 York Sun york...@freescale.com wrote: From: J. German Rivera german.riv...@freescale.com Changed MC firmware loading to comply with the new MC boot architecture. Flush D-cache hierarchy after loading MC images. Add environment variables mcboottimeout for

Re: [U-Boot] [PATCH 14/28] drivers/fsl-mc: Changed MC firmware loading for new boot architecture

2015-03-19 Thread Kim Phillips
On Thu, 19 Mar 2015 09:45:45 -0700 York Sun york...@freescale.com wrote: From: J. German Rivera german.riv...@freescale.com Changed MC firmware loading to comply with the new MC boot architecture. Flush D-cache hierarchy after loading MC images. Add environment variables mcboottimeout for

[U-Boot] [PATCH] mpc83xx: preempt premature board support removal by setting GENERIC_BOARD (was: [PATCH 7/7] powerpc: mpc83xx: remove non-generic freescale boards)

2015-03-17 Thread Kim Phillips
/README.generic-board for details.) Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com Cc: Ilya Yanok ya...@emcraft.com Cc: Dave Liu dave...@freescale.com Cc: Michael Barkowski michael.barkow...@freescale.com Cc: Kim Phillips kim.phill...@freescale.com Nacked-by: Kim Phillips kim.phill

Re: [U-Boot] checkstack.pl

2015-01-28 Thread Kim Phillips
From 42512a3fe6f2434cfc9381328d2a4755ebe6d051 Mon Sep 17 00:00:00 2001 From: Kim Phillips kim.phill...@freescale.com Date: Wed, 28 Jan 2015 13:15:01 -0600 Subject: [PATCH] scripts/checkstack.pl: update to get AArch64 port from Linux Bring checkstack.pl up to date from its upstream Linux

Re: [U-Boot] [PATCH v2 0/7] Update gdsys board support

2014-11-18 Thread Kim Phillips
On Tue, 18 Nov 2014 13:59:12 +0100 Stefan Roese s...@denx.de wrote: On 18.11.2014 13:55, Dirk Eibach wrote: do you see any chance too get this merged soon? mpc83xx: Add gdsys hrcon board fixes a build failure and it would be nice to get it in before rc2. Kim offered to ack mpc83xx:

Re: [U-Boot] [PATCH v1 0/6] Update gdsys board support

2014-11-07 Thread Kim Phillips
On Fri, 7 Nov 2014 13:58:37 +0100 Dirk Eibach dirk.eib...@gdsys.cc wrote: Hi Kim, 2014-11-05 11:43 GMT+01:00 Stefan Roese s...@denx.de: Hi Dirk, On 05.11.2014 10:45, Dirk Eibach wrote: will you look after this series? I ask because there is also some mpc83xx stuff involved.

Re: [U-Boot] [PATCH v1 0/6] Update gdsys board support

2014-11-06 Thread Kim Phillips
On Thu, 6 Nov 2014 10:25:58 -0800 York Sun york...@freescale.com wrote: On 11/05/2014 10:29 PM, Dirk Eibach wrote: Hi Stefan, Is Kim still active? Last message from him on the list I found was from may. Not sure. Lets wait and see if Kim responds... looks not so good. I put

Re: [U-Boot] [PATCH v1 6/6] mpc83xx: Add gdsys hrcon board

2014-11-06 Thread Kim Phillips
On Wed, 29 Oct 2014 16:03:57 +0100 dirk.eib...@gdsys.cc wrote: From: Dirk Eibach dirk.eib...@gdsys.cc The gdsys hrcon board is based on a Freescale MPC8308 SOC. It boots from NOR-Flash, kernel and rootfs are stored on SD-Card. On board peripherals include: - 1x GbE (optional) - Lattice

[U-Boot] [PATCH] mpc85xx: configs: remove c=ffe from default environment

2014-05-14 Thread Kim Phillips
AFAICT, c=ffe does nothing and was a typo from the original commit d17123696c6180ac8b74fbd318bf14652623e982 powerpc/p4080: Add support for the P4080DS board and just kept on getting duplicated in subsequently added board config files. Signed-off-by: Kim Phillips kim.phill...@freescale.com

Re: [U-Boot] [PATCH 4/4] mpc8313: add CONFIG_SYS_GENERIC_BOARD to ids8313 board

2014-05-08 Thread Kim Phillips
On Thu, 8 May 2014 13:05:18 +0200 Heiko Schocher h...@denx.de wrote: - add CONFIG_SYS_GENERIC_BOARD - remove CONFIG_OF_CONTROL to boot again Signed-off-by: Heiko Schocher h...@denx.de Cc: Simon Glass s...@chromium.org Cc: Kim Phillips kim.phill...@freescale.com --- on behalf of mpc83xx

Re: [U-Boot] [PATCH 2/4] mpc8313, signed fit: disable legacy image format on ids8313 board

2014-05-08 Thread Kim Phillips
: Kim Phillips kim.phill...@freescale.com Cc: Michael Conrad michael.con...@ids.de --- on behalf of mpc83xx: Acked-by: Kim Phillips kim.phill...@freescale.com Kim ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [GIT PULL] u-boot-mpc83xx: add support for the ids8313 board

2014-04-23 Thread Kim Phillips
Hi Tom, Please pull support for the ids8313 board: The following changes since commit adcdeacc3eda1e5949e54062aa99c299e12483be: Merge branch 'master' of git://git.denx.de/u-boot-mips (2014-04-23 11:07:11 -0400) are available in the git repository at: git://git.denx.de/u-boot-mpc83xx.git

Re: [U-Boot] declaring and initializing variables

2013-10-15 Thread Kim Phillips
On Mon, 14 Oct 2013 12:05:52 -0700 York Sun york...@freescale.com wrote: On 10/07/2013 03:03 PM, Kim Phillips wrote: On Mon, 30 Sep 2013 17:04:33 -0700 York Sun york...@freescale.com wrote: Kim, et al., I know I have asked this before. Pardon me as I don't consider myself a savy

Re: [U-Boot] declaring and initializing variables

2013-10-07 Thread Kim Phillips
On Mon, 30 Sep 2013 17:04:33 -0700 York Sun york...@freescale.com wrote: Kim, et al., I know I have asked this before. Pardon me as I don't consider myself a savy programmer. I am cleaning up the DDR driver for mpc83xx, mpc85xx and mpc86xx. The question is the accetable formats of

Re: [U-Boot] [PATCH v6 1/2] exynos5250: Add arndale board support

2013-08-20 Thread Kim Phillips
On Tue, 6 Aug 2013 20:05:46 +0530 Chander Kashyap chander.kash...@linaro.org wrote: Arndale board is based on samsung's exynos5250 soc. Signed-off-by: Inderpal Singh inderpal.si...@linaro.org Signed-off-by: Chander Kashyap chander.kash...@linaro.org --- This patchseries gives me:

Re: [U-Boot] [PATCH 1/2] powerpc/83xx/km: MV88e6122 errata fix for 1.9V

2013-05-21 Thread Kim Phillips
On Mon, 6 May 2013 15:02:39 +0200 Holger Brunck holger.bru...@keymile.com wrote: Errata Fix: 1.9V Output from Internal 1.8V Regulator, acc. MV-S300889-00D.pdf , clause 4.5 Signed-off-by: Holger Brunck holger.bru...@keymile.com --- patches 1-2 applied to mpc83xx/next. Thanks, Kim

Re: [U-Boot] [PATCH v2 0/45] Verified boot implementation based on FIT

2013-04-22 Thread Kim Phillips
On Sat, 20 Apr 2013 16:03:20 -0700 Simon Glass s...@chromium.org wrote: On Mon, Apr 1, 2013 at 5:13 PM, Kim Phillips kim.phill...@freescale.com wrote: On Mon, 18 Mar 2013 16:51:20 -0700 Simon Glass s...@chromium.org wrote: I have received a number of off-list comments - please do copy

Re: [U-Boot] [PATCH v2 0/45] Verified boot implementation based on FIT

2013-04-01 Thread Kim Phillips
On Mon, 18 Mar 2013 16:51:20 -0700 Simon Glass s...@chromium.org wrote: I have received a number of off-list comments - please do copy the list when replying so that everyone can see your comments. I don't have time to fully review 45 patches, let alone the subject matter (e.g., no support for

Re: [U-Boot] [PATCH 0/5] FSL SECURE BOOT: Add support for next level image validation

2013-03-29 Thread Kim Phillips
On Fri, 29 Mar 2013 04:43:23 + Gupta Ruchika-R66431 r66...@freescale.com wrote: From: otavio.salva...@gmail.com [mailto:otavio.salva...@gmail.com] On Behalf Of Otavio Salvador Sent: Thursday, March 28, 2013 8:23 PM To: Gupta Ruchika-R66431 Cc: U-Boot Mailing List; Fleming

Re: [U-Boot] [PATCH 1/4 v7] Exynos: Add hardware accelerated SHA256 and SHA1

2013-03-21 Thread Kim Phillips
On Thu, 21 Mar 2013 03:12:54 -0400 Akshay Saraswat aksha...@samsung.com wrote: On Mon, 18 Mar 2013 02:06:15 -0400 Akshay Saraswat aksha...@samsung.com wrote: + while ((readl(ace_sha_reg-hash_status) ACE_HASH_MSGDONE_MASK) == + ACE_HASH_MSGDONE_OFF) { + /* +

Re: [U-Boot] [PATCH 1/4 v7] Exynos: Add hardware accelerated SHA256 and SHA1

2013-03-19 Thread Kim Phillips
On Mon, 18 Mar 2013 02:06:15 -0400 Akshay Saraswat aksha...@samsung.com wrote: SHA-256 and SHA-1 accelerated using ACE hardware. Signed-off-by: ARUN MANKUZHI aru...@samsung.com Signed-off-by: Akshay Saraswat aksha...@samsung.com Acked-by: Simon Glass s...@chromium.org --- +++

Re: [U-Boot] [PATCH 4/5 v4] gen: Add ACE acceleration to hash

2013-03-12 Thread Kim Phillips
On Mon, 11 Mar 2013 17:53:37 -0700 Simon Glass s...@chromium.org wrote: On Mon, Mar 11, 2013 at 5:44 PM, Kim Phillips kim.phill...@freescale.comwrote: On Thu, 7 Mar 2013 19:11:16 -0800 Simon Glass s...@chromium.org wrote: OK so let's look at adding the hash_register() idea

Re: [U-Boot] [PATCH 4/5 v4] gen: Add ACE acceleration to hash

2013-03-12 Thread Kim Phillips
On Tue, 12 Mar 2013 16:40:38 -0700 Simon Glass s...@chromium.org wrote: Hi Kim, On Tue, Mar 12, 2013 at 12:32 PM, Kim Phillips kim.phill...@freescale.com wrote: On Mon, 11 Mar 2013 17:53:37 -0700 Simon Glass s...@chromium.org wrote: On Mon, Mar 11, 2013 at 5:44 PM, Kim Phillips

Re: [U-Boot] [PATCH 4/5 v4] gen: Add ACE acceleration to hash

2013-03-11 Thread Kim Phillips
On Thu, 7 Mar 2013 19:11:16 -0800 Simon Glass s...@chromium.org wrote: Hi Kim, On Thu, Mar 7, 2013 at 6:18 PM, Kim Phillips kim.phill...@freescale.com wrote: On Thu, 7 Mar 2013 17:05:15 -0800 Simon Glass s...@chromium.org wrote: On Thu, Mar 7, 2013 at 4:25 PM, Kim Phillips kim.phill

Re: [U-Boot] [PATCH 4/5 v4] gen: Add ACE acceleration to hash

2013-03-07 Thread Kim Phillips
On Wed, 6 Mar 2013 18:08:21 -0800 Simon Glass s...@chromium.org wrote: On Wed, Mar 6, 2013 at 5:22 PM, Kim Phillips kim.phill...@freescale.com wrote: On Tue, 5 Mar 2013 22:22:09 -0800 Simon Glass s...@chromium.org wrote: On Tue, Mar 5, 2013 at 9:04 PM, Kim Phillips kim.phill

Re: [U-Boot] [PATCH 4/5 v4] gen: Add ACE acceleration to hash

2013-03-07 Thread Kim Phillips
On Thu, 7 Mar 2013 17:05:15 -0800 Simon Glass s...@chromium.org wrote: On Thu, Mar 7, 2013 at 4:25 PM, Kim Phillips kim.phill...@freescale.com wrote: On Wed, 6 Mar 2013 18:08:21 -0800 Simon Glass s...@chromium.org wrote: On Wed, Mar 6, 2013 at 5:22 PM, Kim Phillips kim.phill

Re: [U-Boot] [PATCH 4/5 v4] gen: Add ACE acceleration to hash

2013-03-06 Thread Kim Phillips
On Tue, 5 Mar 2013 22:22:09 -0800 Simon Glass s...@chromium.org wrote: On Tue, Mar 5, 2013 at 9:04 PM, Kim Phillips kim.phill...@freescale.com wrote: On Tue, 5 Mar 2013 17:51:00 -0800 Simon Glass s...@chromium.org wrote: [snip for Kim] and others too, I hope. Changes sice v3

Re: [U-Boot] [PATCH 3/4 v5] gen: Add ACE acceleration to hash

2013-03-06 Thread Kim Phillips
On Wed, 6 Mar 2013 09:18:55 -0500 Akshay Saraswat aksha...@samsung.com wrote: ACE H/W acceleration support is added to hash which can be used to test SHA 256 hash algorithm. Signed-off-by: ARUN MANKUZHI aru...@samsung.com Signed-off-by: Akshay Saraswat aksha...@samsung.com --- this

Re: [U-Boot] [PATCH 4/5 v4] gen: Add ACE acceleration to hash

2013-03-06 Thread Kim Phillips
On Wed, 6 Mar 2013 10:29:46 -0500 Akshay Saraswat aksha...@samsung.com wrote: I have removed tested with in the new set of patches. And I tested those patches with that command before mailing for review. I have tested them for various sizes this time which includes 8 MB as well. I have

Re: [U-Boot] [PATCH 4/5 v4] gen: Add ACE acceleration to hash

2013-03-05 Thread Kim Phillips
On Tue, 5 Mar 2013 08:19:59 -0500 Akshay Saraswat aksha...@samsung.com wrote: Tested with command hash sha256 0x40008000 0x2B 0x40009000. Used mm and md to write a standard string to memory location 0x40008000 and ran the above command to verify the output. patches 1,2,4,5 all contain this

Re: [U-Boot] [PATCH 4/5 v4] gen: Add ACE acceleration to hash

2013-03-05 Thread Kim Phillips
On Tue, 5 Mar 2013 17:51:00 -0800 Simon Glass s...@chromium.org wrote: Hi Kim, On Tue, Mar 5, 2013 at 2:43 PM, Kim Phillips kim.phill...@freescale.com wrote: On Tue, 5 Mar 2013 08:19:59 -0500 Akshay Saraswat aksha...@samsung.com wrote: Tested with command hash sha256 0x40008000 0x2B

Re: [U-Boot] [PATCH 1/4] Exynos: Add hardware accelerated SHA 256

2013-03-04 Thread Kim Phillips
On Fri, 1 Mar 2013 16:11:36 + Akshay Saraswat aksha...@samsung.com wrote: Samsung Enterprise Portal mySingle Hi Kim, On Thu, 28 Feb 2013 11:08:21 + Akshay Saraswat aksha...@samsung.com wrote: On Wed, 27 Feb 2013 10:24:39 -0500 Akshay Saraswat

Re: [U-Boot] [PATCH 1/4 v3] Exynos: Add hardware accelerated SHA 256

2013-03-04 Thread Kim Phillips
On Fri, 1 Mar 2013 11:16:22 -0500 Akshay Saraswat aksha...@samsung.com wrote: SHA-256 and SHA-1 accelerated using ACE hardware. curious about the rationale: how much faster is this than software? --- Changes since v2: - Added falling back to software sha256 in case length exceeds

Re: [U-Boot] [PATCH 3/4 v3] gen: Add ACE acceleration to hash

2013-03-04 Thread Kim Phillips
On Fri, 1 Mar 2013 11:16:24 -0500 Akshay Saraswat aksha...@samsung.com wrote: +#include ace_sha.h /* * These are the hash algorithms we support. Chips which support accelerated * crypto could perhaps add named version of these algorithms here. */ static struct hash_algo

Re: [U-Boot] dtb vs. kernel command line arguments

2013-03-04 Thread Kim Phillips
On Mon, 4 Mar 2013 10:30:45 -0800 Curt Brune c...@cumulusnetworks.com wrote: Hello - I want to pass a number of arguments from u-boot to the booted kernel. The arguments are needed by user space applications, not the kernel. I can think of two ways: 1. append args by setting bootargs.

Re: [U-Boot] [PATCH 1/3 v2] Exynos: Add hardware accelerated SHA 256

2013-02-28 Thread Kim Phillips
On Thu, 28 Feb 2013 06:00:21 -0500 Akshay Saraswat aksha...@samsung.com wrote: --- Changes since v1: - Moved code to drivers/crypto. - Fixed few other nits. be more specific wrt changes between revisions. + if (buf_len == 0) { + /* ACE H/W cannot compute hash

Re: [U-Boot] [PATCH 1/4] Exynos: Add hardware accelerated SHA 256

2013-02-28 Thread Kim Phillips
On Thu, 28 Feb 2013 11:08:21 + Akshay Saraswat aksha...@samsung.com wrote: On Wed, 27 Feb 2013 10:24:39 -0500 Akshay Saraswat aksha...@samsung.com wrote: can you fix your mailer to not double space lines? +/* Maximum input data size is 8 MB. Timeout observed for data size above 8MB

[U-Boot] [PATCH 2/2] tools: enable more checkpatch tests by default

2013-02-28 Thread Kim Phillips
without this, patches don't get checked for proper alignment, and e.g., for spaces after a cast and/or before a semicolon. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- .checkpatch.conf | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.checkpatch.conf b/.checkpatch.conf index

[U-Boot] [PATCH 1/2] tools: update checkpatch to latest upstream version

2013-02-28 Thread Kim Phillips
i.e., from the linux kernel's commit be987d9f80354e2e919926349282facd74992f90 Signed-off-by: Kim Phillips kim.phill...@freescale.com --- tools/checkpatch.pl | 752 ++-- 1 file changed, 558 insertions(+), 194 deletions(-) diff --git a/tools

Re: [U-Boot] [PATCH 1/4] Exynos: Add hardware accelerated SHA 256

2013-02-27 Thread Kim Phillips
On Wed, 27 Feb 2013 10:24:39 -0500 Akshay Saraswat aksha...@samsung.com wrote: SHA-256 and SHA-1 accelerated using ACE hardware. TEST=sha256 0x40008000 0x2B 0x40009000 Used mm and md to write a standard string to memory location 0x40008000 and ran the above command to verify the output.

Re: [U-Boot] [PATCH 4/4] Exynos: Flush memory region before starting SHA DMA operation

2013-02-27 Thread Kim Phillips
On Wed, 27 Feb 2013 10:24:42 -0500 Akshay Saraswat aksha...@samsung.com wrote: SHA256 commands weren't giving desired output since the data we set in the memory addresses through mw.l commands was not getting updated in actual memory. Adding this patch resolves the issue. why not use the

Re: [U-Boot] [PATCH v2 01/16] km/common: remove unneeded ifdefs for I2C

2013-02-15 Thread Kim Phillips
On Mon, 21 Jan 2013 14:55:13 +0100 Holger Brunck holger.bru...@keymile.com wrote: All boards from this serie use i2c. There is no need to #ifdef the header. Signed-off-by: Holger Brunck holger.bru...@keymile.com --- applied 16 out of 16 patches. Thanks, Kim

[U-Boot] [GIT PULL] u-boot-mpc83xx: keymile board updates

2013-02-15 Thread Kim Phillips
Hi Tom, Please pull the latest set of keymile 83xx board updates: The following changes since commit 9a82b10c6657c5744802971036bb564ebc660291: Merge branch 'master' of git://git.denx.de/u-boot (2013-02-15 17:46:50 -0600) are available in the git repository at:

Re: [U-Boot] [PATCH 04/15] km82xx, km83xx: move ethernet_present() from common to cpu specific

2013-01-18 Thread Kim Phillips
On Fri, 18 Jan 2013 09:07:32 +0100 Holger Brunck holger.bru...@keymile.com wrote: On 01/18/2013 01:23 AM, Kim Phillips wrote: On Thu, 17 Jan 2013 13:47:50 +0100 Holger Brunck holger.bru...@keymile.com wrote: +int piggy_present(void) +{ + struct km_bec_fpga *base

Re: [U-Boot] [PATCH 05/15] powerpc/83xx/km: add MV88E6122 switch support for kmvect1

2013-01-18 Thread Kim Phillips
On Fri, 18 Jan 2013 10:14:39 +0100 Holger Brunck holger.bru...@keymile.com wrote: On 01/18/2013 01:44 AM, Kim Phillips wrote: On Thu, 17 Jan 2013 13:47:51 +0100 Holger Brunck holger.bru...@keymile.com wrote: + if (piggy_present()) { + uchar enetaddr[6

Re: [U-Boot] [RFC PATCH v2 1/5] Initial net6.h

2013-01-18 Thread Kim Phillips
-by: Chris Packham chris.pack...@alliedtelesis.co.nz --- Reviewed-by: Kim Phillips kim.phill...@freescale.com NOTICE: This message contains privileged and confidential information intended only for the use of the addressee named above. If you are not the intended recipient of this message you

Re: [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h

2013-01-17 Thread Kim Phillips
On Wed, 16 Jan 2013 18:36:03 -0600 Scott Wood scottw...@freescale.com wrote: On 01/16/2013 05:59:04 PM, Kim Phillips wrote: and, if including libfdt.h which includes libfdt_env.h in the correct order, don't include fdt.h before libfdt.h. this is needed to get the fdt type definitions

Re: [U-Boot] [PATCH 04/15] km82xx, km83xx: move ethernet_present() from common to cpu specific

2013-01-17 Thread Kim Phillips
On Thu, 17 Jan 2013 13:47:50 +0100 Holger Brunck holger.bru...@keymile.com wrote: +int piggy_present(void) +{ + struct km_bec_fpga *base = + (struct km_bec_fpga *)CONFIG_SYS_KMBEC_FPGA_BASE; + + return in_8(base-bprth) PIGGY_PRESENT; +} this change produces two new

Re: [U-Boot] [PATCH 05/15] powerpc/83xx/km: add MV88E6122 switch support for kmvect1

2013-01-17 Thread Kim Phillips
On Thu, 17 Jan 2013 13:47:51 +0100 Holger Brunck holger.bru...@keymile.com wrote: +#if defined(CONFIG_KMVECT1) +#include mv88e6352.h +/* Marvell MV88E6122 switch configuration */ +struct mv88e_sw_reg extsw_conf[] = { + /* port 1, FRONT_MDI, autoneg */ + { PORT(1), PORT_PHY,

Re: [U-Boot] [PATCH V2] MPC8308: Fixup clocks in PCI Host configuration

2013-01-17 Thread Kim Phillips
On Tue, 15 Jan 2013 11:39:51 -0800 Ira W. Snyder i...@ovro.caltech.edu wrote: On Mon, Jan 14, 2013 at 06:59:59PM -0600, Kim Phillips wrote: On Tue, 8 Jan 2013 18:25:11 -0800 Barry Grussling ba...@grussling.com wrote: While trying to bring up a custom MPC8308 based board I found

Re: [U-Boot] [RFC PATCH 1/2] Initial net6.h

2013-01-16 Thread Kim Phillips
On Wed, 16 Jan 2013 23:01:26 +1300 Chris Packham judge.pack...@gmail.com wrote: +typedef union ip6addr_t { + __u8u6_addr8[16]; + __u16 u6_addr16[8]; __be16, for use with sparse. + __u32 u6_addr32[4]; __be32 +} IP6addr_t; Kim

[U-Boot] [PATCH 3/5] libfdt: update fdt.h from upstream dtc

2013-01-16 Thread Kim Phillips
upstream dtc commit feafcd972cb744750a65728440c99526e6199a6d dtc/libfdt: introduce fdt types for annotation by endian checkers. Signed-off-by: Kim Phillips kim.phill...@freescale.com Cc: Jerry Van Baren gvb.ub...@gmail.com --- include/fdt.h | 42 +- 1 file

[U-Boot] [PATCH 1/2] net: make IPaddr type big endian

2013-01-16 Thread Kim Phillips
for use with sparse. Signed-off-by: Kim Phillips kim.phill...@freescale.com Cc: Joe Hershberger joe.hershber...@gmail.com --- include/net.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/net.h b/include/net.h index 970d4d1..4c5a000 100644 --- a/include/net.h +++ b

[U-Boot] [PATCH 2/2] net/tftp: sparse fixes

2013-01-16 Thread Kim Phillips
tftp.c:464:17: warning: cast to restricted __be16 tftp.c:552:29: warning: cast to restricted __be16 tftp.c:640:33: warning: cast to restricted __be16 tftp.c:642:25: warning: cast to restricted __be16 Signed-off-by: Kim Phillips kim.phill...@freescale.com Cc: Joe Hershberger joe.hershber

[U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h

2013-01-16 Thread Kim Phillips
and, if including libfdt.h which includes libfdt_env.h in the correct order, don't include fdt.h before libfdt.h. this is needed to get the fdt type definitions set from the project environment before fdt.h uses them. Signed-off-by: Kim Phillips kim.phill...@freescale.com Cc: Jerry Van Baren

[U-Boot] [PATCH 2/5] libfdt_env.h: add fdt type definitions

2013-01-16 Thread Kim Phillips
fdt types are big endian. Signed-off-by: Kim Phillips kim.phill...@freescale.com Cc: Jerry Van Baren gvb.ub...@gmail.com --- include/libfdt_env.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/libfdt_env.h b/include/libfdt_env.h index bf63583..3e3defc 100644 --- a/include

[U-Boot] [PATCH 4/5] libfdt: update from upstream dtc commit 142419e

2013-01-16 Thread Kim Phillips
commit 142419e dtc/libfdt: sparse fixes, for u-boot's libfdt copy. Signed-off-by: Kim Phillips kim.phill...@freescale.com Cc: Jerry Van Baren gvb.ub...@gmail.com --- include/libfdt.h | 32 lib/libfdt/fdt.c | 2 +- lib/libfdt/fdt_ro.c | 2 +- lib/libfdt

[U-Boot] [PATCH 5/5 v2] common/fdt_support.c: sparse fixes

2013-01-16 Thread Kim Phillips
' qualifier from pointer target type [enabled by default] See also linux kernel commit 0131d897 of/address: use proper endianess in get_flags. Signed-off-by: Kim Phillips kim.phill...@freescale.com Cc: Jerry Van Baren gvb.ub...@gmail.com --- v1 sent Tue, 16 Oct 2012 14:28:30 - v2: updated to use

[U-Boot] [GIT PULL] u-boot-mpc83xx: support for new SPL

2013-01-16 Thread Kim Phillips
to a6187dccd813920524987792e49ef2feffb213e6: Merge remote-tracking branch 'mpc83xx/next' (2013-01-16 18:34:09 -0600) Kim Phillips (1): Merge remote-tracking branch 'mpc83xx/next' Scott Wood (3): spl: Change PAD_TO to CONFIG_SPL_PAD_TO powerpc

Re: [U-Boot] [PATCH V2] MPC8308: Fixup clocks in PCI Host configuration

2013-01-14 Thread Kim Phillips
On Tue, 8 Jan 2013 18:25:11 -0800 Barry Grussling ba...@grussling.com wrote: While trying to bring up a custom MPC8308 based board I found that the clocking was wrong. The comment in include/configs/mpc8308_p1m.h led me to believe setting HRCWH_PCI_HOST and HRCWH_PCI1_ARBITER_ENABLE in the

Re: [U-Boot] [PATCH] MPC8308: Fixup clocks in PCI Host configuration

2013-01-08 Thread Kim Phillips
On Tue, 8 Jan 2013 10:24:05 -0800 Barry Grussling ba...@grussling.com wrote: While trying to bring up a custom MPC8308 based board I found that the clocking was wrong. The comment in include/configs/mpc8308_p1m.h led me to believe setting HRCWH_PCI_HOST and HRCWH_PCI1_ARBITER_ENABLE in the

Re: [U-Boot] [PATCH v2] env: don't generate callback list entries for SPL

2012-12-20 Thread Kim Phillips
--- Acked-by: Kim Phillips kim.phill...@freescale.com Thanks, Kim ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 1/3] spl: Change PAD_TO to CONFIG_SPL_PAD_TO

2012-12-19 Thread Kim Phillips
On Thu, 6 Dec 2012 17:33:16 -0600 Scott Wood scottw...@freescale.com wrote: This was already used by some SPL targets, and allows the pad amount to be specified by board config headers rather than only in makefile fragments. Also supply a pad-to of zero if the variable is undefined. It

Re: [U-Boot] [PATCH v4 3/4] dtc/libfdt: introduce fdt types for annotation by endian checkers

2012-11-28 Thread Kim Phillips
and regular integers. This patch adds these new fdtXX_t types and, ifdef __CHECKER__ (a symbol sparse defines), includes the bitwise annotation. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: adds bitwise awareness: determine host endianness manually, and annotate swabs with __force

Re: [U-Boot] [PATCH v2] include/linux/byteorder: Always defines __fswab64, __swab64p and __swab64s

2012-11-15 Thread Kim Phillips
. Signed-off-by: Nobuhiro Iwamatsu nobuhiro.iwamatsu...@renesas.com CC: Kim Phillips kim.phill...@freescale.com --- v2: Fix mail address. Reviewed-by: Kim Phillips kim.phill...@freescale.com Kim ___ U-Boot mailing list U-Boot@lists.denx.de http

[U-Boot] [PATCH v2 4/4] dtc/libfdt: uintXX_t to fdtXX_t conversion

2012-11-14 Thread Kim Phillips
Now that fdt types are defined and annotated, use them to make sparse happy. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: reworded commit text libfdt/fdt.c | 2 +- libfdt/fdt_ro.c | 2 +- libfdt/fdt_rw.c | 4 ++-- libfdt/fdt_sw.c | 4 ++-- libfdt/fdt_wip.c | 2

[U-Boot] [PATCH v2 1/4] dtc/tests: don't include fdt.h prior to libfdt.h

2012-11-14 Thread Kim Phillips
tests will need fdt type definitions provided in a subsequent patch to libfdt_env.h. Since libfdt.h includes libfdt_env.h in the right order anyway, just remove the fdt.h include. Signed-off-by: Kim Phillips kim.phill...@freescale.com Acked-by: David Gibson da...@gibson.dropbear.id.au --- v2

[U-Boot] [PATCH v2 2/4] dtc/fdtdump: include libfdt_env.h prior to fdt.h

2012-11-14 Thread Kim Phillips
in order to get the upcoming fdt type definitions. Signed-off-by: Kim Phillips kim.phill...@freescale.com Acked-by: David Gibson da...@gibson.dropbear.id.au --- v2: added David's Acked-by. fdtdump.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fdtdump.c b/fdtdump.c index

[U-Boot] [PATCH v3 3/4] dtc/libfdt: introduce fdt types for annotation by endian checkers

2012-11-14 Thread Kim Phillips
and regular integers. This patch adds these new fdtXX_t types and, ifdef __CHECKER__ (a symbol sparse defines), includes the bitwise annotation. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: adds bitwise awareness: determine host endianness manually, and annotate swabs with __force

Re: [U-Boot] [PATCH v3 3/4] dtc/libfdt: introduce fdt types for annotation by endian checkers

2012-11-14 Thread Kim Phillips
On Thu, 15 Nov 2012 15:43:40 +1100 David Gibson da...@gibson.dropbear.id.au wrote: On Wed, Nov 14, 2012 at 06:59:58PM -0600, Kim Phillips wrote: +#define EXTRACT_BYTE(x, n) ((unsigned long long)((uint8_t *)x)[n]) +#define __SWAB16(x) ((EXTRACT_BYTE(x, 0) 8) | EXTRACT_BYTE(x, 1)) +#define

Re: [U-Boot] [PATCH 1/2] libfdt: Add support for appending the values to a existing property

2012-11-13 Thread Kim Phillips
On Mon, 12 Nov 2012 21:41:17 -0500 Jerry Van Baren gvb.ub...@gmail.com wrote: On 11/08/2012 08:41 PM, Kim Phillips wrote: On Sat, 15 Sep 2012 10:36:28 -0400 Jerry Van Baren gvb.ub...@gmail.com wrote: Queued up 2 of 2 patches to the u-boot-fdt next branch. I'll issue a pull request when

[U-Boot] [PATCH 3/4 v2] dtc/libfdt: introduce fdt types for annotation by endian checkers

2012-11-13 Thread Kim Phillips
and regular integers. This patch adds these new fdtXX_t types and, ifdef __CHECKER__ (a symbol sparse defines), includes the bitwise annotation. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: adds bitwise awareness: determine host endianness manually, and annotate swabs with __force

Re: [U-Boot] [PATCH v2 09/25] common/cmd_*.c: sparse fixes

2012-11-08 Thread Kim Phillips
On Thu, 8 Nov 2012 11:48:54 +0100 Wolfgang Denk w...@denx.de wrote: In message 1351553687-12469-10-git-send-email-kim.phill...@freescale.com you wrote: ... Signed-off-by: Kim Phillips kim.phill...@freescale.com Has anybody tested the impact of this change on the memory footprint

Re: [U-Boot] [PATCH v2 13/25] lib/zlib: sparse fixes

2012-11-08 Thread Kim Phillips
On Thu, 8 Nov 2012 12:20:51 +0100 Wolfgang Denk w...@denx.de wrote: In message 1351553687-12469-14-git-send-email-kim.phill...@freescale.com you wrote: define Z_NULL to (void *)0 include/u-boot/zlib.h to get rid of most of the NULL pointer warnings. ... -#define Z_NULL 0 /* for

Re: [U-Boot] [PATCH 1/2] libfdt: Add support for appending the values to a existing property

2012-11-08 Thread Kim Phillips
On Sat, 15 Sep 2012 10:36:28 -0400 Jerry Van Baren gvb.ub...@gmail.com wrote: On 08/27/2012 08:38 PM, Kim Phillips wrote: From: Minghuan Lian minghuan.l...@freescale.com Some properties may contain multiple values, these values may need to be added to the property respectively

Re: [U-Boot] [PATCH v2] net: Fix endianness bug in link-local

2012-11-05 Thread Kim Phillips
On Mon, 5 Nov 2012 10:13:45 -0600 Joe Hershberger joe.hershber...@ni.com wrote: The ip is stored in network order, so we can't test it in host order. Signed-off-by: Joe Hershberger joe.hershber...@ni.com --- Changes in v2: - Changed htonl() to ntohl() Reviewed-by: Kim Phillips kim.phill

[U-Boot] [PATCH] common: fix help command breakage

2012-11-05 Thread Kim Phillips
the maximum number of commands (not arguments to a command). this patch fixes both by making the allocation manually on the heap. Reported-by: Henrik Nordström hen...@henriknordstrom.net Signed-off-by: Kim Phillips kim.phill...@freescale.com --- tested on an 8572ds board common/command.c | 21

Re: [U-Boot] [PATCH] net: Fix endianness bug in link-local

2012-11-02 Thread Kim Phillips
On Thu, 1 Nov 2012 22:05:54 -0500 Joe Hershberger joe.hershber...@ni.com wrote: The ip is stored in network order, so we can't test it in host order. Signed-off-by: Joe Hershberger joe.hershber...@ni.com --- net/link_local.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[U-Boot] [PATCH] powerpc/mpc8xxx: take fdt_fixup_crypto_node() off the checkstack list

2012-10-31 Thread Kim Phillips
by moving compat_strlist into the .bss section. 0xfe004d80 fdt_fixup_crypto_node [u-boot]: 264 Signed-off-by: Kim Phillips kim.phill...@freescale.com --- arch/powerpc/cpu/mpc8xxx/fdt.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/powerpc/cpu

Re: [U-Boot] [PATCH] 8xxx: Change all 8*xx_DDR addresses to 8xxx

2012-10-31 Thread Kim Phillips
-by: Kim Phillips kim.phill...@freescale.com Kim ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] libfdt: introduce fdt type annotation for use by endian checkers

2012-10-30 Thread Kim Phillips
(a symbol sparse defines), for two new fdt types: fdt32_t and fdt64_t, and subsequently silences warnings emitted by sparse when parsing libfdt. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- note: wasn't sure whether to introduce the new fdt32 types, or just have libfdt use __be32

Re: [U-Boot] [PATCH] libfdt: introduce fdt type annotation for use by endian checkers

2012-10-30 Thread Kim Phillips
On Tue, 30 Oct 2012 16:24:05 -0600 Stephen Warren swar...@wwwdotorg.org wrote: On 10/30/2012 03:57 PM, Kim Phillips wrote: Projects such as linux and u-boot run sparse on libfdt. libfdt contains the notion of endianness via usage of endian conversion functions such as fdt32_to_cpu

[U-Boot] [PATCH v2 24/25] drivers/mtd/nand: sparse fixes

2012-10-29 Thread Kim Phillips
Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: re-worked based upon input from Scott, checkpatch fixes. drivers/mtd/nand/nand_base.c | 13 +++-- drivers/mtd/nand/nand_ecc.c | 1 + 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/drivers/mtd/nand/nand_base.c b

[U-Boot] [PATCH v2 03/25] include/linux/unaligned/generic.h: fix warning: preprocessor token __force redefined

2012-10-29 Thread Kim Phillips
a compiler.h, introduced by commit b1b4e89a0f3b75854c39a62cae41bad56d210adf Add LZO decompressor support. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: no change include/linux/unaligned/generic.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/include/linux/unaligned/generic.h

[U-Boot] [PATCH v2 17/25] powerpc/mpc83xx: sparse fixes

2012-10-29 Thread Kim Phillips
: warning: obsolete struct initializer, use C99 syntax mpc8313erdb.c:86:17: warning: obsolete struct initializer, use C99 syntax mpc8313erdb.c:87:17: warning: obsolete struct initializer, use C99 syntax mpc8313erdb.c:88:17: warning: obsolete struct initializer, use C99 syntax Signed-off-by: Kim

[U-Boot] [PATCH v2 08/25] include/image.h: sparse fixes

2012-10-29 Thread Kim Phillips
-by: Kim Phillips kim.phill...@freescale.com --- v2: no changes include/image.h | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/include/image.h b/include/image.h index 4e5863f..0a895f2 100644 --- a/include/image.h +++ b/include/image.h @@ -34,6 +34,7 @@ #define

[U-Boot] [PATCH v2 06/25] arch/powerpc/lib/extable.c: sparse fix

2012-10-29 Thread Kim Phillips
just remove it. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: doesn't make it static, it just deletes it arch/powerpc/lib/extable.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/arch/powerpc/lib/extable.c b/arch/powerpc/lib/extable.c index 7408d5c..60983ae

[U-Boot] [PATCH v2 14/25] lib/vsprintf.c: sparse fixes

2012-10-29 Thread Kim Phillips
vsprintf.c:31:12: warning: symbol 'hex_asc' was not declared. Should it be static? vsprintf.c:398:18: warning: Using plain integer as NULL pointer Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: no changes lib/vsprintf.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[U-Boot] [PATCH v2 04/25] include/linux/stddef.h: avoid 'warning: preprocessor token offsetof redefined'

2012-10-29 Thread Kim Phillips
-ffreestanding -nostdinc \ - -isystem $(gccincdir) -pipe $(PLATFORM_CPPFLAGS) +CPPFLAGS += -fno-builtin -ffreestanding -pipe $(PLATFORM_CPPFLAGS) Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: no change include/linux/stddef.h | 2 ++ 1 file changed, 2 insertions(+) diff

[U-Boot] [PATCH v2 05/25] arch/powerpc/include/asm/io.h: fix warning: preprocessor token __iomem redefined

2012-10-29 Thread Kim Phillips
812711ce6b3a386125dcf0d6a59588e461abbb87:include/linux/compiler.h fatal: Path 'include/linux/compiler.h' exists on disk, but not in '812711ce6b3a386125dcf0d6a59588e461abbb87'. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: no change arch/powerpc/include/asm/io.h | 1 - 1 file changed, 1 deletion

[U-Boot] [PATCH v2 13/25] lib/zlib: sparse fixes

2012-10-29 Thread Kim Phillips
pointer zutil.c:53:9: warning: non-ANSI definition of function 'zcalloc' zutil.c:64:9: warning: non-ANSI definition of function 'zcfree' inffast.c:70:1: warning: non-ANSI definition of function 'inflate_fast' inftrees.c:33:1: warning: non-ANSI definition of function 'inflate_table' Signed-off-by: Kim

[U-Boot] [PATCH v2 10/25] common/misc: sparse fixes

2012-10-29 Thread Kim Phillips
because not used. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: un-static search_device, it is being used elsewhere. Tested on netspace_lite_v2: console.c:537:18: warning: symbol 'search_device' was not declared. Should it be static? v2 also added checkpatch fixes, except

[U-Boot] [PATCH v2 19/25] drivers/gpio/mpc83xx_gpio.c: sparse fixes

2012-10-29 Thread Kim Phillips
mpc83xx_gpio.c:166:26: warning: non-ANSI function declaration of function 'mpc83xx_gpio_init_f' mpc83xx_gpio.c:190:26: warning: non-ANSI function declaration of function 'mpc83xx_gpio_init_r' Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: no changes drivers/gpio/mpc83xx_gpio.c

[U-Boot] [PATCH v2 16/25] powerpc/mpc85xx: sparse fixes

2012-10-29 Thread Kim Phillips
declaration of function 'disable_8569mds_brd_eeprom_write_protect' ngpixis.c:245:1: error: directive in argument list ngpixis.c:247:1: error: directive in argument list Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2: fix ngpixis command help to depend on CONFIG_SYS_LONGHELP arch/powerpc/cpu

[U-Boot] [PATCH v2 11/25] net/: sparse fixes

2012-10-29 Thread Kim Phillips
it be static? Note: in the ping.c fix, commit a36b12f95a29647a06b5459198684fc142482020 net: Move PING out of net.c mistakenly carried the ifdef CMD_PING clause from when it was necessary to avoid warnings when it was embedded in net.c. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- v2

  1   2   3   4   5   6   7   8   >