Re: [PATCH 2/3] dtc: import latest upstream dtc

2013-02-27 Thread Kim Phillips
On Tue, 26 Feb 2013 21:08:06 -0600 Rob Landley r...@landley.net wrote: On 02/26/2013 07:36:14 PM, Kim Phillips wrote: This updates scripts/dtc to upstream dtc commit 27cdc1b There's an upstream for dts? dtc - the device tree compiler, yes. Added license header to dtc/libfdt/fdt.h

[PATCH 1/3] libfdt: add fdt type definitions in libfdt_env.h

2013-02-26 Thread Kim Phillips
in preparation for the forthcoming dtc update which adds fdt specific types: fdt{16,32,64}_t. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- arch/arm/boot/compressed/libfdt_env.h | 4 include/linux/libfdt_env.h| 6 ++ 2 files changed, 10 insertions(+) diff --git

[PATCH 2/3] dtc: import latest upstream dtc

2013-02-26 Thread Kim Phillips
This updates scripts/dtc to upstream dtc commit 27cdc1b Added license header to dtc/libfdt/fdt.h and libfdt_env.h from git://git.jdl.com/software/dtc.git. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- scripts/dtc/dtc.c | 3 -- scripts/dtc/fdtdump.c | 29

[PATCH 3/3] ARM: zImage: sparse fixes

2013-02-26 Thread Kim Phillips
base types) arch/arm/boot/compressed/atags_to_fdt.c:141:54:expected unsigned int [unsigned] noident arch/arm/boot/compressed/atags_to_fdt.c:141:54:got restricted __be32 [usertype] noident Signed-off-by: Kim Phillips kim.phill...@freescale.com --- arch/arm/boot/compressed/atags_to_fdt.c

Re: [PATCH] Added license header to dtc/libfdt/fdt.h and libfdt_env.h

2013-02-14 Thread Kim Phillips
these files, it looks like Kim Phillips (CCed) also touched them substantially. Kim, did you want to supply an additional copyright line for these files? sure, it'd be: Copyright 2012 Kim Phillips, Freescale Semiconductor. I can spin a patch on top of this one, or - wait... Signed-off

Re: [PATCH] crypto: caam - Added property fsl,sec-era in SEC4.0 device tree binding.

2013-01-23 Thread Kim Phillips
cc'ing devicetree-discuss. Start using scripts/get_maintainers.pl. On Wed, 23 Jan 2013 11:04:55 +0530 Vakul Garg va...@freescale.com wrote: +++ b/Documentation/devicetree/bindings/crypto/fsl-sec4.txt @@ -54,8 +54,13 @@ PROPERTIES - compatible Usage: required Value type:

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: [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: [PATCH 4/4] dtc/libfdt: sparse fixes

2012-11-14 Thread Kim Phillips
On Wed, 14 Nov 2012 21:23:03 +1100 David Gibson da...@gibson.dropbear.id.au wrote: Hrm. The patch comment shows a lot of sparse warnings, but the patch looks like the basic conversion of the code from uint32_t to annontated fdt32_t types. not sure what you're getting at: isn't this to be

[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

[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

[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

[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

Re: [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

[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

[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

[PATCH 2/3] of/irq: sparse fixes

2012-10-08 Thread Kim Phillips
:got restricted __be32 *noident drivers/of/of_pci_irq.c:91:53: warning: incorrect type in argument 4 (different base types) drivers/of/of_pci_irq.c:91:53:expected unsigned int const [usertype] *addr drivers/of/of_pci_irq.c:91:53:got restricted __be32 *noident Signed-off-by: Kim

[PATCH 3/3] of/platform: sparse fix

2012-10-08 Thread Kim Phillips
drivers/of/platform.c:110:59: warning: incorrect type in argument 2 (different base types) drivers/of/platform.c:110:59:expected restricted __be32 const [usertype] *addr drivers/of/platform.c:110:59:got unsigned int const [usertype] *[assigned] reg Signed-off-by: Kim Phillips kim.phill

[PATCH] dt: bindings: move SEC node under new crypto/

2011-07-07 Thread Kim Phillips
Since technically it's not powerpc arch-specific. Also rename it sec2 to differentiate it from its incompatible successor, the SEC 4. Signed-off-by: Kim Phillips kim.phill...@freescale.com --- .../{powerpc/fsl/sec.txt = crypto/fsl-sec2.txt} |2 +- 1 files changed, 1 insertions(+), 1

[PATCH] crypto: caam - de-CHIP-ify device tree compatibles

2011-03-15 Thread Kim Phillips
) ever needed CHIP references. Signed-off-by: Kim Phillips kim.phill...@freescale.com Cc: Kumar Gala kumar.g...@freescale.com Cc: Scott Wood scottw...@freescale.com --- .../devicetree/bindings/crypto/fsl-sec4.txt| 64 arch/powerpc/boot/dts/p4080ds.dts

[PATCH] crypto: caam - standardize device tree naming convention to utilize '-vX.Y'

2011-03-14 Thread Kim Phillips
Help clarify that the number trailing in compatible nomenclature is the version number of the device, i.e., change: fsl,p4080-sec4.0, fsl,sec4.0; to: fsl,p4080-sec-v4.0, fsl,sec-v4.0; Signed-off-by: Kim Phillips kim.phill...@freescale.com Cc: Kumar Gala kumar.g...@freescale.com Cc: Steve

[PATCH 1/3] crypto: add device tree bindings for the Freescale SEC4/CAAM

2011-03-09 Thread Kim Phillips
Add SEC4 device tree binding documentation and add a SEC4 device node to the P4080's dts. Signed-off-by: Steve Cornelius s...@pobox.com Signed-off-by: Kim Phillips kim.phill...@freescale.com --- .../devicetree/bindings/crypto/fsl-sec4.txt| 409 arch/powerpc/boot/dts