11 Mar, 2018
6 commits
-
The README.mxc_hab is outdated and need improvements, add the following
modifications:- Reorganize document and remove duplicate content
- Add CST download link
- Update CST package name
- Align command lines with CST v2.3.3
- Update U-Boot binary name
- Remove CSF padding since is not documented in AN4581Signed-off-by: Breno Lima
-
Currently the High Assurance Boot procedure is documented in two
places:- doc/README.imx6
- doc/README.mxc_habIt is better to consolidate all HAB related information into
README.mxc_hab file, so move the content from README.imx6 to
README.mxc_hab.Signed-off-by: Breno Lima
Reviewed-by: Fabio Estevam -
commit cd2d46003ce1 ("arm: imx: hab: Add IVT header definitions") declares
struct ivt_header as "__attribute__((packed))".commit ed286bc80e9d ("imx: hab: Check if CSF is valid before
authenticating image") declares struct hab_hdr with __packed.This patch makes the __packed convention consistent.
Signed-off-by: Bryan O'Donoghue
Cc: Utkarsh Gupta
Cc: Breno Lima
Cc: Fabio Estevam
Reviewed-by: Fabio Estevam -
commit ed286bc80e9d ("imx: hab: Check if CSF is valid before authenticating
image") makes use of "__packed" as a prefix to the "struct hab_hdr"
declaration.With my compiler "gcc version 7.2.1 20171011 (Linaro GCC 7.2-2017.11)" we
get:./arch/arm/include/asm/mach-imx/hab.h:42:25: error: expected ‘=’, ‘,’, ‘;’,
‘asm’ or ‘__attribute__’ before ‘{’ token
struct __packed hab_hdr {Fix this problem by including
Signed-off-by: Bryan O'Donoghue
Cc: Utkarsh Gupta
Cc: Breno Lima
Cc: Fabio Estevam
Reviewed-by: Fabio Estevam -
This patch fixes the wrongly included dtsi file which was
breaking mainline support for Engicam i.CoreM6 DualLite/Solo RQS.Linux commit details for the same change as
"ARM: dts: imx6dl: Include correct dtsi file for Engicam i.CoreM6
DualLite/Solo RQS"
(sha1: c0c6bb2322964bd264b4ddedaa5776f40c709f0c)Signed-off-by: Jagan Teki
Reviewed-by: Fabio Estevam -
usdhc4 node need to update pinctrl, bus-width and non-removable
properties, sync the same from Linux.Signed-off-by: Jagan Teki
Reviewed-by: Fabio Estevam
10 Mar, 2018
9 commits
-
Rsync all defconfig files using moveconfig.py
Signed-off-by: Tom Rini
-
The system call used by mkimage to run dtc redirects stdout to a
temporary file. This can cause problems on Windows (with a MinGW
cross-compiled version). Using the "-o" dtc parameter avoids
this problem.Signed-off-by: Stefan Theil
Reviewed-by: Tom Rini -
Other filesystem drivers don't do this.
Signed-off-by: Marek Behun
-
NETWORK should be after NAND_FLASH.
Signed-off-by: Heinrich Schuchardt
-
kmerr: verify that malloc and calloc are followed by a check to verify
that we are not out of memory.badzero: Compare pointer-typed values to NULL rather than 0
Both checks are copied from the Linux kernel archive.
Signed-off-by: Heinrich Schuchardt
-
The iterator of list_for_each() is never NULL.
Identified with coccinelle.
Signed-off-by: Heinrich Schuchardt
-
After the UART was initialized, we may still have bogus data in the
RX queue if it was enabled with incorrect pin muxing before.So let's flush the RX queue whenever we initialize baud rates.
This fixes a regression with the dynamic pinmuxing code when enable_uart=1
is not set in config.txt on Raspberry Pis that use pl011 for serial.Fixes: caf2233b28 ("bcm283x: Add pinctrl driver")
Reported-by: Göran Lundberg
Reported-by: Peter Robinson
Signed-off-by: Alexander Graf
Tested-by: Peter Robinson
Tested-by: Tuomas Tynkkynen -
After the UART was initialized, we may still have bogus data in the
RX queue if it was enabled with incorrect pin muxing before.So let's flush the RX queue whenever we initialize baud rates.
This fixes a regression with the dynamic pinmuxing code when enable_uart=1
is not set in config.txt.Fixes: caf2233b28 ("bcm283x: Add pinctrl driver")
Reported-by: Göran Lundberg
Reported-by: Peter Robinson
Signed-off-by: Alexander Graf
Tested-by: Peter Robinson
Tested-by: Tuomas Tynkkynen
09 Mar, 2018
7 commits
-
The following config symbols are only defined once and never referenced
anywhere else:CONFIG_AT91SAM9263EK
CONFIG_AT91SAM9RLEK
CONFIG_BARIX_IPAM390
CONFIG_BOARD_H2200
CONFIG_EP9301
CONFIG_KZM_A9_GT
CONFIG_PICOSAM
CONFIG_PLATINUM_PICON
CONFIG_PLATINUM_TITANIUM
CONFIG_PM9261
CONFIG_PM9263
CONFIG_PM9G45
CONFIG_SIEMENS_DRACO
CONFIG_SIEMENS_PXM2
CONFIG_SIEMENS_RUT
CONFIG_SMDKC100
CONFIG_SMDKV310
CONFIG_STM32F4DISCOVERYMost of them are config symbols named after the respective boards which
seems to have been a standard practice at some point.Signed-off-by: Tuomas Tynkkynen
-
The following config symbols are only defined once and never referenced
anywhere else:CONFIG_ARM926EJS
CONFIG_CPUAT91
CONFIG_EXYNOS5800
CONFIG_SYS_CORTEX_R4Most of them are config symbols named after the respective SoCs which
seems to have been a standard practice at some point.Signed-off-by: Tuomas Tynkkynen
-
The following config symbols are only defined once and never referenced
anywhere else:CONFIG_DBAU1X00
CONFIG_PB1X00Most of them are config symbols named after the respective boards which
seems to have been a standard practice at some point.Signed-off-by: Tuomas Tynkkynen
-
The @gdsys.cc addresses are supposed to be used for mailing lists.
Switch all occurrences of @gdsys.de mail addresses to their @gdsys.cc
equivalent.Also, Dirk's address was wrong in one place; fix that as well.
Signed-off-by: Mario Six
-
CONFIG_SYS_CBSIZE determines the maximum length of the kernel command
line, and the default value of 256 is too small for booting some Linux
images in the wild.Signed-off-by: Tuomas Tynkkynen
-
Without the volatile attribute, compilers are entitled to optimize out
the same asm(). In the case of __udelay() in syscounter.c, it calls
`get_ticks()` twice, one for the starting time and the second in the
loop to check the current time. When compilers inline `get_ticks()`
they see the same `mrrc` instructions and optimize out the second one.
This leads to infinite loop since we don't get updated value from the
system counter.Here is a portion of the disassembly of __udelay:
88: 428b cmp r3, r1
8a: f8ce 20a4 str.w r2, [lr, #164] ; 0xa4
8e: bf08 it eq
90: 4282 cmpeq r2, r0
92: f8ce 30a0 str.w r3, [lr, #160] ; 0xa0
96: d3f7 bcc.n 88
98: e8bd 8cf0 ldmia.w sp!, {r4, r5, r6, r7, sl, fp, pc}Note that final jump / loop at 96 to 88, we don't have any `mrrc`.
With a volatile attribute, the above changes to this:
8a: ec53 2f0e mrrc 15, 0, r2, r3, cr14
8e: 42ab cmp r3, r5
90: f8c1 20a4 str.w r2, [r1, #164] ; 0xa4
94: bf08 it eq
96: 42a2 cmpeq r2, r4
98: f8c1 30a0 str.w r3, [r1, #160] ; 0xa0
9c: d3f5 bcc.n 8a
9e: e8bd 8cf0 ldmia.w sp!, {r4, r5, r6, r7, sl, fp, pc}
a2: bf00 nopI'm advised[1] to put volatile on all asm(), so this commit also adds it
to the asm() in timer_init().[1]: https://lists.denx.de/pipermail/u-boot/2018-March/322062.html
Signed-off-by: Yasushi SHOJI
Reviewed-by: Fabio Estevam -
Sometimes imximage throws the following error:
CFGS board/freescale/vf610twr/imximage.cfg.cfgtmp
CFGS board/freescale/vf610twr/imximage.cfg.cfgtmp
MKIMAGE u-boot-dtb.imx
Error: No BOOT_FROM tag in board/freescale/vf610twr/imximage.cfg.cfgtmp
arch/arm/mach-imx/Makefile:100: recipe for target 'u-boot-dtb.imx' failedLater on, when running mkimage for the u-boot.imx it will succeed in
finding the IVT offset.Looks like some race condition happening during parallel build when
processing mkimage for u-boot-dtb.imx and u-boot.imx.A proper fix still needs to be implemented, but as a workaround let's
remove the error when the IVT offset is not found.It is useful to have such message, especially during bring-up phase,
but the build error that it causes is severe, so better avoid the
build error for now.The error checking can be re-implemented later when we have a proper
fix.Reported-by: Breno Lima
Reported-by: Thomas Petazzoni
Signed-off-by: Fabio Estevam
06 Mar, 2018
2 commits
-
Signed-off-by: Tom Rini
05 Mar, 2018
16 commits
-
This was missing the 'S' line causing a warning from genboardscfg.py
Signed-off-by: Tom Rini
-
This should clean up a warning about a missing phy-cells
Signed-off-by: Adam Ford
-
POWER should be after ONENAND
Signed-off-by: Heinrich Schuchardt
-
Keep spelling.txt in sync with the version from kernel 4.16-rc4.
Signed-off-by: Fabio Estevam
Reviewed-by: Heinrich Schuchardt -
Due to evolution of the MMC driver and better support, let's
remove unnecessary undefs.Signed-off-by: Adam Ford
-
With the migration to Kconfig, the I2C block no longer exists in here.
Let's clean up the comment.Signed-off-by: Adam Ford
-
Enable DHCP support for this board.
Signed-off-by: Ed Bartosh
Signed-off-by: Marek Vasut
Signed-off-by: Tom Rini -
Thomas reported U-Boot failed to build host tools if libfdt-devel
package is installed because tools include libfdt headers from
/usr/include/ instead of using internal ones.This commit moves the header code:
include/libfdt.h -> include/linux/libfdt.h
include/libfdt_env.h -> include/linux/libfdt_env.hand replaces include directives:
#include -> #include
#include -> #includeReported-by: Thomas Petazzoni
Signed-off-by: Masahiro Yamada -
There are no headers for libfdt in lib/libfdt, as they are instead
located in scripts/dtc/libfdt. Specifying lib/libfdt for headers
inclusion in host tools results in using the system libfdt headers,
which is not what we want. Change this to the proper path.Signed-off-by: Paul Kocialkowski
-
Add a comment indicating that the German key map assumes code page 437.
Add support for character ² (square sign) in the German key map.
Signed-off-by: Heinrich Schuchardt
-
Signed-off-by: Marek Behun
-
The PCA953x driver uses "gpio@%x_" as the GPIO bank name, where "%x" is
instantiated with the I2C address of the chip. While this works, it
becomes very confusing if a board has multiple PCAs with the same
address on different I2C busses, and it also becomes an issue when a
GPIO's value is to be set via the 'gpio' command, because this command
only ever sets the value of the first device it encounters, leaving the
other devices inaccessible to the command.As to not break boards that rely on this naming scheme, we introduce a
new device tree string property "label" for the driver. If it exists, it
is used to build a bank name of the form "%s@%x_" (where %x is still
instantiated with the I2C address). If it does not exist, the legacy
labeling scheme is used.Signed-off-by: Mario Six
-
Rename CONFIG_SPL_USBETH_SUPPORT to CONFIG_SPL_USB_ETHER.
This enables users to block text using CONFIG_IS_ENABLED() instead
of resorting to #if ladders with SPL and non-SPL cases.Signed-off-by: Faiz Abbas
-
Treat the first equal sign as a key/value separation too. This makes
the script files compatible with mkenvimage input file format. It
won't support variables with equal signs anymore, but this seems not
really like a loss.Signed-off-by: Stefan Agner
-
Ensure that the Atmel TPM driver performs sufficient
validation of the length returned in the TPM response header.
This patch prevents memory corruption if the header contains a
length value that is larger than the destination buffer.Signed-off-by: Jeremy Boone
-
Ensure that the Infineon I2C and SPI TPM driver performs adequate
validation of the length extracted from the TPM response header.
This patch prevents integer underflow when the length was too small,
which could lead to memory corruption.Signed-off-by: Jeremy Boone
