(le32_to_cpu(buf[63])) {
---
base-commit: 9d47ff66c3892c5a6ddd4704993365a797fbeb68
change-id: 20241018-overflow-dc42def7e4f6
Best regards,
--
Abdelrahman Youssef
ufsize)
> if (bufsize < 256)
> return filetype_unknown;
>
> + if (buf[16] == 0x31305341)
> + return filetype_socfpga_xload;
> +
> if (strncmp(buf8, "STM\x32", 4) == 0) {
> if (buf8[74] == 0x01) {
>
993365a797fbeb68
change-id: 20241018-overflow-dc42def7e4f6
Best regards,
--
Abdelrahman Youssef
sound overly
picky, it's quite important not to introduce random unrelated changes
into commits to make review easier and not needlessly complicate
use of git blame.
Cheers,
Ahmad
> if (buf8[74] == 0x01) {
> switch(le32_to_cpu(buf[63])) {
>
:
---
base-commit: 9d47ff66c3892c5a6ddd4704993365a797fbeb68
change-id: 20241018-overflow-dc42def7e4f6
Best regards,
--
Abdelrahman Youssef
On Thu, 17 Oct 2024 02:47:04 +0300, Abdelrahman Youssef via B4 Relay wrote:
> I was testing barebox with randconfig, so I got a warning (treated as an
> error) : implicit declaration of do_div().
> So needs to included.
>
>
Applied, thanks!
[1/1] include: Include to resolve implicit declar
Hi,
On 18.10.24 13:45, Abdelrahman Youssef via B4 Relay wrote:
> From: Abdelrahman Youssef
>
> in file_detect_type() to detect file of type socfpga_xload you need at least
> 68 bytes bytes, so we need to check if we have enough bufsize.
> So I moved it after checking if `bufsize >= 256`.
>
> Si
Hello Abdelrahman,
On 18.10.24 13:45, Abdelrahman Youssef via B4 Relay wrote:
> From: Abdelrahman Youssef
>
> I was testing barebox with randconfig, so I got a warning (treated as an
> error) : implicit declaration of do_div().
> So needs to included.
>
> Signed-off-by: Abdelrahman Youssef
From: Abdelrahman Youssef
in file_detect_type() to detect file of type socfpga_xload you need at least
68 bytes bytes, so we need to check if we have enough bufsize.
So I moved it after checking if `bufsize >= 256`.
Signed-off-by: Abdelrahman Youssef
---
common/filetype.c | 8
1 file
Signed-off-by: Abdelrahman Youssef
---
Abdelrahman Youssef (2):
include: Include to resolve implicit declaration of
do_div()
common: buffer access out-of-bounds
common/filetype.c | 8
include/linux/ktime.h | 1 +
2 files changed, 5 insertions(+), 4 deletions(-)
---
bas
From: Abdelrahman Youssef
I was testing barebox with randconfig, so I got a warning (treated as an error)
: implicit declaration of do_div().
So needs to included.
Signed-off-by: Abdelrahman Youssef
---
include/linux/ktime.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/linux/k
On Fri, Oct 18, 2024 at 08:12:40AM +0300, Alexander Shiyan wrote:
> Hello Sascha.
>
> Since you're here, I've found another new patch for the driver:
> https://patchwork.kernel.org/project/linux-rockchip/patch/20241017-rk809-dcdc3-v1-1-e3c3de92f...@gmail.com/
Applied this as an extra commit.
Sas
On Wed, Oct 16, 2024 at 11:32:40AM +0200, Yann Sionneau wrote:
> Hi Ahmad,
>
> On 16/10/2024 10:55, Ahmad Fatoum wrote:
> > We have stack dump support in sandbox via AddressSanitizer, if it's
> > compiled in. To make it easier to test proper operation, let's a cpuinfo
> Small typo in the commit ms
On Thu, 17 Oct 2024 11:09:47 +0200, Ahmad Fatoum wrote:
> If the continue statement in the do {} while () loop is
> reached on the first iteration, the loop condition is evaluated without
> having first being set.
>
> Fix this by initializing it to the maximum length of the I2C message.
>
>
>
On Wed, 16 Oct 2024 10:56:07 +0200, Ahmad Fatoum wrote:
> We have a lot of kernel string functions already for use by common,
> driver and board code, but we are lacking strlcat.
>
> Add it to simplify porting code using it.
>
>
Applied, thanks!
[1/2] string: add strlcat support
https:
On Tue, 15 Oct 2024 10:24:45 +0200, Ahmad Fatoum wrote:
> full.config is meant to be a snippet that enables as much as possible to
> increase coverage beyond what's in the defconfig, e.g. to enable
> selftests for a wider range of functionality.
>
> We have no configs enabling CONFIG_JWT, so thi
On Wed, 16 Oct 2024 10:52:18 +0200, Ahmad Fatoum wrote:
> IS_ENABLED() is meant to be used on options with their CONFIG_ prefix,
> fix that and while at it use the recently added more descriptive
> CONFIG_ARM32, CONFIG_ARM64 config symbols.
>
>
Applied, thanks!
[1/1] optee: fix use of IS_ENAB
On Tue, 15 Oct 2024 13:10:57 +0200, Sascha Hauer wrote:
> This series adds support for the RK808 PMIC found on several Rockchip
> boards. Unlike the earlier versions of this PMIC this one comes with
> SPI support. As we do not have a Rockchip SPI driver yet, this is added
> in this series as well
On Mon, 14 Oct 2024 14:58:37 +0200, Ahmad Fatoum wrote:
> Devices can be inherently DMA coherent and thus dma_alloc_coherent could
> be implemented as a normal allocation.
>
> While using uncached memory in this case isn't wrong, it can lead
> to a a very stark performance degradation as identif
Hi All,
This is mostly a cleanup release. One thing worth noting is that Ahmad
has updated the CI container so that barebox is now tested on Debian
Bookworm with GCC 14.2.0 and with the latest Labgrid Release.
Also he ported a bigger font which is looking great on high res
monitors.
Have Fun!
S
20 matches
Mail list logo
