Re: [PATCH v1 4/4] ARM: boards: add support for Samsung Galaxy S20 5G (x1s)

[Ivaylo Ivanov <ivo.ivanov.ivanov1@gmail.com>]

On 7/30/25 11:48, Ahmad Fatoum wrote:
> Hi Ivaylo,
>
> On 7/29/25 22:36, Ivaylo Ivanov wrote:
>> Add support for Samsung Galaxy S20 5G, based on exynos990, to the
>> current samsung board support. This platform, just like exynos8895,
>> needs a bit to be set in order to allow the framebuffer to refresh.
>>
> Oh, a shame that I sold the old S20 FE I had. :/
> I still have a rooted S10e though. I should give this a try when I have
> time.

Heh, I was thinking about getting an s10e to bring up mainline linux on it,
but I do have quite a lot of devices to work on right now. At the very least,
I expect to throw in support for 7580 and 8890 in the future for both
mainline linux and barebox. The plan ideally will be to bring up support for
exynos7580 in barebox to a decent state, with mmc and usb working at least.
S20 series are decently supported, but nothing exciting yet.

Another qustion: are there any plans to support booting fit images from an
address in ram instead of a file? I suspect porting over all the ufs stuff from
linux will be a hassle, so for S20 (which does not have an sd card slot, only ufs)
booting a fit image that s-boot has loaded into ram for us might be neat.
As far as I've seen, only "go" can do that, but not "bootm".

Best regards,
Ivaylo

>
>> diff --git a/arch/arm/dts/exynos990-x1s.dts b/arch/arm/dts/exynos990-x1s.dts
>> new file mode 100644
>> index 00000000..19d59eaa
>> --- /dev/null
>> +++ b/arch/arm/dts/exynos990-x1s.dts
>> @@ -0,0 +1,13 @@
>> +// SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause
>> +/*
>> + * Samsung Galaxy S20 5G (x1s/SM-G981B) barebox device tree source
>> + *
>> + * Copyright (c) 2025, Ivaylo Ivanov <ivo.ivanov.ivanov1@gmail.com>
>> + */
>> +
>> +/dts-v1/;
>> +#include <arm64/exynos/exynos990-x1s.dts>
>> +
>> +/ {
>> +	barebox,disable-deep-probe;
> Same comment: we want deep probe as default eventually. Did you run into
> problems without this?
>
> Cheers,
> Ahmad
>
>> +};