31 Jan, 2018
5 commits
-
On top of RDIMM support, add new register calculation to support
3DS RDIMMs. Only symmetrical 3DS is supported at this time.Signed-off-by: York Sun
-
Wrong field was masked in this workaround due to wrong endianness. The
impacted SoCs have big-endian.Signed-off-by: York Sun
-
For DDR4, command/address delay in mode registers and parity latency
in timing config register are only needed for UDIMMs, but not RDIMMs.
Add additional register rcw_3 for DDR4 RDIMM. Fix mirrored bit for
dual rank RDIMMs. Set sdram_cfg_3[DIS_MRS_PAR] for RDIMMs. Fix
calculation of timing config registers. Use hexadecimal format for
printing RCW (register control word) registers.Signed-off-by: York Sun
-
Signed-off-by: Ashish Kumar
Reviewed-by: York Sun -
For for case of lazyapply method, API fdt_fixup_board_enet() gets
invoked before DPL being deployed. This leads to an issue that
fsl-mc fdt fixup status marked as fail and dprc driver didn't get
registered in linux boot.Fixes this issue by calling fdt_fixup_board_enet() for case when
DPL is deployed successfully in lazyapply method.Signed-off-by: Yogesh Gaur
Reviewed-by: York Sun
29 Jan, 2018
35 commits
-
Commit 608b0c4ad4e5ec0c ("serial: Use next serial device if probing fails")
added code to search for more serial devices if the default one was not
probed correctly.Unfortunately, that breaks omap3_evm. So while investigating why that is
the case, let's disable the full search for everyone but bcm283x where it
is needed.Fixes: 608b0c4ad4e5ec0c ("serial: Use next serial device if probing fails")
Reported-by: Derald D. Woods
Signed-off-by: Alexander Graf -
Fix a typo in help output (awailable -> available).
Tidy the grammar - not the board connects to a port, we do.While at it, be consistent in upper-casing the comments.
Fixes: eee4835d22 ("tools: Add Marvell recovery image download script")
Cc: Konstantin Porotchkin
Cc: Stefan Roese
Cc: Igal Liberman
Signed-off-by: Andreas Färber
Reviewed-by: Konstantin Porotchkin
Reviewed-by: Stefan Roese
Signed-off-by: Stefan Roese -
minicom doesn't inherit the baudrate from stty but uses its own
defaults, such as for example 57600, whereas we expect 115200 here.
Explicitly tell minicom which baudrate to use.Fixes: eee4835d22 ("tools: Add Marvell recovery image download script")
Cc: Konstantin Porotchkin
Cc: Stefan Roese
Cc: Igal Liberman
Signed-off-by: Andreas Färber
Reviewed-by: Konstantin Porotchkin
Reviewed-by: Stefan Roese
Signed-off-by: Stefan Roese -
The variable cfi_flash_num_flash_banks is defined iff
CONFIG_SYS_MAX_FLASH_BANKS_DETECT is defined, but it is used
unconditionally in the function cfi_flash_init_dm. This leads to a
undefined variable compile error when CONFIG_SYS_MAX_FLASH_BANKS_DETECT
is not defined, but DM is enabled.Fix this by always defining the cfi_flash_num_flash_banks variable.
Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
When long expressions surrounded by parentheses are split into multiple
lines, each consecutive line should be aligned with the corresponding
parenthesis. Fix all instances where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
Long lines (>80 characters) should be avoided where possible. Break up
some long lines where it's not detrimental to readability.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
In a while loop in cfi_flash.c the array "start" is accessed at the index
"sector" before the index variable "sector" is bounds-checked, which
might lead to accesses beyond the bounds of the array.Swap the order of the checks in the "&&" expression, so that the
short-circuit evaluation prevents out-of-bounds array accesses.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
checkpatch.pl complains about the spelling of ERR_TIMOUT. Since the
error is only used in a handful of files, we rename the error to
ERR_TIMEOUT.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
Camel-case naming should be avoided. Rename two camel-case variables,
and fix their usage accordingly.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
Strings should not be split accross multiple lines. Where possible and
not detrimental to readability, fix the instances where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
According to the C standard, pointer arithmetic for pointers of type
void is undefined behavior (the assumption that they're 8-bit wide is a
GCC-specific assumption). In the interest of keeping the code
standards-compliant, and also better communicate intent, switch all
void* variables where pointer arithmetic is used to u8* variables.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
The condition in if statements should not be used for variable
assignment. Instead, the assignment should be done in a separate step
beforehand. Fix all instances where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
void functions don't need an explicit return at the end.
Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
The opening brace of block statements should be attached to the
statement itself, and not be on a separate line.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
If in a loop, the if block in a if/else statement ends in a break, the
statements in the else blockcan be extracted, since the break stops the
execution.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
"Unkown" should be spelled "Unknown".
Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
In if/else statements, either both blocks (if and else) should have
braces or both blocks should not have braces, but mixed configurations
are discouraged. Fix all instances where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
"==" and "!=" bind tighter than the boolean operators, so parentheses
around them in compound logical statements are unnecessary. Fix all
instances where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
Comment blocks should end with a "*/" on a separate line, not with the
"*/" attached to the end of the last line of text. Fix all instances
where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
printf/debug statements should not include the file name as a hardcoded
string, but instead use the __func__ macro. Fix all instances where this
occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
When splitting long logical statements across multiple lines, the
logical operators should be at the end of the lines. Fix all instances
where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
Blocks with a single statement should not be enclosed in braces. Fix all
instances where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
There should be no consecutive blank lines, and no blank lines at the
end of blocks. But there should be blank lines between variable
declarations and code. Fix all instances where either occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
There should be spaces around operators, and no spaces between a cast
and the variable its being applied to. Fix all instances where this
occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
case statements should be at the same level of indent as the switch
keyword. Fix all instances where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
When casting to a pointer type, the asterisk should be attached to the
type name, not separated by a space. Fix all instances where this
occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
There should not be additional spaces when nesting parentheses
("( (...) )"). Fix all instances where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
In a pointer declaration there should not be a space between the
asterisk and the pointer name. Fix all instances where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
There should not be a space between a function name and a parenthesis
("func (...)"). Fix all instances where this occurs.Signed-off-by: Mario Six
Signed-off-by: Stefan Roese -
Patch queue for efi - 2018-01-28
This is the second part of patches for 2018.03-rc1, fixing
a few minor issues and adding a readme file for iSCSI booting. -
The appended README explains how U-Boot and iPXE can be used
to boot a diskless system from an iSCSI SAN.The maintainer for README.efi and README.iscsi is set.
Signed-off-by: Heinrich Schuchardt
[agraf: s/Adress/Address/]
Signed-off-by: Alexander Graf -
Now that we have %pD support in vsprintf we should avoid separate
logic for printing device paths in other places.Signed-off-by: Heinrich Schuchardt
Signed-off-by: Alexander Graf -
In an image is loaded from memory we do not have a device path.
Do not install NULL as device path in this case.Signed-off-by: Heinrich Schuchardt
Signed-off-by: Alexander Graf
