In Documentation/devicetree/bindings/riscv/cpus.yaml I find for riscv,isa:
enum:
- rv64imac
- rv64imafdc
This implies that 'rv64imafc' or 'rv64imafdqc' would be illegal values
while these combinations of extensions would be compliant with "The
RISC-V Instruction Set Manual".
To
We can use just the same flags as for aarch64.
Signed-off-by: Heinrich Schuchardt
---
RISC-V patches for gnu-efi are not yet accepted in upstream. Cf.
[Gnu-efi-discuss] [PATCH 1/1] Initial support for RISCV64
https://sourceforge.net/p/gnu-efi/mailman/gnu-efi-discuss/thread/20210401153553.103286
On 08.03.21 11:55, Anup Patel wrote:
> On Mon, Mar 8, 2021 at 1:19 PM Atish Patra wrote:
>>
>> On Sat, Mar 6, 2021 at 4:12 AM Anup Patel wrote:
>>>
>>> On Sat, Mar 6, 2021 at 11:19 AM Heinrich Schuchardt
>>> wrote:
>>>>
>>>
v0.2 extension definitions")
Reported-by: Sean Anderson
Signed-off-by: Heinrich Schuchardt
---
arch/riscv/include/asm/sbi.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/riscv/include/asm/sbi.h b/arch/riscv/include/asm/sbi.h
index 99895d9c3bdd..d7027411d
506070
Unicode code point 0x2018 (‘) is counted as three characters
instead of one.
This code point is used in the GCC 11 warning sizeof-array-div.
Citing the warning verbatim lead to the observation.
Signed-off-by: Heinrich Schuchardt
---
foo.txt | 0
1 file changed, 0
UEFI variables ConInDev, ConOutDev, ConErrDev are volatile variables that
are set by the firmware on every reset. It does not make sense to set these
variables via the efivarfs filesystem.
Signed-off-by: Heinrich Schuchardt
---
drivers/firmware/efi/vars.c | 3 ---
1 file changed, 3 deletions
-off-by: Heinrich Schuchardt
---
Documentation/admin-guide/efi-stub.rst | 47 +++---
1 file changed, 27 insertions(+), 20 deletions(-)
diff --git a/Documentation/admin-guide/efi-stub.rst
b/Documentation/admin-guide/efi-stub.rst
index 833edb0d0bc4..9e1bb79e8655 100644
Add missing items to table of parameters set in the /chosen node by the EFI
stub.
Signed-off-by: Heinrich Schuchardt
---
Documentation/arm/uefi.rst | 7 +++
1 file changed, 7 insertions(+)
diff --git a/Documentation/arm/uefi.rst b/Documentation/arm/uefi.rst
index f732f957421f..9b0b5e458a1e
EFI_RUNTIME_GET_REVISION \
_IOR('p', 0x0D, unsigned int)
unsigned int revision;
fd = open("/dev/efi_test", O_RDWR);
ret = ioctl(fd, EFI_RUNTIME_GET_REVISION, );
For UEFI specification 2.8 this will yield
revision == 0x250 == (2 << 16) | 80
Signed-off-by: Heinrich Schuchardt
On 11/27/20 8:20 PM, Heinrich Schuchardt wrote:
Since the UEFI 2.8A specification the UEFI enabled firmware provides a
configuration table EFI_RT_PROPERTIES_TABLE which indicates which runtime
services are enabled. The EFI stub reads this table and saves the value of
the field
On 12/22/20 11:12 AM, Heinrich Schuchardt wrote:
On 12/22/20 9:56 AM, Marek Szyprowski wrote:
Dump adc-keys bindings documentation from Linux kernel source tree v5.10.
Signed-off-by: Marek Szyprowski
---
doc/device-tree-bindings/input/adc-keys.txt | 49 +
1 file changed
is closest to press-threshold-microvolt.
This patch nails down the definitions and provides an interpretation of the
supplied example.
Signed-off-by: Heinrich Schuchardt
---
I know that this file needs to be converted to YAML. But lets first get the
text right.
---
.../devicetree/bindings/input/ad
On 12/22/20 9:56 AM, Marek Szyprowski wrote:
Dump adc-keys bindings documentation from Linux kernel source tree v5.10.
Signed-off-by: Marek Szyprowski
---
doc/device-tree-bindings/input/adc-keys.txt | 49 +
1 file changed, 49 insertions(+)
create mode 100644
On 14.12.20 18:01, Ilias Apalodimas wrote:
> At the moment the EFI stub tries to load an initrd from the
> cmdline provided option only if the LoadFile2 protocol does not exist
> on the initrd device path.
>
> This might prove problematic for EFI installers that need their own
> version of initrd
The following commit has been merged into the efi/core branch of tip:
Commit-ID: ff20661bb54cd57a18207b33cc57eb8d5c758a86
Gitweb:
https://git.kernel.org/tip/ff20661bb54cd57a18207b33cc57eb8d5c758a86
Author:Heinrich Schuchardt
AuthorDate:Fri, 27 Nov 2020 20:20:51 +01:00
On 12/3/20 2:20 AM, ivanhu wrote:
On 12/2/20 7:38 PM, Heinrich Schuchardt wrote:
On 11/30/20 11:38 AM, ivanhu wrote:
On 11/30/20 5:17 PM, Heinrich Schuchardt wrote:
On 11/30/20 9:16 AM, ivanhu wrote:
Hi Heinrich,
Thanks for the patch.
It looks good to me, but I noticed
On 11/30/20 11:38 AM, ivanhu wrote:
On 11/30/20 5:17 PM, Heinrich Schuchardt wrote:
On 11/30/20 9:16 AM, ivanhu wrote:
Hi Heinrich,
Thanks for the patch.
It looks good to me, but I noticed that the runtime_supported_mask was
introduced after 5.7-rc1.
Maybe we should add the kernel version
.
Best regards
Heinrich
Cheers,
Ivan
On 11/28/20 3:20 AM, Heinrich Schuchardt wrote:
Since the UEFI 2.8A specification the UEFI enabled firmware provides a
configuration table EFI_RT_PROPERTIES_TABLE which indicates which runtime
services are enabled. The EFI stub reads this table and saves
("net: phy: realtek: fix rtl8211e rx/tx delay config")
Signed-off-by: Heinrich Schuchardt
---
arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo-plus2.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h5-nanopi-neo-plus2.dts
b
("net: phy: realtek: fix rtl8211e rx/tx delay config")
Signed-off-by: Heinrich Schuchardt
---
arch/arm64/boot/dts/allwinner/sun50i-a64-sopine-baseboard.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-sopine-baseboard.dts
b
On 11/27/20 8:29 PM, Ard Biesheuvel wrote:
On Fri, 27 Nov 2020 at 20:28, Colin Ian King wrote:
On 27/11/2020 19:20, Heinrich Schuchardt wrote:
Since the UEFI 2.8A specification the UEFI enabled firmware provides a
configuration table EFI_RT_PROPERTIES_TABLE which indicates which runtime
= open("/dev/efi_test", O_RDWR);
ret = ioctl(fd, EFI_RUNTIME_GET_SUPPORTED_MASK, );
Signed-off-by: Heinrich Schuchardt
---
drivers/firmware/efi/test/efi_test.c | 16
drivers/firmware/efi/test/efi_test.h | 3 +++
2 files changed, 19 insertions(+)
diff --git
On 14.11.20 00:52, Daniel Kiper wrote:
> Hey,
>
> This is next attempt to create firmware and bootloader log specification.
> Due to high interest among industry it is an extension to the initial
> bootloader log only specification. It takes into the account most of the
> comments which I got up
The following commit has been merged into the efi/core branch of tip:
Commit-ID: bc13809f1c47245cd584f4ad31ad06a5c5f40e54
Gitweb:
https://git.kernel.org/tip/bc13809f1c47245cd584f4ad31ad06a5c5f40e54
Author:Heinrich Schuchardt
AuthorDate:Sat, 03 Oct 2020 08:03:56 +02:00
On 03.10.20 09:34, Atish Patra wrote:
> On Fri, Oct 2, 2020 at 11:38 PM Heinrich Schuchardt
> wrote:
>>
>> Describe how a device tree and an initial RAM disk can be passed to the EFI
>> Boot Stub.
>>
>> Signed-off-by: Heinrich Schuchar
in
the admin-guide.
Heinrich Schuchardt (2):
docs: admin-guide: fdt and initrd load in EFI stub
docs: admin-guide: EFI stub to main Linux handoff
Documentation/admin-guide/efi-stub.rst | 47 ++
1 file changed, 47 insertions(+)
--
2.28.0
Describe how a device tree and an initial RAM disk can be passed to the EFI
Boot Stub.
Signed-off-by: Heinrich Schuchardt
---
v2:
mention EFI_GENERIC_STUB_INITRD_CMDLINE_LOADER (thx Atish)
---
Documentation/admin-guide/efi-stub.rst | 35 ++
1 file changed, 35
Describe the handoff from the EFI stub to the main Linux kernel.
Signed-off-by: Heinrich Schuchardt
---
v2:
new patch
---
Documentation/admin-guide/efi-stub.rst | 12
1 file changed, 12 insertions(+)
diff --git a/Documentation/admin-guide/efi-stub.rst
b/Documentation
CONFIG_EFI_MIXED depends on CONFIG_X86_64=y.
There is no need to check CONFIG_X86_64 again.
Signed-off-by: Heinrich Schuchardt
---
arch/x86/include/asm/efi.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/x86/include/asm/efi.h b/arch/x86/include/asm/efi.h
index b9c2667ac46c
The following commit has been merged into the x86/urgent branch of tip:
Commit-ID: 0c7689830e907668288a1a1da84dca66dbdb4728
Gitweb:
https://git.kernel.org/tip/0c7689830e907668288a1a1da84dca66dbdb4728
Author:Heinrich Schuchardt
AuthorDate:Fri, 02 Oct 2020 21:06:23 +02:00
On 10/2/20 9:21 PM, Ard Biesheuvel wrote:
> On Fri, 2 Oct 2020 at 21:14, Heinrich Schuchardt wrote:
>>
>> On 10/2/20 7:21 PM, Ard Biesheuvel wrote:
>>> Hi Heinrich,
>>>
>>> Thanks for documenting this.
>>>
>>>
>>> On Fri, 2
On 10/2/20 7:21 PM, Ard Biesheuvel wrote:
> Hi Heinrich,
>
> Thanks for documenting this.
>
>
> On Fri, 2 Oct 2020 at 19:11, Heinrich Schuchardt wrote:
>>
>> Describe how a device tree and an initial RAM disk can be passed to the EFI
>> Boot Stub.
>
File zero-page.txt does not exit. Add links for zero-page.rst.
Signed-off-by: Heinrich Schuchardt
---
Documentation/x86/boot.rst | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/Documentation/x86/boot.rst b/Documentation/x86/boot.rst
index 5325c71ca877..49c3ebe8a439
On 02.10.20 19:21, Ard Biesheuvel wrote:
> Hi Heinrich,
>
> Thanks for documenting this.
>
>
> On Fri, 2 Oct 2020 at 19:11, Heinrich Schuchardt wrote:
>>
>> Describe how a device tree and an initial RAM disk can be passed to the EFI
>> Boot Stub.
>
Describe how a device tree and an initial RAM disk can be passed to the EFI
Boot Stub.
Signed-off-by: Heinrich Schuchardt
---
Documentation/admin-guide/efi-stub.rst | 35 ++
1 file changed, 35 insertions(+)
diff --git a/Documentation/admin-guide/efi-stub.rst
b
The reusable and the no-map property are mutually exclusive.
Clarify this in the documentation.
Signed-off-by: Heinrich Schuchardt
---
.../devicetree/bindings/reserved-memory/reserved-memory.txt| 3 +++
1 file changed, 3 insertions(+)
diff --git
a/Documentation/devicetree/bindings
On 07.09.20 09:00, Maxim Uvarov wrote:
> On Fri, 4 Sep 2020 at 18:50, Heinrich Schuchardt wrote:
>>
>> In the memory map the regions with the lowest addresses may be of type
>> EFI_RESERVED_TYPE. The reserved areas may be discontinuous relative to the
>> rest of the
not
be taken into account.
Signed-off-by: Heinrich Schuchardt
---
drivers/firmware/efi/libstub/efi-stub.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/firmware/efi/libstub/efi-stub.c
b/drivers/firmware/efi/libstub/efi-stub.c
index c2484bf75c5d..13058ac75765 100644
On 7/5/20 9:34 AM, Greg Kroah-Hartman wrote:
> On Sun, Jul 05, 2020 at 03:41:48AM +0200, Heinrich Schuchardt wrote:
>> On 7/4/20 4:02 PM, Greg Kroah-Hartman wrote:
>>> Test the functionality of readfile(2) in various ways.
>>
>> Hello Greg,
>>
>> I exp
On 7/4/20 4:02 PM, Greg Kroah-Hartman wrote:
> readfile(2) is a new syscall to remove the need to do the
> open/read/close dance for small virtual files in places like procfs or
> sysfs.
>
> Signed-off-by: Greg Kroah-Hartman
> ---
>
> This patch is for the man-pages project, not the kernel source
On 7/4/20 4:02 PM, Greg Kroah-Hartman wrote:
> Test the functionality of readfile(2) in various ways.
Hello Greg,
I expect readfile() to generate fanotify events FAN_OPEN_PERM, FAN_OPEN,
FAN_ACCESS_PERM, FAN_ACCESS, FAN_CLOSE_NOWRITE in this sequence.
Looking at patch 1/3 you took care of
On 6/26/20 1:45 AM, Atish Patra wrote:
> arm-init is responsible for setting up efi runtime and doesn't actually
> do any ARM specific stuff. RISC-V can use the same source code as it is.
>
> Rename it to efi-init so that RISC-V can use it.
>
> Signed-off-by: Atish Patra
> ---
>
On 6/26/20 1:45 AM, Atish Patra wrote:
> 32bit gcc doesn't support modulo operation on 64 bit data. It results in
> a __umoddi3 error while building EFI for 32 bit.
>
> Use bitwise operations instead of modulo operations to fix the issue.
>
> Signed-off-by: Atish Patra
> ---
>
The following commit has been merged into the efi/urgent branch of tip:
Commit-ID: 8c0a839c2bccb756454e35e8977e44fcf2bd417e
Gitweb:
https://git.kernel.org/tip/8c0a839c2bccb756454e35e8977e44fcf2bd417e
Author:Heinrich Schuchardt
AuthorDate:Tue, 16 Jun 2020 01:42:31 +02:00
Document the efi=novamap kernel command line parameter.
Put the efi parameters into alphabetic order.
Signed-off-by: Heinrich Schuchardt
---
.../admin-guide/kernel-parameters.txt | 18 ++
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git a/Documentation/admin
Provide missing descriptions for EFI stub helper functions.
Adjust formatting of existing descriptions to kernel style.
Signed-off-by: Heinrich Schuchardt
---
.../firmware/efi/libstub/efi-stub-helper.c| 78 ---
drivers/firmware/efi/libstub/efistub.h| 10 ++-
2 files
Let the description of the efi/libstub/random.c functions appear in the
kernel API documentation in the following chapters:
The Linux driver implementer’s API guide
Linux Firmware API
UEFI Support
UEFI stub library functions
Signed-off-by: Heinrich
The lengths of underlines must match the titles to avoid build warnings.
Signed-off-by: Heinrich Schuchardt
---
.../hw-vuln/special-register-buffer-data-sampling.rst | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
a/Documentation/admin-guide/hw-vuln/special
)
| ^~~~
Guard cxgb4_uld_in_use() with #ifdef CONFIG_CHELSIO_TLS_DEVICE.
Signed-off-by: Heinrich Schuchardt
---
drivers/net/ethernet/chelsio/cxgb4/cxgb4_uld.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_uld.c
b/drivers/net/ethernet/chelsio
Underline lengths should match the lengths of headings to avoid build
warnings with Sphinx.
Signed-off-by: Heinrich Schuchardt
---
Documentation/dev-tools/coccinelle.rst | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/Documentation/dev-tools/coccinelle.rst
b
Fix CAAM related typos.
Signed-off-by: Heinrich Schuchardt
---
v2:
fix additional typos as indicated by Horia
replaces https://lkml.org/lkml/2020/6/3/1129
---
drivers/crypto/caam/Kconfig | 2 +-
drivers/crypto/caam/ctrl.c | 18 +-
drivers/crypto/caam/desc.h
%s/suppying/supplying/
Signed-off-by: Heinrich Schuchardt
---
drivers/crypto/caam/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/crypto/caam/Kconfig b/drivers/crypto/caam/Kconfig
index a62f228be6da..bc35aa0ec07a 100644
--- a/drivers/crypto/caam/Kconfig
+++ b
The following commit has been merged into the efi/urgent branch of tip:
Commit-ID: 081d5150845ba3fa49151a2f55d3cc03b0987509
Gitweb:
https://git.kernel.org/tip/081d5150845ba3fa49151a2f55d3cc03b0987509
Author:Heinrich Schuchardt
AuthorDate:Sun, 26 Apr 2020 21:49:46 +02:00
On 9/22/19 8:58 AM, Michal Hocko wrote:
> On Thu 19-09-19 14:33:24, Eric W. Biederman wrote:
>> Michal Hocko writes:
>>
>>> On Tue 17-09-19 12:26:18, Eric W. Biederman wrote:
> [...]
Michal is it a very small effect your customers are seeing?
Is it another bug somewhere else?
>>>
>>> I
On 9/17/19 12:03 PM, Michal Hocko wrote:
> Hi,
> I have just stumbled over 16db3d3f1170 ("kernel/sysctl.c: threads-max
> observe limits") and I am really wondering what is the motivation behind
> the patch. We've had a customer noticing the threads_max autoscaling
> differences btween 3.12 and
On 9/5/19 11:20 AM, Stefan Hajnoczi wrote:
On Wed, Sep 04, 2019 at 08:07:36PM +0200, Heinrich Schuchardt wrote:
If an application tries to access memory that is not mapped, an error
ENOSYS, "load/store instruction decoding not implemented" may occur.
QEMU will hang with a reg
ment.
Now the virtual machine can react to the event in the most appropriate
way - by recovering, by writing an informative log, or by rebooting.
Signed-off-by: Heinrich Schuchardt
---
virt/kvm/arm/mmio.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/virt/kvm/arm/mmio.c
On 5/17/19 10:08 PM, Gregory CLEMENT wrote:
> Hi Heinrich Schuchardt,
>
>> Running a graphics adapter on the MACCHIATObin fails due to an
>> insufficently sized memory window.
> I think "insufficient" is enough or I miss something.
Thanks for reviewing. Do I have t
=512K]
Signed-off-by: Heinrich Schuchardt
---
arch/arm64/boot/dts/marvell/armada-8040-mcbin.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dtsi
b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dtsi
index 329f8ceeebea..205071b45a32 100644
This patch avoids
ERROR: "__sync_icache_dcache" [drivers/xen/xen-privcmd.ko] undefined!
observed when compiling v4.19.34.
The xen-privcmd driver, which can be modular, calls set_pte_at()
which in turn may call __sync_icache_dcache().
Cc: sta...@vger.kernel.org
Signed-off-by: Heinrich
On 1/10/19 12:04 PM, Gregory CLEMENT wrote:
> Hi Heinrich,
>
> On ven., déc. 21 2018, Heinrich Schuchardt wrote:
>
>> The memory area [0x400-0x420[ is occupied by the PSCI firmware. Any
>> attempt to access it from Linux leads to an immediate crash.
>>
&g
On 7/1/18 1:08 PM, Andreas Färber wrote:
> The IMST WiMOD uses a SLIP based binary UART protocol. Two separate
> firmwares are available. By default it ships with a LoRaWAN firmware.
> The alternative firmware is a custom P2P addressing mode based on LoRa.
>
> Cc: Jon Ortego
> Signed-off-by:
On 12/26/18 5:16 PM, Gregory CLEMENT wrote:
> Hi Heinrich,
>
> On ven., déc. 21 2018, Heinrich Schuchardt wrote:
>
>> The memory area [0x400-0x420[ is occupied by the PSCI firmware. Any
>> attempt to access it from Linux leads to an immediate crash.
>>
&g
The memory area [0x400-0x420[ is occupied by the PSCI firmware. Any
attempt to access it from Linux leads to an immediate crash.
So let's make the same memory reservation as the vendor kernel.
Signed-off-by: Heinrich Schuchardt
---
v2
Move the change from armada-8040-mcbin.dts
The memory area [0x400-0x420[ is occupied by the PSCI firmware. Any
attempt to access it from Linux leads to an immediate crash.
So let's make the same memory reservation as the vendor kernel.
Signed-off-by: Heinrich Schuchardt
---
arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts | 11
On 12/17/18 11:42 PM, Ard Biesheuvel wrote:
> On Mon, 17 Dec 2018 at 23:33, Heinrich Schuchardt wrote:
>>
>> On 12/17/18 7:16 PM, tip-bot for Heinrich Schuchardt wrote:
>>> Commit-ID: 793423cf07e51e3185b8680167115813589c057d
>>> Gitweb:
On 12/17/18 7:16 PM, tip-bot for Heinrich Schuchardt wrote:
> Commit-ID: 793423cf07e51e3185b8680167115813589c057d
> Gitweb:
> https://git.kernel.org/tip/793423cf07e51e3185b8680167115813589c057d
> Author: Heinrich Schuchardt
> AuthorDate: Mon, 17 Dec 2018 19:02:14 +0
Commit-ID: 793423cf07e51e3185b8680167115813589c057d
Gitweb: https://git.kernel.org/tip/793423cf07e51e3185b8680167115813589c057d
Author: Heinrich Schuchardt
AuthorDate: Mon, 17 Dec 2018 19:02:14 +0100
Committer: Ingo Molnar
CommitDate: Mon, 17 Dec 2018 19:12:48 +0100
efi: Align
The UEFI Specification Version 2.7 Errata A defines:
"EFI_GUID
128-bit buffer containing a unique identifier value.
Unless otherwise specified, aligned on a 64-bit boundary."
Before this patch efi_guid_t was 8-bit aligned.
Signed-off-by: Heinrich Schuchardt
---
include/linux/efi.h
On 06/20/2018 09:57 PM, Heinrich Schuchardt wrote:
> On 06/20/2018 11:15 AM, Heiko Stübner wrote:
>> Hi Heinrich,
>>
>> Am Mittwoch, 20. Juni 2018, 07:59:34 CEST schrieb Heinrich Schuchardt:
>>> On 06/20/2018 01:21 AM, Heiko Stuebner wrote:
>>>> Am Donner
On 06/20/2018 09:57 PM, Heinrich Schuchardt wrote:
> On 06/20/2018 11:15 AM, Heiko Stübner wrote:
>> Hi Heinrich,
>>
>> Am Mittwoch, 20. Juni 2018, 07:59:34 CEST schrieb Heinrich Schuchardt:
>>> On 06/20/2018 01:21 AM, Heiko Stuebner wrote:
>>>> Am Donner
On 06/20/2018 11:15 AM, Heiko Stübner wrote:
> Hi Heinrich,
>
> Am Mittwoch, 20. Juni 2018, 07:59:34 CEST schrieb Heinrich Schuchardt:
>> On 06/20/2018 01:21 AM, Heiko Stuebner wrote:
>>> Am Donnerstag, 14. Juni 2018, 14:55:27 CEST schrieb Heiko Stuebner:
>>>&g
On 06/20/2018 11:15 AM, Heiko Stübner wrote:
> Hi Heinrich,
>
> Am Mittwoch, 20. Juni 2018, 07:59:34 CEST schrieb Heinrich Schuchardt:
>> On 06/20/2018 01:21 AM, Heiko Stuebner wrote:
>>> Am Donnerstag, 14. Juni 2018, 14:55:27 CEST schrieb Heiko Stuebner:
>>>&g
On 06/20/2018 01:21 AM, Heiko Stuebner wrote:
> Hi Heinrich,
>
> Am Donnerstag, 14. Juni 2018, 14:55:27 CEST schrieb Heiko Stuebner:
>> Am Montag, 4. Juni 2018, 19:15:23 CEST schrieb Heinrich Schuchardt:
>>> Without this patch the Firefly-RK3399 board boot process han
On 06/20/2018 01:21 AM, Heiko Stuebner wrote:
> Hi Heinrich,
>
> Am Donnerstag, 14. Juni 2018, 14:55:27 CEST schrieb Heiko Stuebner:
>> Am Montag, 4. Juni 2018, 19:15:23 CEST schrieb Heinrich Schuchardt:
>>> Without this patch the Firefly-RK3399 board boot process han
On 06/04/2018 07:15 PM, Heinrich Schuchardt wrote:
> Without this patch the Firefly-RK3399 board boot process hangs after these
> lines:
>
>fan53555-regulator 0-0040: FAN53555 Option[8] Rev[1] Detected!
>fan53555-reg: supplied by vcc_sys
>vcc1v8_s3: s
On 06/04/2018 07:15 PM, Heinrich Schuchardt wrote:
> Without this patch the Firefly-RK3399 board boot process hangs after these
> lines:
>
>fan53555-regulator 0-0040: FAN53555 Option[8] Rev[1] Detected!
>fan53555-reg: supplied by vcc_sys
>vcc1v8_s3: s
of fcs,suspend-voltage-selector different to
any other board.
Changing this setting to the usual value is sufficient to enable booting.
Signed-off-by: Heinrich Schuchardt
---
arch/arm64/boot/dts/rockchip/rk3399-firefly.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch
of fcs,suspend-voltage-selector different to
any other board.
Changing this setting to the usual value is sufficient to enable booting.
Signed-off-by: Heinrich Schuchardt
---
arch/arm64/boot/dts/rockchip/rk3399-firefly.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch
This patch creates a false positive:
file '0001-foo.patch' doesn't appear to be a patch.
Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
samples/mic/noned/none | 0
1 file changed, 0 insertions(+), 0 deletions(-)
mode change 100755 => 100644 samples/mic/noned/none
d
This patch creates a false positive:
file '0001-foo.patch' doesn't appear to be a patch.
Signed-off-by: Heinrich Schuchardt
---
samples/mic/noned/none | 0
1 file changed, 0 insertions(+), 0 deletions(-)
mode change 100755 => 100644 samples/mic/noned/none
diff --git a/samples/mic/noned/non
This patch creates a false positive:
ERROR: Does not appear to be a unified-diff format patch
Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
samples/mic/noned/none | 0
1 file changed, 0 insertions(+), 0 deletions(-)
mode change 100755 => 100644 samples/mic/noned/none
d
This patch creates a false positive:
ERROR: Does not appear to be a unified-diff format patch
Signed-off-by: Heinrich Schuchardt
---
samples/mic/noned/none | 0
1 file changed, 0 insertions(+), 0 deletions(-)
mode change 100755 => 100644 samples/mic/noned/none
diff --git a/samples/mic/no
On 05/07/2018 10:43 PM, Joe Perches wrote:
> On Mon, 2018-05-07 at 20:31 +0200, Heinrich Schuchardt wrote:
>> This patch creates a false positive:
>> ERROR: Macros with complex values should be enclosed in parentheses
>>
>> Here we define a constant that can be use
On 05/07/2018 10:43 PM, Joe Perches wrote:
> On Mon, 2018-05-07 at 20:31 +0200, Heinrich Schuchardt wrote:
>> This patch creates a false positive:
>> ERROR: Macros with complex values should be enclosed in parentheses
>>
>> Here we define a constant that can be use
a function
Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
foo.h | 8
1 file changed, 8 insertions(+)
create mode 100644 foo.h
diff --git a/foo.h b/foo.h
new file mode 100644
index ..e2cba533f065
--- /dev/null
+++ b/foo.h
@@ -0,0 +1,8 @@
+/* SPDX-License-Identifier:
a function
Signed-off-by: Heinrich Schuchardt
---
foo.h | 8
1 file changed, 8 insertions(+)
create mode 100644 foo.h
diff --git a/foo.h b/foo.h
new file mode 100644
index ..e2cba533f065
--- /dev/null
+++ b/foo.h
@@ -0,0 +1,8 @@
+/* SPDX-License-Identifier: BSD-2 */
+#define
The Asus Tinker Board uses serial 2 with 115,200 baud by default for
communication in U-Boot. The same value is also chosen for other RK3288
boards.
So let us set the same value in the Tinker Board device tree.
Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
arch/arm/boot/dts/
The Asus Tinker Board uses serial 2 with 115,200 baud by default for
communication in U-Boot. The same value is also chosen for other RK3288
boards.
So let us set the same value in the Tinker Board device tree.
Signed-off-by: Heinrich Schuchardt
---
arch/arm/boot/dts/rk3288-tinker.dts | 4
Allow a space between a colon and subsequent opening bracket.
This sequence may occur in inline assembler statements like
asm(
"ldr %[out], [%[in]]\n\t"
: [out] "=r" (ret)
: [in] "r" (addr)
);
Sign
Allow a space between a colon and subsequent opening bracket.
This sequence may occur in inline assembler statements like
asm(
"ldr %[out], [%[in]]\n\t"
: [out] "=r" (ret)
: [in] "r" (addr)
);
Sign
This patch leads to errors "space prohibited before open square
bracket '['" for the asm statement.
In the inline assembler statement the bracket is not used for an index.
Adding a space after a colon or a comma should be allowable.
Signed-off-by: Heinrich Schuchardt <xypro
This patch leads to errors "space prohibited before open square
bracket '['" for the asm statement.
In the inline assembler statement the bracket is not used for an index.
Adding a space after a colon or a comma should be allowable.
Signed-off-by: Heinrich Schuchardt
---
musb->endpoints[] has array size MUSB_C_NUM_EPS.
We must check array bounds before accessing the array and not afterwards.
Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
v3
Remove superfluous braces.
v2
Only the 4 low bits of epnum are relevant for
musb->endpoints[] has array size MUSB_C_NUM_EPS.
We must check array bounds before accessing the array and not afterwards.
Signed-off-by: Heinrich Schuchardt
---
v3
Remove superfluous braces.
v2
Only the 4 low bits of epnum are relevant for indexing.
---
drivers/usb/m
musb->endpoints[] has array size MUSB_C_NUM_EPS.
We must check array bounds before accessing the array and not afterwards.
Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
v2
Only the 4 low bits of epnum are relevant for indexing.
---
drivers/usb/musb/musb_gadget_
musb->endpoints[] has array size MUSB_C_NUM_EPS.
We must check array bounds before accessing the array and not afterwards.
Signed-off-by: Heinrich Schuchardt
---
v2
Only the 4 low bits of epnum are relevant for indexing.
---
drivers/usb/musb/musb_gadget_ep0.c | 9 +++--
1 f
musb->endpoints[] has array size MUSB_C_NUM_EPS.
We must check array bounds before accessing the array and not afterwards.
Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
drivers/usb/musb/musb_gadget_ep0.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
d
musb->endpoints[] has array size MUSB_C_NUM_EPS.
We must check array bounds before accessing the array and not afterwards.
Signed-off-by: Heinrich Schuchardt
---
drivers/usb/musb/musb_gadget_ep0.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/m
In
dwc3_request*r = NULL;
r = A;
the first assignment has no effect. Remove it.
Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
drivers/usb/dwc3/ep0.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/dwc3/ep0.c b/drivers/us
1 - 100 of 647 matches
Mail list logo