Bulk(R):LIBUSB_ERROR_IO(0s)
>
> The "fastboot reboot" command detaches the USB when it still needs to be
> used. So let's detach the USB just before the reset.
>
> CC: Mattijs Korpershoek
> Fixes: 5f7e01e9d5d800 ("usb: gadget: fastboot: detach usb on reboot com
gt;> 3:72>Start Cmd:FB: reboot
>> 3:72>Fail Bulk(R):LIBUSB_ERROR_IO(0s)
>>
>> The "fastboot reboot" command detaches the USB when it still needs to be
>> used. So let's detach the USB just before the reset.
>>
>> CC: Mattijs Korper
On Thu, Jan 05, 2023 at 10:10, Dario Binacchi
wrote:
> Hi Mattijs,
>
> On Thu, Jan 5, 2023 at 10:06 AM Mattijs Korpershoek
> wrote:
>>
>> On Wed, Jan 04, 2023 at 17:46, Dario Binacchi
>> wrote:
>>
>> > The patch fixes the following error when
On Thu, Jan 05, 2023 at 14:25, Dario Binacchi
wrote:
> Hi Mattijs,
>
> On Thu, Jan 5, 2023 at 12:03 PM Mattijs Korpershoek
> wrote:
>>
>> On Thu, Jan 05, 2023 at 10:10, Dario Binacchi
>> wrote:
>>
>> > Hi Mattijs,
>> >
>> >
On Thu, Jan 05, 2023 at 14:25, Dario Binacchi
wrote:
> Hi Mattijs,
>
> On Thu, Jan 5, 2023 at 12:03 PM Mattijs Korpershoek
> wrote:
>>
>> On Thu, Jan 05, 2023 at 10:10, Dario Binacchi
>> wrote:
>>
>> > Hi Mattijs,
>> >
>> >
Hi Dario,
On Sat, Jan 07, 2023 at 17:45, Dario Binacchi
wrote:
> Hi Mattijs,
>
> On Fri, Jan 6, 2023 at 5:42 PM Mattijs Korpershoek
> wrote:
>>
>> On Thu, Jan 05, 2023 at 14:25, Dario Binacchi
>> wrote:
>>
>> > Hi Mattijs,
>> >
&g
rt Cmd:FB: reboot
> 3:72>Fail Bulk(R):LIBUSB_ERROR_IO(0s)
>
> The "fastboot reboot" command detaches the USB when it still needs to be
> used. So let's detach the USB just before the reset.
>
> CC: Mattijs Korpershoek
> Fixes: 5f7e01e9d5d800 ("usb: gadget:
() is called.
Signed-off-by: Mattijs Korpershoek
---
On some boards using the dwc2 controller, like the Khadas VIM3L, whenever
usb_gadget_release() is called, the D+ and D- lines are in an unknown state.
Because of that, the host can't detect usb disconnection.
It was attempted to be be fixed
On Tue, Jan 10, 2023 at 09:33, Mattijs Korpershoek
wrote:
> Hi Dario,
>
> Thank you for your patch.
>
> On Sat, Jan 07, 2023 at 17:48, Dario Binacchi
> wrote:
>
>> The patch fixes the following error when updating a BSH SMM S2 board:
>> 3:72>Start Cmd:FB[
On Wed, Dec 14, 2022 at 15:49, neil.armstr...@linaro.org wrote:
> Hi,
>
> On 14/12/2022 15:45, Mattijs Korpershoek wrote:
>> Hi Neil,
>
>
>
>>>
>>> I'll run some tests on other G12 boards but so far this looks very good,
>>> thanks !
>&
Hi Harald,
Thank you for your review.
On Tue, Jan 10, 2023 at 14:30, Harald Seiler wrote:
> Hi,
>
> On Tue, 2023-01-10 at 10:11 +0100, Mattijs Korpershoek wrote:
>> Pullup is used by the usb framework in order to do software-controlled
>> usb_gadget_connect() and
On Tue, Jan 10, 2023 at 13:56, Alexey Romanov wrote:
> This function can be used by other modules. Also add comments.
>
> Signed-off-by: Alexey Romanov
> Reviewed-by: Neil Armstrong
Reviewed-by: Mattijs Korpershoek
> ---
> arch/arm/include/asm/ar
On Tue, Jan 10, 2023 at 13:56, Alexey Romanov wrote:
> Added docs for Meson Security Monitor command.
>
> Signed-off-by: Alexey Romanov
Reviewed-by: Mattijs Korpershoek
> ---
> doc/usage/cmd/sm.rst | 44
> doc/usage/index.rst
On Tue, Jan 10, 2023 at 13:56, Alexey Romanov wrote:
> Added docs about sm efusedump command, that reads bytes
> from efuse memory bank and print them to the console.
>
> Signed-off-by: Alexey Romanov
> Reviewed-by: Neil Armstrong
Reviewed-by: Mattijs Korpershoek
> --
On Tue, Jan 10, 2023 at 14:45, Marek Vasut wrote:
> On 1/10/23 14:39, Mattijs Korpershoek wrote:
>> Hi Harald,
>>
>> Thank you for your review.
>>
>> On Tue, Jan 10, 2023 at 14:30, Harald Seiler wrote:
>>
>>> Hi,
>>>
>>
especially useful when a gadget disconnection is initiated but
no board_usb_cleanup() is called.
Signed-off-by: Mattijs Korpershoek
---
On some boards using the dwc2 controller, like the Khadas VIM3L, whenever
usb_gadget_release() is called, the D+ and D- lines are in an unknown state.
Because of
On Tue, Jan 10, 2023 at 18:23, Marek Vasut wrote:
> On 1/10/23 17:29, Mattijs Korpershoek wrote:
>
> [...]
>
>> diff --git a/drivers/usb/gadget/dwc2_udc_otg.c
>> b/drivers/usb/gadget/dwc2_udc_otg.c
>> index 77988f78ab30..d0a9be49ad9c 100644
>> --- a/driver
especially useful when a gadget disconnection is initiated but
no board_usb_cleanup() is called.
Signed-off-by: Mattijs Korpershoek
---
On some boards using the dwc2 controller, like the Khadas VIM3L, whenever
usb_gadget_release() is called, the D+ and D- lines are in an unknown state.
Because of
aylibre.com/
>
> Signed-off-by: Neil Armstrong
Thank you for mentioning me, I appreciate it.
Reviewed-by: Mattijs Korpershoek
> ---
> drivers/usb/dwc3/dwc3-meson-gxl.c | 18 ++
> 1 file changed, 18 insertions(+)
>
> diff --git a/drivers/usb/dwc3/dwc3-meson-gxl.
; https://lore.kernel.org/all/20221024-meson-dm-usb-v1-2-2ab077a50...@baylibre.com/
>
> Signed-off-by: Neil Armstrong
Reviewed-by: Mattijs Korpershoek
> ---
> arch/arm/mach-meson/board-axg.c | 129 -
> arch/arm/mach-meson/board-gx.c | 137
AOSP on the Khadas VIM 3 board.
Tested-by: Mattijs Korpershoek # on vim3
Reviewed-by: Mattijs Korpershoek
Some small nit/question below.
> ---
>
> cmd/fastboot.c| 2 +-
> drivers/fastboot/fb_command.c | 13 -
> drivers/fastboot/fb_common.c | 15
; +#define CLK_TOP_VPLL_DPIX_EN 133
> +#define CLK_TOP_SSUSB_TOP_CK_EN 134
> +#define CLK_TOP_SSUSB_PHY_CK_EN 135
> +#define CLK_TOP_CONN_32K 136
> +#define CLK_TOP_CONN_26M 137
> +#define CLK_TOP_DSP_32K
les:
> - clk-mt8365.c (a96cbb146a9736f501fe66ebda6a9018735e5e8a)
> - clk-mt8365-apmixedsys.c (65c9ad77cbc0eed78db94d80041aba675cfbdfa9)
> And adapted following the clk attributes supported by U-Boot.
>
> Signed-off-by: Julien Masson
Reviewed-by: Mattijs Korpershoek
> ---
> driver
Hi Julien,
Thank you for your patch.
On lun., déc. 04, 2023 at 11:48, Julien Masson wrote:
> Add power-domain header for MediaTek MT8365 SoC copied from Linux
> source code tag v6.7-rc2.
> (commit a1571f1f333c2fced076f0d54ed771d1838d827f)
>
> Signed-off-by: Julien Masson
Revie
On Tue, Dec 05, 2023 at 20:54, Simon Glass wrote:
> Hi Mattijs,
>
> On Tue, 5 Dec 2023 at 02:16, Mattijs Korpershoek
> wrote:
>>
>> Hi Simon,
>>
>> Thank you for your patch.
>>
>> On dim., déc. 03, 2023 at 17:31, Simon Glass wrote:
>>
o the bootm command.
>
> Signed-off-by: Simon Glass
> Reviewed-by: Tom Rini
Reviewed-by: Mattijs Korpershoek
> ---
>
> (no changes since v1)
>
> boot/bootm.c | 8 +---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/boot/bootm.c b/boot/bootm
Hi Simon,
Thank you for the patch.
On jeu., déc. 14, 2023 at 09:50, Simon Glass wrote:
> This is an exported function, so move the function comment to the
> bootm.h header file.
>
> Signed-off-by: Simon Glass
Reviewed-by: Mattijs Korpershoek
> ---
>
> Changes in v2:
&
s
Reviewed-by: Mattijs Korpershoek
> ---
>
> Changes in v2:
> - Avoid changing the logic when a command is not set
>
> drivers/fastboot/Kconfig | 1 -
> drivers/fastboot/fb_common.c | 30 ++
> 2 files changed, 14 insertions(+), 17 deletio
y: Tom Rini
Reviewed-by: Mattijs Korpershoek
> ---
>
> (no changes since v1)
>
> doc/android/fastboot.rst | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/doc/android/fastboot.rst b/doc/android/fastboot.rst
> index 1ad8a897c853..933a652538c7 100644
happens when log_reg is enabled via
CONFIG_LOG_ERROR_RETURN.
> Otherwise we may return an invalid/truncated clock rate value.
>
> Signed-off-by: Julien Masson
I'm wondering if there are any other places where this happens,
but this change looks good to me.
Reviewed-by: Mattijs Korp
On ven., déc. 15, 2023 at 13:34, Mattijs Korpershoek
wrote:
> Hi Julien,
>
> Thank you for the patch.
>
> On mar., nov. 21, 2023 at 15:42, Julien Masson wrote:
>
>> When we call clk_get_rate(), we expect to get clock rate value as
>> ulong.
>> In that ca
t;
> Signed-off-by: Julien Masson
Reviewed-by: Mattijs Korpershoek
> ---
> Changes in v2:
> - remove local var ret (fix warning)
> - Link to v1: https://lore.kernel.org/all/871qcj5frz@baylibre.com
> ---
> drivers/clk/clk-uclass.c | 7 +--
> 1 file changed, 1 ins
Commit 19a91f2464a8 ("Create a new boot/ directory") moved the
android_ab.c code under boot/android_ab but did not update
the MAINTAINERS entry.
Update it so that the maintainer will get cc'ed again.
Signed-off-by: Mattijs Korpershoek
---
MAINTAINERS | 2 +-
1 file changed, 1
free(abc);
> + log_err("ANDROID: failed to store boot control block:
> %d\n", ret);
Same here
With that addressed:
Reviewed-by: Mattijs Korpershoek
> + return ret;
> + }
> }
> free(backup_abc);
> #endif
> --
> 2.39.2
phyinit command is run
> only for such SoCs.
>
> Signed-off-by: Manorit Chawdhry
> Signed-off-by: Siddharth Vadapalli
Reviewed-by: Mattijs Korpershoek
> ---
>
> Hello,
>
> This patch is based on commit
> 65eed68772 test/py: Disable error E0611 in two cases for pyli
Hi Sam,
Thank you for your review.
On Thu, Dec 21, 2023 at 08:15, Sam Protsenko wrote:
> Hi Mattijs,
>
> On Wed, Dec 20, 2023 at 10:43 AM Mattijs Korpershoek
> wrote:
>>
>> Commit 19a91f2464a8 ("Create a new boot/ directory") moved the
>> android_ab.
Commit 19a91f2464a8 ("Create a new boot/ directory") moved the
android_ab.c code under boot/android_ab but did not update
the MAINTAINERS entry.
Update it so that the maintainer will get cc'ed again.
Fixes: 19a91f2464a8 ("Create a new boot/ directory")
Signed-of
Hi Jon,
Thank you for the patch.
On ven., mai 31, 2024 at 17:20, Jonathan Humphreys wrote:
> Few cosmetic fixes for clarity and spelling mistakes.
>
> Signed-off-by: Jonathan Humphreys
Reviewed-by: Mattijs Korpershoek
> ---
> doc/board/ti/k3.rst | 10 +-
>
IG_CMD_BCB=n.
Fixes: dfeb4f0d7935 ("cmd: bcb: extend BCB C API to allow read/write the
fields")
Signed-off-by: Mattijs Korpershoek
---
include/bcb.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/include/bcb.h b/include/bcb.h
index 1941d8c28b4f..a56b547595a6
Fix some trivial typos found by browsing the code.
Done with flyspell.
Signed-off-by: Mattijs Korpershoek
---
include/bootmeth.h | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/include/bootmeth.h b/include/bootmeth.h
index 0fc36104ece0..529c4d813d82 100644
--- a
Hi,
On Tue, 28 May 2024 10:35:03 +0200, Neil Armstrong wrote:
> If the ep0 stalls or request are dequeued when gagdet is stopped,
> the request dma may not be mapped yet and dwc3_flush_cache() may be
> called with a NULL pointer.
>
> Check req->request.dma before calling dwc3_flush_cache() and la
his function);
>
> Add the missing dependency.
>
> Signed-off-by: Heinrich Schuchardt
Reviewed-by: Mattijs Korpershoek
> ---
> drivers/dfu/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/dfu/Kconfig b/drivers/dfu/Kconfig
> index 0360d9da142..9712
able to execute any
> arbitrary commands just like in the u-boot's shell.
Can we please add CONFIG_DFU_SCSI in "Configuration Options" section at
the beginning of this document?
See:
https://docs.u-boot.org/en/latest/usage/dfu.html#configuration-options
Note: I requested that h
Hi Quentin,
On mar., juin 04, 2024 at 11:47, Quentin Schulz
wrote:
> Hi Mattijs,
>
> On 6/3/24 11:11 AM, Mattijs Korpershoek wrote:
>> Fix some trivial typos found by browsing the code.
>> Done with flyspell.
>>
>> Signed-off-by: Mattijs Korpershoek &
ixes some trivial typos, second patch replaces all
occurences of bootmethod(s) -> bootmeth(s).
Signed-off-by: Mattijs Korpershoek
---
Changes in v2:
- Made into a series
- New patch replaces all bootmethod(s) -> bootmeth(s)
- Link to v1:
https://lore.kernel.org/r/20240603-bootmeth-typos-v1
Fix some trivial typos found by browsing the code.
Done with flyspell.
Reviewed-by: Quentin Schulz
Signed-off-by: Mattijs Korpershoek
---
include/bootmeth.h | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/include/bootmeth.h b/include/bootmeth.h
index
- bootmeth - because 'method' is too vanilla, appears 1300 times in
U-Boot
"""
Replace all occurences in various comments for consistency.
[1] https://lore.kernel.org/u-boot/20211023232635.9195-1-...@chromium.org/
Signed-off-by: Mattijs Korpershoek
---
Hi Quentin,
Thanks for the review!
On mar., juin 04, 2024 at 14:22, Quentin Schulz
wrote:
> Hi Mattijs,
>
> On 6/4/24 2:04 PM, Mattijs Korpershoek wrote:
[...]
>>
>> There seems indeed to be some inconsistencies around bootmeths versus
>> bootmethods.
>>
Hi Quentin,
Thank you for the review.
On mar., juin 04, 2024 at 17:25, Quentin Schulz
wrote:
> Hi Mattijs,
>
> On 6/4/24 5:15 PM, Mattijs Korpershoek wrote:
>> According to [1], we should use bootmeth when describing the
>> struct bootmeth:
>>
>> ""
y: Quentin Schulz
Reviewed-by: Mattijs Korpershoek
> ---
> drivers/power/regulator/rk8xx.c | 26 +-
> 1 file changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/power/regulator/rk8xx.c b/drivers/power/regulator/rk8xx.c
> index bd5a37e
Hi,
On Tue, 04 Jun 2024 07:44:25 +0200, Heinrich Schuchardt wrote:
> Building the SPI flash DFU driver fails if SPI flash support is missing.
>
> drivers/dfu/dfu_sf.c:123:29: error:
> ‘CONFIG_SF_DEFAULT_MODE’ undeclared (first use in this function);
>
> Add the missing dependency.
>
> [
v6.10-rc1).
>
> Signed-off-by: Nishanth Menon
Reviewed-by: Mattijs Korpershoek
> ---
> arch/arm/dts/Makefile| 1 -
> arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi | 4 +-
> arch/arm/dts/k3-am625-beagleplay.dts | 932 ---
Hi Tom,
Please find some fixes for master:
- dwc3 fix crash when ep0 stalls or gadget is stopped
- Kconfig build fix for DFU_SF (SPI flash DFU driver)
The CI job is at
https://source.denx.de/u-boot/custodians/u-boot-dfu/-/pipelines/20990
Thanks,
Mattijs
The following changes since commit c0ea
Hi Tom,
Please find some fixes for master:
- dwc3 fix crash when ep0 stalls or gadget is stopped
- Kconfig build fix for DFU_SF (SPI flash DFU driver)
The CI job is at
https://source.denx.de/u-boot/custodians/u-boot-dfu/-/pipelines/20990
Thanks,
Mattijs
The following changes since commit c0ea
Hi everyone,
On jeu., mai 16, 2024 at 17:12, Mattijs Korpershoek
wrote:
> Hi Tom,
>
> Please find some fixes for master:
>
> - dwc3 fix crash when ep0 stalls or gadget is stopped
> - Kconfig build fix for DFU_SF (SPI flash DFU driver)
>
> The CI job is at
> ht
nfigs/meson64_android.h
[5] https://lore.kernel.org/r/all/20230914165615.1058529-17-...@chromium.org/
[6]
https://software-dl.ti.com/processor-sdk-android/esd/AM62X/09_02_00/docs/android/Overview.html
Signed-off-by: Mattijs Korpershoek
---
Mattijs Korpershoek (6):
boot: android: Pr
When calling android_image_get_dtb_by_index() using boot image v3+,
we should also pass the vendor_boot ramdisk address.
Use get_avendor_bootimg_addr() to do so.
Note: on boot image v2, this is harmless since get_avendor_bootimg_addr()
returns -1.
Signed-off-by: Mattijs Korpershoek
---
boot
When reading a boot image header, we may need to retrieve the header
version.
Add a helper function for it.
Signed-off-by: Mattijs Korpershoek
---
boot/image-android.c | 7 ++-
include/image.h | 7 +++
2 files changed, 13 insertions(+), 1 deletion(-)
diff --git a/boot/image
Some bootflows might be able to only boot from MMC devices.
Add a helper function these bootflows can use.
Signed-off-by: Mattijs Korpershoek
---
boot/bootflow.c| 12
include/bootflow.h | 9 +
2 files changed, 21 insertions(+)
diff --git a/boot/bootflow.c b/boot
useful for implementing an Android bootmethod.
Signed-off-by: Mattijs Korpershoek
---
cmd/abootimg.c | 10 ++
include/image.h | 14 ++
2 files changed, 24 insertions(+)
diff --git a/cmd/abootimg.c b/cmd/abootimg.c
index 88c77d999290..33381e22dec2 100644
--- a/cmd/abootimg.c
BOOTMETH_ANDROID as a dependency on sandbox so that we can test
this with:
$ ./test/py/test.py --bd sandbox --build -k test_ut # to build the mmc7.img
$ ./test/py/test.py --bd sandbox --build -k bootflow_android
Signed-off-by: Mattijs Korpershoek
---
arch/sandbox/dts/test.dts | 8 +
configs
://lore.kernel.org/r/all/20230914165615.1058529-17-...@chromium.org/
Signed-off-by: Mattijs Korpershoek
---
MAINTAINERS | 7 +
boot/Kconfig| 14 ++
boot/Makefile | 2 +
boot/bootmeth_android.c | 522
boot/bootmeth_android.h
gt;
> Signed-off-by: Nishanth Menon
Reviewed-by: Mattijs Korpershoek
Boot tested to main U-Boot via DFU on AM62X SK EVM.
Tested-by: Mattijs Korpershoek
> ---
> arch/arm/dts/Makefile|3 +-
> arch/arm/dts/k3-am62-main.dtsi | 1058 --
gadget: remove usb_gadget_controller_number()")
>
> Signed-off-by: Marek Vasut
Compared with linux commit, and looks good to me.
Reviewed-by: Mattijs Korpershoek
Tested that I could use fastboot, ums and scan for storage devices on
khadas vim3
Tested-by: Mattijs Korpershoek # vim3
>
gadget: remove usb_gadget_controller_number()")
>
> Signed-off-by: Marek Vasut
Reviewed-by: Mattijs Korpershoek
Tested-by: Mattijs Korpershoek # on vim3
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> Cc: Lukasz Majewski
> Cc: Mattijs Korpershoek
> Cc: Nis
need this quirk anymore.
>
> This patch removes the newly unused function. Linux stopped using this
> functionality in 2012, remove it from U-Boot as well.
>
> Matching Linux kernel commit:
> ed9cbda63d45 ("usb: gadget: remove usb_gadget_controller_number()")
>
>
s.h as well.
>
> Signed-off-by: Marek Vasut
Reviewed-by: Mattijs Korpershoek
Tested-by: Mattijs Korpershoek # on vim3
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> Cc: Lukasz Majewski
> Cc: Mattijs Korpershoek
> Cc: Nishanth Menon
> Cc: Simon Glass
>
y
the linux implementation (usb_ep_autoconfig_ss()).
Reviewed-by: Mattijs Korpershoek
Tested-by: Mattijs Korpershoek # on vim3
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> Cc: Lukasz Majewski
> Cc: Mattijs Korpershoek
> Cc: Nishanth Menon
> Cc: Simon
eeks or so, otherwise I'll apply the changes.
>
> Signed-off-by: Marek Vasut
To me, this looks good.
Reviewed-by: Mattijs Korpershoek
Tested-by: Mattijs Korpershoek # on vim3
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> Cc: Lukasz Majewski
> Cc: Mattijs
Hi Igor,
Thank you for the review.
On lun., juin 10, 2024 at 11:20, Igor Opaniuk wrote:
> Hi Mattijs,
>
> On Thu, Jun 6, 2024 at 2:24 PM Mattijs Korpershoek
> wrote:
>>
>> When reading a boot image header, we may need to retrieve the header
>> version.
>
Hi Igor,
Thank you for the review.
On lun., juin 10, 2024 at 11:31, Igor Opaniuk wrote:
> Hi Mattijs,
>
> On Thu, Jun 6, 2024 at 2:24 PM Mattijs Korpershoek
> wrote:
>>
>> Some bootflows might be able to only boot from MMC devices.
>>
>> Add a hel
Hi Lukasz,
On mar., juin 11, 2024 at 10:51, Lukasz Majewski wrote:
> On Tue, 11 Jun 2024 09:20:33 +0200
> Mattijs Korpershoek wrote:
[...]
>
>> > --
>> > 2.43.0
>
> FInally. :-)
>
> Thanks Mattijs for this cleanup.
You should thank Marek, i
Hi Igor,
Thank you for your quick review.
On lun., juin 10, 2024 at 17:15, Igor Opaniuk wrote:
> Hi Mattijs,
>
> On Thu, Jun 6, 2024 at 2:24 PM Mattijs Korpershoek
> wrote:
>>
>> Android boot flow is a bit different than a regular Linux distro.
>> Android re
Hi Simon,
Thank you for your review.
On mar., juin 11, 2024 at 12:52, Simon Glass wrote:
> Hi Mattijs,
>
> On Thu, 6 Jun 2024 at 06:24, Mattijs Korpershoek
> wrote:
>>
>> Android boot flow is a bit different than a regular Linux distro.
>> Android relies on m
://lore.kernel.org/r/all/20230914165615.1058529-17-...@chromium.org/
Reviewed-by: Simon Glass
Signed-off-by: Mattijs Korpershoek
---
MAINTAINERS | 7 +
boot/Kconfig| 16 ++
boot/Makefile | 2 +
boot/bootmeth_android.c | 553
Glass
Signed-off-by: Mattijs Korpershoek
---
arch/sandbox/dts/test.dts | 8 +
configs/sandbox_defconfig | 2 +-
test/boot/bootflow.c | 65 ++--
test/py/tests/test_ut.py | 76 +++
4 files changed, 147
useful for implementing an Android bootmethod.
Reviewed-by: Igor Opaniuk
Signed-off-by: Mattijs Korpershoek
---
cmd/abootimg.c | 10 ++
include/image.h | 14 ++
2 files changed, 24 insertions(+)
diff --git a/cmd/abootimg.c b/cmd/abootimg.c
index 327712a536c0..ae7a1a7c83b0 10
nfigs/meson64_android.h
[5] https://lore.kernel.org/r/all/20230914165615.1058529-17-...@chromium.org/
[6]
https://software-dl.ti.com/processor-sdk-android/esd/AM62X/09_02_00/docs/android/Overview.html
Signed-off-by: Mattijs Korpershoek
---
Changes in v2:
- Dropped patch 2/6 boot: android
don't have CMD_ABOOTIMG, add a weak
implementation to avoid linking errors.
Signed-off-by: Mattijs Korpershoek
---
boot/image-android.c | 5 +
boot/image-fdt.c | 2 +-
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/boot/image-android.c b/boot/image-android.c
Some bootflows might be able to only boot from MMC devices.
Add a helper function these bootflows can use.
Reviewed-by: Igor Opaniuk
Signed-off-by: Mattijs Korpershoek
---
boot/bootflow.c| 12
include/bootflow.h | 9 +
2 files changed, 21 insertions(+)
diff --git a
Hi Guillaume,
Thank you for testing.
On ven., juin 14, 2024 at 11:53, Guillaume LA ROQUE
wrote:
> Hi,
>
> i apply patch series with commit you give in cover letter and test on
> TI AM62S-SK board.
> Android boot properly , just with a small changes in uboot eenv
>
> setenv vendor_boot_comp_a
Hi Heinrich,
Thank you for your review.
On dim., juin 16, 2024 at 09:38, Heinrich Schuchardt wrote:
> On 6/4/24 17:15, Mattijs Korpershoek wrote:
>> According to [1], we should use bootmeth when describing the
>> struct bootmeth:
>>
>> """
>> For
Hi Simon,
On lun., juin 17, 2024 at 07:53, Simon Glass wrote:
> Hi Mattijs,
>
> On Thu, 13 Jun 2024 at 04:13, Mattijs Korpershoek
> wrote:
>>
>> Android boot flow is a bit different than a regular Linux distro.
>> Android relies on multiple partitions in order
rivers/usb/musb-new/ux500.c| 22 ++
> include/linux/usb/gadget.h | 8
> 14 files changed, 172 insertions(+), 81 deletions(-)
Tested on Khadas vim3 using fastboot, ums and usb storage scanning.
Tested-by: Mattijs Korpershoek # vim3
>
>
: Marek Vasut
Reviewed-by: Mattijs Korpershoek
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> Cc: Lukasz Majewski
> Cc: Mattijs Korpershoek
> Cc: Nishanth Menon
> Cc: Simon Glass
> Cc: Thinh Nguyen
> Cc: Tom Rini
> Cc: u-boot@lists.
Hi Jonas,
On sam., mars 02, 2024 at 14:00, Jonas Karlman wrote:
[...]
>>
>> I will keep you posted.
>
> Thanks, much appreciated!
>
> Please also keep in mind that changing the interrupt handling probably
> only fixes the second of the two build errors reported and fixed by this
> patch.
>
> T
efine its own
> IRQ handling implementation without colliding with other controller
> drivers.
>
> Keep the dm_usb_gadget_handle_interrupts() in this driver for non-DM
> case for now, until this driver gets fully converted to DM USB gadget.
>
> Signed-off-by: Marek Vasut
Re
efine its own
> IRQ handling implementation without colliding with other controller
> drivers.
>
> Signed-off-by: Marek Vasut
Reviewed-by: Mattijs Korpershoek
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> Cc: Lukasz Majewski
> Cc: Mattijs Korpershoek
>
efine its own
> IRQ handling implementation without colliding with other controller
> drivers.
>
> Signed-off-by: Marek Vasut
Reviewed-by: Mattijs Korpershoek
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> Cc: Lukasz Majewski
> Cc: Mattijs Korpershoek
>
efine its own
> IRQ handling implementation without colliding with other controller
> drivers.
>
> Signed-off-by: Marek Vasut
Reviewed-by: Mattijs Korpershoek
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> Cc: Lukasz Majewski
> Cc: Mattijs Korpershoek
>
efine its own
> IRQ handling implementation without colliding with other controller
> drivers.
>
> Signed-off-by: Marek Vasut
Reviewed-by: Mattijs Korpershoek
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> Cc: Lukasz Majewski
> Cc: Mattijs Korpershoek
>
efine its own
> IRQ handling implementation without colliding with other controller
> drivers.
>
> Signed-off-by: Marek Vasut
Reviewed-by: Mattijs Korpershoek
nitpick below (up to you if you want to fix it)
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> C
efine its own
> IRQ handling implementation without colliding with other controller
> drivers.
>
> Signed-off-by: Marek Vasut
Reviewed-by: Mattijs Korpershoek
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> Cc: Lukasz Majewski
> Cc: Mattijs Korpershoek
>
Hi Marek,
Thank you for the patch.
On ven., juin 14, 2024 at 02:51, Marek Vasut
wrote:
> Drop dm_usb_gadget_handle_interrupts() in favor of empty default
> implementation of the same in drivers/usb/gadget/udc/udc-uclass.c .
>
> Signed-off-by: Marek Vasut
Reviewed-by: Mattijs
by: Marek Vasut
Reviewed-by: Mattijs Korpershoek
> ---
> Cc: Alexander Sverdlin
> Cc: Felipe Balbi
> Cc: Lukasz Majewski
> Cc: Mattijs Korpershoek
> Cc: Nishanth Menon
> Cc: Simon Glass
> Cc: Thinh Nguyen
> Cc: Tom Rini
> Cc: u-boot@lists.denx.de
> ---
> d
> Signed-off-by: Heinrich Schuchardt
Reviewed-by: Mattijs Korpershoek
> ---
> v3:
> plural controllers
> v2:
> add 'found' at end of message
> keep printf
> ---
> drivers/usb/host/usb-uclass.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletio
Hi Simon.
On mar., juin 18, 2024 at 21:03, Simon Glass wrote:
> Hi Mattijs,
>
> On Mon, 17 Jun 2024 at 09:15, Mattijs Korpershoek
> wrote:
>>
>> Hi Simon,
>>
>> On lun., juin 17, 2024 at 07:53, Simon Glass wrote:
>>
>> > Hi Mattijs,
>>
Hi Tom,
On jeu., juin 20, 2024 at 08:23, Tom Rini wrote:
> On Thu, Jun 13, 2024 at 12:13:07PM +0200, Mattijs Korpershoek wrote:
>
>> Android boot flow is a bit different than a regular Linux distro.
>> Android relies on multiple partitions in order to boot.
>>
>>
Hi,
On Sun, 09 Jun 2024 23:32:14 +0200, Marek Vasut wrote:
> The bcdDevice field is defined as
> |Device release number in binary-coded decimal
> in the USB 2.0 specification. We use this field to distinguish the UDCs
> from each other. In theory this could be used on the host side to apply
> cert
Hi,
On Fri, 14 Jun 2024 02:51:15 +0200, Marek Vasut wrote:
> Introduce .ops for USB_GADGET_GENERIC uclass. The first new ops is
> .handle_interrupts which must be implemented by DM capable USB gadget
> controller drivers and must implement interrupt handling similar to
> dm_usb_gadget_handle_inter
101 - 200 of 979 matches
Mail list logo
