Follow bootz's pattern by moving the booti_setup to arch/arm/lib.
This allows to use booti_setup in other paths, e.g booting
an Android image containing Image format.
Note that kernel relocation is move out of booti_setup and it is the
caller's responsibility to do it and allows them do it differe
The second area of android image was intended to put a 2nd stage
bootloader but in practice were rarely used (in my knowledge).
An proposal was made to the AOSP to (re)use the second area as the dtb[1],
This patch itself doesn't depend on that proposal being accepted but it won't
be that helpful a
I rebased the two patches submitted (quite )a while ago on top of
current U-boot master fb4413295c765aa8c013650984dc2d908964c81d
(and there were no conflicts).
The first patch added the support of parsing the second area of Android image
so
that it can be used for good. On Poplar board, we ar
With the merge of the new u-boot.itb size check now the build for
pine64_plus_defconfig breaks, as this file gets too large:
=
u-boot.itb exceeds file size limit:
limit: 516096 bytes
actual: 521852 bytes
excess: 5756 bytes
make: *** [u-boot.itb] Error 1
=
One easily f
On Fri, Jan 26, 2018 at 02:45:29PM -0700, Simon Glass wrote:
> Hi Tom,
>
> Here are some additions to logging and 64-bit sandbox support.
>
>
> The following changes since commit ab12aa24e619b5e81cbde7de88c6d9a19f04313b:
>
> ARM: socfpga: Convert callers of cm_write_with_phase for
> wait_for
Hi Michael,
On Thu, Jan 25, 2018 at 11:06 AM, Michael Trimarchi
wrote:
> SION bit should be used in the situation that we need
> to read back the value of a pin and should not be set by
> default macro.
>
> We get some malfunction as raised by following thread
>
> https://www.spinics.net/lists/li
I'd like to build my own application - something like
examples/standalone/hello_world - but if possible, I'd like to build it
outside the u-boot tree. The standalone examples are meant to be invoked
with a recursive make from an ancestor directory in the u-boot tree. They
don't work if you just cop
Hi Michael,
On 26/01/2018 23:14, Michael Nazzareno Trimarchi wrote:
> Hi Stefano
>
> Do I need to resend with you in CC?
>
> For some reason I forget to include you
>
No, don't worry, I have already got it :-)
Regards,
Stefano
> Michael
>
> On Fri, Jan 26, 2018 at 12:00 PM, Benoît Thébaudea
Hi Stefano
Do I need to resend with you in CC?
For some reason I forget to include you
Michael
On Fri, Jan 26, 2018 at 12:00 PM, Benoît Thébaudeau wrote:
> On 25/01/2018 at 22:48, Benoît Thébaudeau wrote:
>> On Thu, Jan 25, 2018 at 2:06 PM, Michael Trimarchi
>> wrote:
>>> SION bit should be u
Signed-off-by: Álvaro Fernández Rojas
---
configs/comtrend_ar5315u_ram_defconfig | 1 +
configs/comtrend_ar5387un_ram_defconfig | 1 +
configs/comtrend_ct5361_ram_defconfig| 1 +
configs/comtrend_vr3032u_ram_defconfig | 1 +
configs/comtrend_wap5813n_ram_defconfig | 1 +
configs/netgear
I just found out I was missing OF_EMBED, which confirms that it was
working because CFE was relocating u-boot and not u-boot itself...
Sorry for the noise...
El 26/01/2018 a las 22:53, Álvaro Fernández Rojas escribió:
Hello Paul & Daniel,
I've finally had some time to do more testing and it
Hello Paul & Daniel,
I've finally had some time to do more testing and it appears that the
relocation error I saw on BMIPS is due to a DT not found error caused by
mips relocation:
https://gist.github.com/Noltari/9017b3d28e2109d1de4158e5ad5b1c65
Any ideas on how to fix it?
BTW, the reason
Hi Tom,
Here are some additions to logging and 64-bit sandbox support.
The following changes since commit ab12aa24e619b5e81cbde7de88c6d9a19f04313b:
ARM: socfpga: Convert callers of cm_write_with_phase for
wait_for_bit_le32 (2018-01-26 13:08:03 -0500)
are available in the Git repository at:
On 28 December 2017 at 13:14, Simon Glass wrote:
> Add a test of the 'log format' and 'log rec' commands. This also covers
> things like log_get_cat_by_name(), since they are used by these commands.
> Fix a style nit in the tests also.
>
> Signed-off-by: Simon Glass
> ---
>
> test/py/tests/test_
On 26 January 2018 at 08:25, Simon Glass wrote:
> On 22 January 2018 at 12:10, Heinrich Schuchardt wrote:
>> The EFI implementation does not fit into any of the existing categories.
>>
>> Provide LOGC_EFI so that EFI related message can be filtered.
>>
>> Signed-off-by: Heinrich Schuchardt
>> --
On 19 January 2018 at 00:17, Mario Six wrote:
> From: Mario Six
>
> To debug device tree issues involving 32- and 64-bit platforms, it is useful
> to
> have a generic 64-bit platform available.
>
> Add a version of the sandbox that uses 64-bit integers for its physical
> addresses as well as a m
On 28 December 2017 at 13:14, Simon Glass wrote:
> When functions return an error it propagates up the stack to the point
> where it is reported. Often the error code provides enough information
> about the root cause of the error that this is obvious what went wrong.
>
> However in some cases the
On 28 December 2017 at 13:14, Simon Glass wrote:
> Add a 'log rec' command which allows a log record to be manually output.
> This is useful for scripts which want full control over what is logged. It
> also permits easy testing of the log system.
>
> Signed-off-by: Simon Glass
> ---
>
> cmd/log
On 28 December 2017 at 13:14, Simon Glass wrote:
> Add a 'log format' command which can display or change the log output
> format. This is useful for changing how much information is displayed. The
> ordering of the fields is fixed.
>
> Signed-off-by: Simon Glass
> ---
>
> cmd/log.c | 43 +++
On 28 December 2017 at 13:14, Simon Glass wrote:
> Add some notes about recent new features.
>
> Signed-off-by: Simon Glass
> ---
>
> doc/README.log | 27 +++
> 1 file changed, 27 insertions(+)
Applied to u-boot-dm.
___
U-Boot
On 28 December 2017 at 13:14, Simon Glass wrote:
> It is useful to be able to control the output format of log records on the
> console. As a starting point, add definitions for controlling which
> elements of the log record are displayed. Use function and message as the
> default, since these are
On 28 December 2017 at 13:14, Simon Glass wrote:
> At present this just outputs the message. Update it to output whatever the
> format requests.
>
> Signed-off-by: Simon Glass
> ---
>
> common/log_console.c | 27 ++-
> test/py/tests/test_log.py | 4 ++--
> 2 files c
On 28 December 2017 at 13:14, Simon Glass wrote:
> Category and level both use an enum for their ID values. Add functions to
> convert these IDs to strings and vice versa. This will allow the log to
> output the strings instead of the (inscrutable) values.
>
> At the same time, add a new 'driver-m
On 28 December 2017 at 13:14, Simon Glass wrote:
> Each uclass has a driver name which we can use to look up the uclass. This
> is useful for logging, where the uclass ID is used as the category.
>
> Add a function to handle this, as well as a test.
>
> Signed-off-by: Simon Glass
> ---
>
> drive
On Fri, Jan 26, 2018 at 08:05:41PM +0530, Jagan Teki wrote:
> Hi Tom,
>
> Please pull this PR.
>
> thanks!
> Jagan.
>
> The following changes since commit 557767ed29968af0294e3aae48433e5d5a298e0b:
>
> Merge git://git.denx.de/u-boot-marvell (2018-01-20 08:39:47 -0500)
>
> are available in th
Signed-off-by: Álvaro Fernández Rojas
---
arch/mips/dts/Makefile | 1 +
arch/mips/dts/netgear,dgnd3700v2.dts | 121 +++
arch/mips/mach-bmips/Kconfig | 12 +++
board/netgear/dgnd3700v2/Kconfig | 12 +++
board/netgear/dgnd370
Signed-off-by: Álvaro Fernández Rojas
---
arch/mips/dts/brcm,bcm6362.dtsi| 186 +
arch/mips/mach-bmips/Kconfig | 12 ++
include/configs/bmips_bcm6362.h| 25 +++
include/dt-bindings/clock/bcm6362-clock.h |
Signed-off-by: Álvaro Fernández Rojas
---
drivers/cpu/bmips_cpu.c | 51 +
1 file changed, 51 insertions(+)
diff --git a/drivers/cpu/bmips_cpu.c b/drivers/cpu/bmips_cpu.c
index 4ad291a56e..6c612bacdc 100644
--- a/drivers/cpu/bmips_cpu.c
+++ b/driver
BCM6362 is a dual core BCM63xx SoC.
Álvaro Fernández Rojas (3):
dm: cpu: bmips: add BCM6362 support
MIPS: add support for Broadcom MIPS BCM6362 SoC family
MIPS: add BMIPS Netgear DGND3700v2 board
arch/mips/dts/Makefile | 1 +
arch/mips/dts/brcm,bcm6362.dtsi
README.efi describes two different concepts:
* U-Boot exposing the UEFI API
* U-Boot running on top of UEFI.
This patch splits the document in two.
Religious references are removed.
The separation of the concepts makes sense before detailing the internals
of U-Boot exposing the UEFI API in a futu
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
---
v2
mention work on TCP and wget
remove VLAN drawing
fix reference of EFI
On 01/26/2018 17:28, Tom Rini wrote:
> Now that we have and use wait_for_bit_le32() available, the callers of
> cm_write_with_phase() should not be casting values to u32 and instead we
> expect a const void *, so provide that directly.
>
> Fixes: 48263504c8d5 ("wait_bit: use wait_for_bit_le32 and
On Fri, Jan 26, 2018 at 01:45:32PM +0100, Daniel Schwierzeck wrote:
> The following changes since commit fb4413295c765aa8c013650984dc2d908964c81d:
>
> Merge git://git.denx.de/u-boot-mmc (2018-01-24 11:28:44 -0500)
>
> are available in the Git repository at:
>
> git://git.denx.de/u-boot-mips
On Fri, Jan 26, 2018 at 11:28:22AM -0500, Tom Rini wrote:
> Now that we have and use wait_for_bit_le32() available, the callers of
> cm_write_with_phase() should not be casting values to u32 and instead we
> expect a const void *, so provide that directly.
>
> Fixes: 48263504c8d5 ("wait_bit: use
On Wed, Jan 24, 2018 at 12:32:33AM +0100, Angelo Dureghello wrote:
> The following changes since commit c761a7e29d703d60208585bb7d8415e00aa22556:
>
> Revert "travis-ci: Add qemu-x86_64 target" (2018-01-22 21:06:41 -0500)
>
> are available in the Git repository at:
>
> git://git.denx.de/u-bo
On Fri, Jan 26, 2018 at 11:57:27AM +0530, Jagan Teki wrote:
> Hi Tom,
>
> Please pull this PR.
>
> Changes for v2:
> - Fixed patches on wait_bit
> - Collected new patches.
>
> thanks!
> Jagan.
>
> The following changes since commit 98691a60abffb44303d7dae6e9e699d0daded930:
>
> Merge git://g
For devices running I2C over 400KHz, this needs to be in HS mode.
Currently, the code is only running in HS mode when the speed is
equal or greater than 3.4MHz.
Using the Linux kernel setup as a reference, I used much of their
calculation for fsscll, fssclh, hsscll, and hssclh. However, Linux
use
On Fri, Jan 26, 2018 at 2:27 PM, Bryan O'Donoghue
wrote:
> This patch adds a sec_init call into arch_misc_init(). Doing so in
> conjunction with the patch "drivers/crypto/fsl: assign job-rings to
> non-TrustZone" enables use of the CAAM in Linux when OPTEE/TrustZone is
> active.
>
> u-boot will in
On Fri, 2018-01-26 at 16:27 +, Bryan O'Donoghue wrote:
> This patch adds a sec_init call into arch_misc_init(). Doing so in
> conjunction with the patch "drivers/crypto/fsl: assign job-rings to
> non-TrustZone" enables use of the CAAM in Linux when OPTEE/TrustZone
> is
> active.
>
> u-boot wil
On 01/26/2018 05:28 PM, Tom Rini wrote:
> Now that we have and use wait_for_bit_le32() available, the callers of
> cm_write_with_phase() should not be casting values to u32 and instead we
> expect a const void *, so provide that directly.
>
> Fixes: 48263504c8d5 ("wait_bit: use wait_for_bit_le32 a
On Fri, 2018-01-26 at 12:24 +, Bryan O'Donoghue wrote:
> This patch adds a sec_init call into arch_misc_init(). Doing so in
> conjunction with the patch "drivers/crypto/fsl: assign job-rings to
> non-TrustZone" enables use of the CAAM in Linux when OPTEE/TrustZone
> is
> active.
>
> u-boot wil
Now that we have and use wait_for_bit_le32() available, the callers of
cm_write_with_phase() should not be casting values to u32 and instead we
expect a const void *, so provide that directly.
Fixes: 48263504c8d5 ("wait_bit: use wait_for_bit_le32 and remove wait_for_bit")
Cc: Marek Vasut
Signed-o
This patch adds a sec_init call into arch_misc_init(). Doing so in
conjunction with the patch "drivers/crypto/fsl: assign job-rings to
non-TrustZone" enables use of the CAAM in Linux when OPTEE/TrustZone is
active.
u-boot will initialise the RNG and assign ownership of the job-ring
registers to a
V4:
- Fix dim-witted omission of #include to remove warning
V3:
- Changed location of sec_init() from warp.c::board_init() to
soc.c::arch_misc_init() which will allow any i.MX7 which defines
CONFIG_FSL_CAAM to forget about running sec_init().
V2:
- Add an explicit assignment of JRMID when se
After enabling TrustZone various parts of the CAAM silicon become
inaccessible to non TrustZone contexts. The job-ring registers are designed
to allow non TrustZone contexts like Linux to still submit jobs to CAAM
even after TrustZone has been enabled.
The default job-ring permissions after the Bo
On 01/26/2018 05:16 PM, Marek Vasut wrote:
> On 01/26/2018 05:05 PM, Stefano Babic wrote:
>> On 26/01/2018 15:57, Lukasz Majewski wrote:
>>> This patch tries to solve the problem described in following patch:
>>> https://patchwork.ozlabs.org/patch/796237/
>>>
>>> The main argument against the above
On 01/26/2018 05:05 PM, Stefano Babic wrote:
> On 26/01/2018 15:57, Lukasz Majewski wrote:
>> This patch tries to solve the problem described in following patch:
>> https://patchwork.ozlabs.org/patch/796237/
>>
>> The main argument against the above code was the potential lack of
>> consistency if
On 26/01/2018 15:57, Lukasz Majewski wrote:
> This patch tries to solve the problem described in following patch:
> https://patchwork.ozlabs.org/patch/796237/
>
> The main argument against the above code was the potential lack of
> consistency if we boot SPL from the SD card (and then eMMC may loa
On Fri, Jan 26, 2018 at 04:31:06PM +0100, Daniel Schwierzeck wrote:
> The build script should not manipulate shell flags (especially '-e').
> A non-zero exit value can also be catched with 'cmd || ret=$?'.
>
> Signed-off-by: Daniel Schwierzeck
Reviewed-by: Tom Rini
--
Tom
signature.asc
Des
On 01/26/2018 03:17 AM, Marc Zyngier wrote:
On 25/01/18 21:45, Nishanth Menon wrote:
As recommended by Arm in [1], ACR needs to be set[2] to enable
invalidation of BTB. This has to be enabled unconditionally for
ICIALLU to be functional on Cortex-A15 processors. Provide a config
Not quite. ACT
From: Heinrich Schuchardt
To: Duncan Hare ; Alexander Graf
Subject: Re: [U-Boot] [PATCH 1/1] efi_loader: add a README.iscsi describing
booting via iSCSI
On 01/25/2018 08:39 PM, Duncan Hare wrote:
>>> - Forwarded Message -
>>> From: Alexander Graf
>>> To: Heinrich Schuchardt
>>
Add a new option '-E' for treating all compiler warnings as errors.
Eventually this will pass 'KCFLAGS=-Werror' to Kbuild.
Signed-off-by: Daniel Schwierzeck
---
Changes in v2:
- replace 'W=err' with 'KCFLAGS=-Werror'
tools/buildman/builder.py | 5 -
tools/buildman/builderthread.py |
This forces all compiler warnings to be treated as errors.
Signed-off-by: Daniel Schwierzeck
---
Changes in v2: None
.travis.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.travis.yml b/.travis.yml
index 1e55e1b7f1..59d1dd99e8 100644
--- a/.travis.yml
+++ b/.travis.ym
The build script should not manipulate shell flags (especially '-e').
A non-zero exit value can also be catched with 'cmd || ret=$?'.
Signed-off-by: Daniel Schwierzeck
---
Changes in v2: None
.travis.yml | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/.travis.yml b/.tr
Probably not all users are aware of this possibility, thus add a
pointer to the README. Also add a useful example.
Signed-off-by: Daniel Schwierzeck
---
Changes in v2:
- new patch
README | 5 +
1 file changed, 5 insertions(+)
diff --git a/README b/README
index b53ea7dfe3..b055ae7ef2 1006
To enforce a zero-warnings policy (e.g. in CI builds), all compiler
warnings have to be treated as errors.
Extend Kbuild and buildman with according options to achieve this.
Enable these new options in all Travis CI builds. All builds with
compiler warnings will now fail. Only DTC warnings are sti
Hi Masahiro,
On 21 January 2018 at 02:34, Masahiro Yamada
wrote:
> All of these host tools are apparently written for Python2,
> not Python3.
>
> Use 'python2' in the shebang line according to PEP 394
> (https://www.python.org/dev/peps/pep-0394/).
>
> Signed-off-by: Masahiro Yamada
> ---
>
> I s
On 22 January 2018 at 12:10, Heinrich Schuchardt wrote:
> The EFI implementation does not fit into any of the existing categories.
>
> Provide LOGC_EFI so that EFI related message can be filtered.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> rebase on git://git.denx.de/u-boot-dm.gi
On 01/26/2018 03:57 PM, Lukasz Majewski wrote:
> This patch tries to solve the problem described in following patch:
> https://patchwork.ozlabs.org/patch/796237/
You should explain what the problem is in the commit message. Random
link to a random website which may go away at some point is useless
This patch tries to solve the problem described in following patch:
https://patchwork.ozlabs.org/patch/796237/
The main argument against the above code was the potential lack of
consistency if we boot SPL from the SD card (and then eMMC may load
u-boot proper).
This patch preserves this consisten
Hi Tom,
Please pull this PR.
thanks!
Jagan.
The following changes since commit 557767ed29968af0294e3aae48433e5d5a298e0b:
Merge git://git.denx.de/u-boot-marvell (2018-01-20 08:39:47 -0500)
are available in the git repository at:
git://git.denx.de/u-boot-sunxi.git master
for you to fetch c
On Wed, Jan 24, 2018 at 10:12 PM, Jagan Teki wrote:
> On Tue, Jan 16, 2018 at 2:14 PM, Maxime Ripard
> wrote:
>> Hi,
>>
>> As we discussed already, this is the list of options that we will need
>> to disable by default.
>>
>> I also added to the mix a size check, which even though it's not
>> rea
We should take the MX6DP and MX6QP options in consideration
in the I2C_PADS_INFO macro.
Based on a patch by Pierluigi Passaro
Signed-off-by: Eran Matityahu
---
arch/arm/include/asm/mach-imx/mxc_i2c.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/arch/arm/include/asm/mac
We should take the MX6DP and MX6QP options in consideration
when defining imx_iomux_v3_setup_pad().
Based on a patch by Pierluigi Passaro
Signed-off-by: Eran Matityahu
---
arch/arm/include/asm/mach-imx/iomux-v3.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/incl
This patch contain all the include headers for Broadwell-DE architecture.
Signed-off-by: Vincenzo Bove
---
.../asm/arch-broadwell-de/acpi/global_nvs.asl | 15 +
.../asm/arch-broadwell-de/acpi/irq_helper.h| 36 ++
.../asm/arch-broadwell-de/acpi/irqlinks.asl| 454 ++
Hi Siva,
Sorry for the very late reply!
> -Original Message-
> From: Siva Durga Prasad Paladugu [mailto:siva.durga.palad...@xilinx.com]
> Sent: Thursday, January 04, 2018 1:08 PM
> To: u-boot@lists.denx.de
> Cc: jagannadh.t...@gmail.com; Siva Durga Prasad Paladugu
>
> Subject: [PATCH v2
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
board/cobra5272/flash.c | 2 +-
common/flash.c
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 31 +++
1 file change
> Betreff: [PATCH 4/5] Convert CONFIG_SPL_NAND_BASE et al to Kconfig
>
> This converts the following to Kconfig:
>CONFIG_SPL_NAND_BASE
>CONFIG_SPL_NAND_LOAD
>CONFIG_SPL_NAND_SOFTECC
>CONFIG_SPL_NAND_RAW_ONLY
>
> Signed-off-by: Adam Ford
> ---
> configs/brppt1_nand_defconfig
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/
> Datum: 24.01.2018 21:57
> Betreff: [PATCH 3/5] Convert CONFIG_SPL_NAND_DRIVERS et al to Kconfig
>
> This converts the following to Kconfig:
>CONFIG_SPL_NAND_DRIVERS
>CONFIG_SPL_NAND_ECC
>
> Signed-off-by: Adam Ford
> ---
> configs/brppt1_nand_defconfig| 2 ++
> includ
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
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 6 +++---
1 file changed,
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 pointe
Camel-case naming should be avoided. Rename two camel-case variables,
and fix their usage accordingly.
Signed-off-by: Mario Six
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff -
"==" 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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 48 +
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 17 -
1 file changed, 8
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 26 +---
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/c
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 428
1
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 6 --
1 file cha
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 200
1 file chan
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/m
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
Blocks with a single statement should not be enclosed in braces. Fix all
instances where this occurs.
Signed-off-by: Mario Six
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 15 +--
1 file changed, 5 insertions(+), 10 deletions(-)
diff --
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 o
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 2 +-
1 file changed, 1 in
"Unkown" should be spelled "Unknown".
Signed-off-by: Mario Six
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/cfi_flash.c b/drivers/mtd/cfi_flash.c
index 53842443eb..4c5e156
void functions don't need an explicit return at the end.
Signed-off-by: Mario Six
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/mtd/cfi_flash.c b/drivers/mtd/cfi_flash.c
index 7aa379b0d8..d1635
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 48 ---
There should not be additional spaces when nesting parentheses
("( (...) )"). Fix all instances where this occurs.
Signed-off-by: Mario Six
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 22 +++---
1 file changed, 11 insertions(+), 11
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 4 ++--
1 file changed, 2 insertio
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
---
v2 -> v3:
* Added proper commit message
v1 -> v2:
None
---
drivers/mtd/cfi_flash.c | 18 +-
1 file cha
On 01/25/2018 08:39 PM, Duncan Hare wrote:
>> - Forwarded Message -
>> From: Alexander Graf
>> To: Heinrich Schuchardt
>
>>> 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.i
Hi Masahiro,
On 26.01.2018 02:09, Masahiro Yamada wrote:
> Hi Daniel,
>
>
> 2018-01-26 2:21 GMT+09:00 Daniel Schwierzeck :
>> Extend the Kbuild's W=N option with an optional 'err' value. This
>> will pass -Werror to the compiler to treat all warnings as errors.
>> This is useful to enforce a zer
On 24-01-2018 10:55, Rasmus Villemoes wrote:
It is sometimes useful to be able to define the entire default
environment in an external file. This implements a Kconfig option for
allowing that.
It is somewhat annoying to have two visible Kconfig options; it would
probably be more user-friendly to
reginfo is redundant with some of the commands in immap.c, so
move reginfo into that file and remove duplicated info.
Signed-off-by: Christophe Leroy
---
arch/powerpc/cpu/mpc8xx/Makefile | 1 -
arch/powerpc/cpu/mpc8xx/immap.c | 20
2 files changed, 20 insertions(+), 1 dele
CONFIG_8xx doesn't mean much outside of arch/powerpc.
This patch renames it CONFIG_MPC8xx just like CONFIG_MPC85xx etc ...
It also rename 8xx_immap.h to immap_8xx.h to be consistent with
other file names.
Signed-off-by: Christophe Leroy
---
api/api_platform-powerpc.c|
None of those values are used at the time being.
Just keep one and call it PVR_8xx
Signed-off-by: Christophe Leroy
---
arch/powerpc/include/asm/processor.h | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/arch/powerpc/include/asm/processor.h
b/arch/powerpc/include/asm/p
1 - 100 of 131 matches
Mail list logo
