RE: [PATCH v4 1/4] fu540: prci: add request and free clock handlers

2020-06-24 Thread Sagar Kadam
Hi Bin, > -Original Message- > From: Sagar Kadam > Sent: Thursday, June 25, 2020 10:17 AM > To: Bin Meng > Cc: U-Boot Mailing List ; Rick Chen > ; Lukasz Majewski ; Jagan Teki > ; Pragnesh Patel > ; Anup Patel ; Simon > Glass ; Sean Anderson > Subject: RE: [PATCH v4 1/4] fu540: prci:

RE: [PATCH v4 1/4] fu540: prci: add request and free clock handlers

2020-06-24 Thread Sagar Kadam
Hi Bin, > -Original Message- > From: Bin Meng > Sent: Thursday, June 25, 2020 4:51 AM > To: Sagar Kadam > Cc: U-Boot Mailing List ; Rick Chen > ; Lukasz Majewski ; Jagan Teki > ; Pragnesh Patel > ; Anup Patel ; Simon > Glass ; Sean Anderson > Subject: Re: [PATCH v4 1/4] fu540: prci:

[PATCH v4 13/14] dtoc: update dtb_platdata to support cd-gpios

2020-06-24 Thread Walter Lozano
Currently dtoc does not support the property cd-gpios used to declare the gpios for card detect in mmc. This patch adds support to cd-gpios property. Signed-off-by: Walter Lozano --- tools/dtoc/dtb_platdata.py | 16 ++-- tools/dtoc/test_dtoc.py| 2 +- 2 files changed, 11

[PATCH v4 10/14] dtoc: extend dtoc to use struct driver_info when linking nodes

2020-06-24 Thread Walter Lozano
In the current implementation, when dtoc parses a dtb to generate a struct platdata it converts the information related to linked nodes as pointers to struct platdata of destination nodes. By doing this, it makes difficult to get pointer to udevices created based on these information. This patch

[PATCH v4 12/14] arm: dts: include gpio nodes for card detect

2020-06-24 Thread Walter Lozano
Several MMC drivers use GPIO for card detection with cd-gpios property in the MMC node pointing to a GPIO node. However, as U-Boot tries to save space by keeping only required nodes using u-boot* properties, several devices tree result in having only in the MMC node but not the GPIO node

[PATCH v4 11/14] dm: doc: update of-plat with new phandle support

2020-06-24 Thread Walter Lozano
Update documentation to reflect the new phandle support when OF_PLATDATA is used. Now phandles are implemented as pointers to U_BOOT_DEVICE, which makes it possible to get a pointer to the actual device. Signed-off-by: Walter Lozano --- doc/driver-model/of-plat.rst | 24

[PATCH v4 09/14] sandbox: Move section u_boot_list to make it RW

2020-06-24 Thread Walter Lozano
In order to be able to update data in u_boot_list, move this section to make it RW. Signed-off-by: Walter Lozano --- arch/sandbox/cpu/u-boot-spl.lds | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/sandbox/cpu/u-boot-spl.lds b/arch/sandbox/cpu/u-boot-spl.lds index

[PATCH v4 14/14] dtoc: add test for cd-gpios

2020-06-24 Thread Walter Lozano
Add a test for dtoc taking into account the cd-gpios property. Signed-off-by: Walter Lozano --- tools/dtoc/dtoc_test_phandle_cd_gpios.dts | 42 ++ tools/dtoc/test_dtoc.py | 67 +++ 2 files changed, 109 insertions(+) create mode 100644

[PATCH v4 08/14] core: extend struct driver_info to point to device

2020-06-24 Thread Walter Lozano
Currently when creating an U_BOOT_DEVICE entry a struct driver_info is declared, which contains the data needed to instantiate the device. However, the actual device is created at runtime and there is no proper way to get the device based on its struct driver_info. This patch extends struct

[PATCH v4 03/14] core: add support for U_BOOT_DRIVER_ALIAS

2020-06-24 Thread Walter Lozano
Currently when using OF_PLATDATA the binding between devices and drivers is done trying to match the compatible string in the node with a driver name. However, usually a single driver supports multiple compatible strings which causes that only devices which its compatible string matches a driver

[PATCH v4 07/14] core: drop const for struct driver_info

2020-06-24 Thread Walter Lozano
In order to prepare for a new support of phandle when OF_PLATDATA is used drop the const for struct driver_info as this struct will need to be updated on runtime. Signed-off-by: Walter Lozano --- drivers/core/device.c| 2 +- drivers/core/root.c | 2 +-

[PATCH v4 04/14] dtoc: add support to scan drivers

2020-06-24 Thread Walter Lozano
Currently dtoc scans dtbs to convert them to struct platdata and to generate U_BOOT_DEVICE entries. These entries need to be filled with the driver name, but at this moment the information used is the compatible name present in the dtb. This causes that only nodes with a compatible name that

[PATCH v4 05/14] dtoc: add option to disable warnings

2020-06-24 Thread Walter Lozano
As dtoc now performs checks for valid driver names, when running dtoc tests several warnings arise as these tests don't use valid driver names. This patch adds an option to disable those warning, which is only intended for running tests. Signed-off-by: Walter Lozano ---

[PATCH v4 06/14] dm: doc: update of-plat with the support for driver aliases

2020-06-24 Thread Walter Lozano
Update the documentation with the support for driver aliases using U_BOOT_DRIVER_ALIAS. Signed-off-by: Walter Lozano --- doc/driver-model/of-plat.rst | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/doc/driver-model/of-plat.rst b/doc/driver-model/of-plat.rst

[PATCH v4 02/14] dtoc: add missing code comments

2020-06-24 Thread Walter Lozano
Add missing information about internal class members in order to make the code easier to follow. Signed-off-by: Walter Lozano --- tools/dtoc/dtb_platdata.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tools/dtoc/dtb_platdata.py b/tools/dtoc/dtb_platdata.py index

[PATCH v4 01/14] drivers: rename drivers to match compatible string

2020-06-24 Thread Walter Lozano
When using OF_PLATDATA, the bind process between devices and drivers is performed trying to match compatible string with driver names. However driver names are not strictly defined, and also there are different names used when declaring a driver with U_BOOT_DRIVER, the name of the symbol used in

[PATCH v3 14/14] dtoc: add test for cd-gpios

2020-06-24 Thread Walter Lozano
Add a test for dtoc taking into account the cd-gpios property. Signed-off-by: Walter Lozano --- tools/dtoc/dtoc_test_phandle_cd_gpios.dts | 42 ++ tools/dtoc/test_dtoc.py | 67 +++ 2 files changed, 109 insertions(+) create mode 100644

[PATCH v3 13/14] dtoc: update dtb_platdata to support cd-gpios

2020-06-24 Thread Walter Lozano
Currently dtoc does not support the property cd-gpios used to declare the gpios for card detect in mmc. This patch adds support to cd-gpios property. Signed-off-by: Walter Lozano --- tools/dtoc/dtb_platdata.py | 16 ++-- tools/dtoc/test_dtoc.py| 2 +- 2 files changed, 11

[PATCH v3 08/14] core: extend struct driver_info to point to device

2020-06-24 Thread Walter Lozano
Currently when creating an U_BOOT_DEVICE entry a struct driver_info is declared, which contains the data needed to instantiate the device. However, the actual device is created at runtime and there is no proper way to get the device based on its struct driver_info. This patch extends struct

[PATCH v3 12/14] arm: dts: include gpio nodes for card detect

2020-06-24 Thread Walter Lozano
Several MMC drivers use GPIO for card detection with cd-gpios property in the MMC node pointing to a GPIO node. However, as U-Boot tries to save space by keeping only required nodes using u-boot* properties, several devices tree result in having only in the MMC node but not the GPIO node

[PATCH v3 10/14] dtoc: extend dtoc to use struct driver_info when linking nodes

2020-06-24 Thread Walter Lozano
In the current implementation, when dtoc parses a dtb to generate a struct platdata it converts the information related to linked nodes as pointers to struct platdata of destination nodes. By doing this, it makes difficult to get pointer to udevices created based on these information. This patch

[PATCH v3 11/14] dm: doc: update of-plat with new phandle support

2020-06-24 Thread Walter Lozano
Update documentation to reflect the new phandle support when OF_PLATDATA is used. Now phandles are implemented as pointers to U_BOOT_DEVICE, which makes it possible to get a pointer to the actual device. Signed-off-by: Walter Lozano --- doc/driver-model/of-plat.rst | 24

[PATCH v3 09/14] sandbox: Move section u_boot_list to make it RW

2020-06-24 Thread Walter Lozano
In order to be able to update data in u_boot_list, move this section to make it RW. Signed-off-by: Walter Lozano --- arch/sandbox/cpu/u-boot-spl.lds | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/sandbox/cpu/u-boot-spl.lds b/arch/sandbox/cpu/u-boot-spl.lds index

[PATCH v3 06/14] dm: doc: update of-plat with the support for driver aliases

2020-06-24 Thread Walter Lozano
Update the documentation with the support for driver aliases using U_BOOT_DRIVER_ALIAS. Signed-off-by: Walter Lozano --- doc/driver-model/of-plat.rst | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/doc/driver-model/of-plat.rst b/doc/driver-model/of-plat.rst

[PATCH v3 07/14] core: drop const for struct driver_info

2020-06-24 Thread Walter Lozano
In order to prepare for a new support of phandle when OF_PLATDATA is used drop the const for struct driver_info as this struct will need to be updated on runtime. Signed-off-by: Walter Lozano --- drivers/core/device.c| 2 +- drivers/core/root.c | 2 +-

[PATCH v3 04/14] dtoc: add support to scan drivers

2020-06-24 Thread Walter Lozano
Currently dtoc scans dtbs to convert them to struct platdata and to generate U_BOOT_DEVICE entries. These entries need to be filled with the driver name, but at this moment the information used is the compatible name present in the dtb. This causes that only nodes with a compatible name that

[PATCH v3 05/14] dtoc: add option to disable warnings

2020-06-24 Thread Walter Lozano
As dtoc now performs checks for valid driver names, when running dtoc tests several warnings arise as these tests don't use valid driver names. This patch adds an option to disable those warning, which is only intended for running tests. Signed-off-by: Walter Lozano ---

[PATCH v3 01/14] drivers: rename drivers to match compatible string

2020-06-24 Thread Walter Lozano
When using OF_PLATDATA, the bind process between devices and drivers is performed trying to match compatible string with driver names. However driver names are not strictly defined, and also there are different names used when declaring a driver with U_BOOT_DRIVER, the name of the symbol used in

[PATCH v3 03/14] core: add support for U_BOOT_DRIVER_ALIAS

2020-06-24 Thread Walter Lozano
Currently when using OF_PLATDATA the binding between devices and drivers is done trying to match the compatible string in the node with a driver name. However, usually a single driver supports multiple compatible strings which causes that only devices which its compatible string matches a driver

[PATCH v3 02/14] dtoc: add missing code comments

2020-06-24 Thread Walter Lozano
Add missing information about internal class members in order to make the code easier to follow. Signed-off-by: Walter Lozano --- tools/dtoc/dtb_platdata.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tools/dtoc/dtb_platdata.py b/tools/dtoc/dtb_platdata.py index

Re: [PATCH v3 0/4] spi: kirkwood: dm-conversion (part4)

2020-06-24 Thread Chris Packham
On Fri, Jun 19, 2020 at 8:31 PM Chris Packham wrote: > > Hi Jagan, > > On Fri, 19 Jun 2020, 5:45 AM Jagan Teki, wrote: >> >> Series of kirkwood boards to switch DM_SPI and drop >> which are not DM enabled still. >> >> Any inputs? >> Jagan. > > > I had a quick glance over the series and it looks

[PATCH 0/2] arm: mvebu: a38x: USB2 serdes changes

2020-06-24 Thread Chris Packham
While debugging this issue with Marvell I found the change that was made to Marvell's u-boot but doesn't appear to have made it upstream. We're in the process of checking the affect it has on the USB compliance testing we're doing. I have regression tested it against a couple of USB devices

[PATCH 1/2] arm: mvebu: a38x: Fix typo

2020-06-24 Thread Chris Packham
Fix spelling of Alignment. Signed-off-by: Chris Packham --- arch/arm/mach-mvebu/serdes/a38x/high_speed_env_spec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/mach-mvebu/serdes/a38x/high_speed_env_spec.c b/arch/arm/mach-mvebu/serdes/a38x/high_speed_env_spec.c

[PATCH 2/2] arm: mvebu: a38x: Adjust UTMI PHY parameters

2020-06-24 Thread Chris Packham
When running USB compliance tests on our Armada-385 hardware platforms we have seen some eye mask violations. Marvell's internal documentation says: Based on silicon test results, it is recommended to change the impedance calibration threshold setting to 0x6 prior to calibration. Port changes

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Fabio Estevam
On Wed, Jun 24, 2020 at 8:19 PM Fabio Estevam wrote: > - We tried via devicetree with the "qca,clk-out-frequency" property, > but it did not work. This is an area where you could add some debug > prints inside drivers/net/phy/atheros.c and see if if > qca,clk-out-frequency is being parsed

Re: [PATCH v4 1/4] fu540: prci: add request and free clock handlers

2020-06-24 Thread Bin Meng
Hi Sagar, On Wed, Jun 24, 2020 at 6:58 PM Sagar Kadam wrote: > > Hi Bin, > > > -Original Message- > > From: Bin Meng > > Sent: Wednesday, June 24, 2020 6:50 AM > > To: Sagar Kadam > > Cc: U-Boot Mailing List ; Rick Chen > > ; Lukasz Majewski ; Jagan Teki > > ; Pragnesh Patel > > ; Anup

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Fabio Estevam
Hi Soeren, On Wed, Jun 24, 2020 at 7:29 PM Soeren Moch wrote: > Unfortunately I have no access to my usual development board (and will > not have until the v2020.07 release). I only can use an old board > without working USB-OTG, limited to eMMC boot. So I tried to avoid > bisecting, because a

Re: [PATCH v3 1/2] riscv: Do not return error if reserved node already exists

2020-06-24 Thread Bin Meng
On Thu, Jun 25, 2020 at 5:56 AM Atish Patra wrote: > > Not all errors are fatal. If a reserved memory node already exists in the > destination device tree, we can continue to boot without failing. > > Signed-off-by: Atish Patra > --- > arch/riscv/lib/fdt_fixup.c | 2 +- > 1 file changed, 1

Re: [PATCH v3 2/4] arm: Remove d2net_v2 board

2020-06-24 Thread Simon Guinot
On Fri, Jun 19, 2020 at 08:25:59PM -0400, Tom Rini wrote: > On Sat, Jun 20, 2020 at 02:24:47AM +0200, Simon Guinot wrote: > > On Thu, Jun 18, 2020 at 06:25:06PM -0400, Tom Rini wrote: > > > On Fri, Jun 19, 2020 at 12:02:17AM +0200, Simon Guinot wrote: > > > > On Thu, Jun 18, 2020 at 11:15:14PM

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Soeren Moch
Hi Fabio, On 24.06.20 23:35, Fabio Estevam wrote: > Hi Soeren, > > On Wed, Jun 24, 2020 at 5:45 PM Sören Moch wrote: > >> Still no luck: > > mx6cuboxi has an AR8035 too and it is working fine in U-Boot mainline. > > I am not sure why tbs2910 is failing. Could you try to bisect? > Unfortunately

[PATCH 3/4] arm: kirkwood: enable DM_USB for LaCie board

2020-06-24 Thread Simon Guinot
This patch enables DM_USB and USB_STORAGE for the following Kirkwood-based LaCie boards: - d2 Network v2 - Internet Space v2 - 2Big Network v2 - Network Space v2 - Network Space Lite v2 - Network Space Max v2 Signed-off-by: Simon Guinot --- configs/d2net_v2_defconfig | 2 ++

[PATCH 4/4] arm: kirkwood: enable DM_ETH for LaCie board

2020-06-24 Thread Simon Guinot
This patch enables DM_ETH for the following Kirkwood-based LaCie boards: - d2 Network v2 - Internet Space v2 - 2Big Network v2 - Network Space v2 - Network Space Lite v2 - Network Space Max v2 - Network Space Mini v2 Signed-off-by: Simon Guinot --- board/LaCie/net2big_v2/net2big_v2.c | 2 +-

[PATCH 2/4] arm: kirkwood: switch LaCie boards to sata_mv driver

2020-06-24 Thread Simon Guinot
This patch switches the SATA driver from mvsata_ide to sata_mv for the following Kirkwood-based LaCie boards: - d2 Network v2 - Internet Space v2 - 2Big Network v2 - Network Space v2 - Network Space Lite v2 - Network Space Max v2 - Network Space Mini v2 Signed-off-by: Simon Guinot ---

[PATCH 0/4] Convert LaCie boards to use DM drivers

2020-06-24 Thread Simon Guinot
This patch converts the following LaCie boards (Marvell Kirkwood-based) to use DM drivers: - d2 Network v2 - Internet Space v2 - 2Big Network v2 - Network Space v2 - Network Space Lite v2 - Network Space Max v2 - Network Space Mini v2 Simon Guinot (4): arm: kirkwood: convert LaCie boards to

[PATCH 1/4] arm: kirkwood: convert LaCie boards to DM_SPI_FLASH

2020-06-24 Thread Simon Guinot
This patch converts the following Kirkwood-based LaCie boards to DM, DM_SPI and DM_SPI_FLASH: - d2 Network v2 - Internet Space v2 - 2Big Network v2 - Network Space v2 - Network Space Lite v2 - Network Space Max v2 - Network Space Mini v2 Signed-off-by: Simon Guinot ---

[PATCH v3 0/2] Assorted fixes related to reserved memory

2020-06-24 Thread Atish Patra
This series has few small assorted fixes related to reserved memory support in RISC-V. The series is rebased on top of the following series http://patchwork.ozlabs.org/project/uboot/patch/1591767391-2669-2-git-send-email-bmeng...@gmail.com/ Changes form v2->v3: 1. Dropped bootefi patch as it is

[PATCH v3 2/2] riscv: Use optimized version of fdtdec_get_addr_size_no_parent

2020-06-24 Thread Atish Patra
fdtdec_get_addr_size_no_parent is not an optimized version if parent node is already available with the caller. Use fdtdec_get_addr_size_auto_parent to read the "reg" property Signed-off-by: Atish Patra Reviewed-by: Bin Meng --- arch/riscv/lib/fdt_fixup.c | 6 +++--- 1 file changed, 3

[PATCH v3 1/2] riscv: Do not return error if reserved node already exists

2020-06-24 Thread Atish Patra
Not all errors are fatal. If a reserved memory node already exists in the destination device tree, we can continue to boot without failing. Signed-off-by: Atish Patra --- arch/riscv/lib/fdt_fixup.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/riscv/lib/fdt_fixup.c

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Fabio Estevam
Hi Soeren, On Wed, Jun 24, 2020 at 5:45 PM Sören Moch wrote: > Still no luck: mx6cuboxi has an AR8035 too and it is working fine in U-Boot mainline. I am not sure why tbs2910 is failing. Could you try to bisect?

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Sören Moch
Hi Fabio, On 24.06.20 22:29, Fabio Estevam wrote: > Hi Soeren, > > On Wed, Jun 24, 2020 at 5:13 PM Soeren Moch wrote: > >> Schematics are at >> https://www.tbsdtv.com/download/document/tbs2910/TBS2910-Matrix-ARM-mini-PC-SCH_rev2.1.pdf > Here is another patch for you to try: >

Re: [PATCH v2 1/1] travis-ci: Fix "ResourceWarning: unclosed file"

2020-06-24 Thread Cristian Ciocaltea
On Wed, Jun 24, 2020 at 02:30:24PM -0600, Stephen Warren wrote: > On 12/30/19 9:41 AM, Stephen Warren wrote: > > On 12/30/19 4:49 AM, Cristian Ciocaltea wrote: > >> This patch gets rid of the warning messages like: > >> > >> uboot-test-hooks/py/travis-ci/travis_tftp.py:43: ResourceWarning: > >>

Re: [PATCH v2 02/10] test: pinmux: Add test for pin muxing

2020-06-24 Thread Sean Anderson
On 6/24/20 9:45 AM, Simon Glass wrote: > Hi Sean, > > On Wed, 24 Jun 2020 at 02:01, Sean Anderson wrote: >> >> On 6/17/20 10:07 AM, Simon Glass wrote: >>> Hi Sean, >>> >>> On Tue, 16 Jun 2020 at 21:18, Sean Anderson wrote: On 6/16/20 11:11 PM, Simon Glass wrote: > Hi Sean, >

Re: [PATCH v2 1/1] travis-ci: Fix "ResourceWarning: unclosed file"

2020-06-24 Thread Stephen Warren
On 12/30/19 9:41 AM, Stephen Warren wrote: > On 12/30/19 4:49 AM, Cristian Ciocaltea wrote: >> This patch gets rid of the warning messages like: >> >> uboot-test-hooks/py/travis-ci/travis_tftp.py:43: ResourceWarning: >> unclosed file <_io.BufferedReader name='.bm-work/qemu_arm/u-boot.bin'> >> >>

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Fabio Estevam
Hi Soeren, On Wed, Jun 24, 2020 at 5:13 PM Soeren Moch wrote: > Schematics are at > https://www.tbsdtv.com/download/document/tbs2910/TBS2910-Matrix-ARM-mini-PC-SCH_rev2.1.pdf Here is another patch for you to try: https://pastebin.com/raw/Dkipgq1n

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Soeren Moch
Hi Fabio On 24.06.20 21:39, Fabio Estevam wrote: > Hi Soeren, > > On Wed, Jun 24, 2020 at 4:20 PM Soeren Moch wrote: > >> I tested current master (v2020.07-rc5 + imx fixes) on tbs2910 (imx6q + >> atheros PHY). >> Unfortunately ethernet is broken now, while it used to work well on >> u-boot

Re: [PATCH 1/1] gitlab: show skipped Python tests

2020-06-24 Thread Simon Glass
Hi Tom, On Wed, 24 Jun 2020 at 09:53, Tom Rini wrote: > > On Wed, Jun 24, 2020 at 09:17:51AM -0600, Simon Glass wrote: > > Hi Heinrich, > > > > On Wed, 24 Jun 2020 at 07:56, Heinrich Schuchardt > > wrote: > > > > > > On 24.06.20 15:49, Simon Glass wrote: > > > > Hi, > > > > > > > > On Mon, 22

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Fabio Estevam
Hi Soeren, On Wed, Jun 24, 2020 at 4:20 PM Soeren Moch wrote: > I tested current master (v2020.07-rc5 + imx fixes) on tbs2910 (imx6q + > atheros PHY). > Unfortunately ethernet is broken now, while it used to work well on > u-boot version v2020.04. Ethernet support was copied from imx6q-sabresd,

Re: U-Boot atheros PHY support and cubox ethernet

2020-06-24 Thread Soeren Moch
On 20.06.20 04:14, Fabio Estevam wrote: > Hi Tom, > > On Thu, Jun 18, 2020 at 10:39 AM Tom Rini wrote: > >> It's a good question what else doesn't work. What has been actively >> verified at this point? > > After the fixes I sent yesterday, the few imx boards I have access at > the moment have

Re: [PULL] please pull u-boot-mmc mmc-2020-6-24

2020-06-24 Thread Tom Rini
On Wed, Jun 24, 2020 at 12:37:00PM +, Peng Fan wrote: > Hi Tom, > > Please pull mmc-2020-6-24 > > CI: https://travis-ci.org/github/MrVan/u-boot/builds/701528851 Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH] [RFC] tools: fitmount: fuse mount fit images

2020-06-24 Thread Tom Rini
On Wed, Jun 24, 2020 at 08:20:38AM +0530, selvamuthukumar v wrote: > On Tue, Jun 23, 2020 at 11:47 PM Tom Rini wrote: > > > > On Tue, Jun 23, 2020 at 06:01:38PM +0530, selvamuthukumar v wrote: > > > On Mon, Jun 15, 2020 at 11:15 PM Selva Muthukumar > > > wrote: > > > > > > > > Allow mounting of

Re: [PATCH v2 3/9] env: correctly handle result in env_init

2020-06-24 Thread Tom Rini
On Wed, Jun 24, 2020 at 11:19:50AM +, Patrick DELAUNAY wrote: > Hi Tom, > > > From: Tom Rini > > Sent: mardi 23 juin 2020 17:17 > > > > On Tue, Jun 23, 2020 at 01:13:55PM +, Patrick DELAUNAY wrote: > > > Hi Tom, > > > > > > > From: Tom Rini > > > > Sent: vendredi 19 juin 2020 20:05 > >

Re: [PATCH U-BOOT v3 00/30] PLEASE TEST fs: btrfs: Re-implement btrfs support using code from btrfs-progs

2020-06-24 Thread Marek Behún
I shall compile for different platforms with and without these patches to see how much these patches increase the size of the resulting U-Boot binary. Maybe it will show us that it is finally time to look into LTO for U-Boot.

[PATCH U-BOOT v3 28/30] fs: btrfs: Imeplement btrfs_list_subvols() using new infrastructure

2020-06-24 Thread Marek Behún
From: Qu Wenruo Reimplement btrfs_list_subvols() to use new code. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/subvolume.c | 78 +--- 1 file changed, 73 insertions(+), 5 deletions(-) diff --git a/fs/btrfs/subvolume.c

[PATCH U-BOOT v3 29/30] fs: btrfs: Cleanup the old implementation

2020-06-24 Thread Marek Behún
From: Qu Wenruo This cleans up the now unneeded code from the old btrfs implementation. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/Makefile | 4 +- fs/btrfs/btrfs.c | 31 fs/btrfs/btrfs.h | 47 -- fs/btrfs/chunk-map.c | 178

[PATCH U-BOOT v3 18/30] fs: btrfs: inode: Allow next_length() to return value > BTRFS_NAME_LEN

2020-06-24 Thread Marek Behún
From: Qu Wenruo All existing next_length() caller handles return value > BTRFS_NAME_LEN, so there is no need to do BTRFS_NAME_LEN check in next_length(). But still, we want to exit early if we're beyond BTRFS_NAME_LEN, so this patch makes next_length() exit as soon as we're beyond

[PATCH U-BOOT v3 30/30] MAINTAINERS: Add btrfs mailing list and myself as reviewer

2020-06-24 Thread Marek Behún
From: Qu Wenruo Since the current code base is mostly from btrfs-progs, anyone contributing to U-Boot btrfs code could also help us to improve btrfs-progs and btrfs kernel module. Also add myself as designated reviewer. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- MAINTAINERS | 2

[PATCH U-BOOT v3 27/30] fs: btrfs: Introduce function to resolve the path of one subvolume

2020-06-24 Thread Marek Behún
From: Qu Wenruo This patch introduces a new function, list_one_subvol(), which will resolve the path to FS_TREE of one subvolume. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/subvolume.c | 81 1 file changed, 81 insertions(+)

[PATCH U-BOOT v3 26/30] fs: btrfs: Introduce function to resolve path in one subvolume

2020-06-24 Thread Marek Behún
From: Qu Wenruo This patch introduces a new function, get_path_in_subvolume(), which resolves inode number into path inside a subvolume. This function will be later used for btrfs subvolume list functionality. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/compat.h| 1 +

[PATCH U-BOOT v3 15/30] fs: btrfs: Crossport open_ctree_fs_info() from btrfs-progs

2020-06-24 Thread Marek Behún
From: Qu Wenruo open_ctree_fs_info() is the main entry point to open btrfs. This version is a simplfied version of __open_ctree_fd() of btrfs-progs, the main differences are: - Parameters on how to specify a block device Instead of @fd and @path, U-Boot uses blk_desc and disk_partition_t. -

[PATCH U-BOOT v3 14/30] fs: btrfs: Crossport btrfs_read_sys_array() and btrfs_read_chunk_tree()

2020-06-24 Thread Marek Behún
From: Qu Wenruo These two functions play a big role in btrfs bootstrap. The following function is removed: - Seed device support Although in theory we can still support multiple devices, we don't have a facility in U-Boot to do device scan without opening them. Signed-off-by: Qu Wenruo

[PATCH U-BOOT v3 16/30] fs: btrfs: Rename path resolve related functions to avoid name conflicts

2020-06-24 Thread Marek Behún
From: Qu Wenruo Since the old code is using __btrfs_path/__btrfs_root which is different from the regular extent buffer based one, we add "__" prefix for the old implementation to avoid name conflicts for the incoming crossport. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún ---

[PATCH U-BOOT v3 17/30] fs: btrfs: Use btrfs_readlink() to implement __btrfs_readlink()

2020-06-24 Thread Marek Behún
From: Qu Wenruo The existing __btrfs_readlink() can be easily re-implemented using the extent buffer based btrfs_readlink(). This is the first step to re-implement U-Boot's btrfs code. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/btrfs.h | 1 + fs/btrfs/inode.c | 101

[PATCH U-BOOT v3 25/30] fs: btrfs: Implement btrfs_file_read()

2020-06-24 Thread Marek Behún
From: Qu Wenruo This version of btrfs_file_read() has the following new features: - Tries all mirrors - More handling on unaligned size - Better compressed extent handling The old implementation doesn't handle compressed extent with offset properly: we need to read out the whole compressed

[PATCH U-BOOT v3 23/30] fs: btrfs: Introduce btrfs_read_extent_inline() and btrfs_read_extent_reg()

2020-06-24 Thread Marek Behún
From: Qu Wenruo These two functions are used to do sector aligned read, which will be later used to implement btrfs_file_read(). Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/ctree.h | 5 ++ fs/btrfs/disk-io.c | 36 ++ fs/btrfs/disk-io.h | 2 +

[PATCH U-BOOT v3 19/30] fs: btrfs: Implement btrfs_lookup_path()

2020-06-24 Thread Marek Behún
From: Qu Wenruo This is the extent buffer based path lookup routine. To implement this, btrfs_lookup_dir_item() is crossported from btrfs-progs, and implements btrfs_lookup_path() from scratch. Unlike the existing __btrfs_lookup_path(), since btrfs_read_fs_root() will check whether a root is a

[PATCH U-BOOT v3 24/30] fs: btrfs: Introduce lookup_data_extent() for later use

2020-06-24 Thread Marek Behún
From: Qu Wenruo This implements lookup_data_extent() function for the incoming new implementation of btrfs_file_read(). Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/inode.c | 101 +++ 1 file changed, 101 insertions(+) diff --git

[PATCH U-BOOT v3 21/30] fs: btrfs: Use btrfs_lookup_path() to implement btrfs_exists() and btrfs_size()

2020-06-24 Thread Marek Behún
From: Qu Wenruo After this the only remaining function that still utilizes __btrfs_lookup_path() is btrfs_read(). Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/btrfs.c | 65 +++- 1 file changed, 48 insertions(+), 17 deletions(-)

[PATCH U-BOOT v3 20/30] fs: btrfs: Use btrfs_iter_dir() to replace btrfs_readdir()

2020-06-24 Thread Marek Behún
From: Qu Wenruo Use extent buffer based infrastructure to re-implement btrfs_readdir(). Along this rework, some small corner cases fixed: - Subvolume tree mtime Mtime of a subvolume tree is recorded in its root item, since there is no INODE_ITEM for it. This needs extra search from tree

[PATCH U-BOOT v3 22/30] fs: btrfs: Rename btrfs_file_read() and its callees to avoid name conflicts

2020-06-24 Thread Marek Behún
From: Qu Wenruo Rename btrfs_file_read() and its callees to avoid name conflicts with the incoming new code. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/btrfs.c | 2 +- fs/btrfs/btrfs.h | 6 +++--- fs/btrfs/extent-io.c | 4 ++-- fs/btrfs/inode.c | 6 +++--- 4

[PATCH U-BOOT v3 07/30] fs: btrfs: Crossport structure accessor into ctree.h

2020-06-24 Thread Marek Behún
From: Qu Wenruo This brings all structure accessors from btrfs-progs/ctree.h, as in kernel's ctree.h. All these accessors handle the endian convert at runtime, and since all of them are defined as static inline functions, those which aren't used won't take space in resulting binary.

[PATCH U-BOOT v3 13/30] fs: btrfs: Crossport btrfs_search_slot() from btrfs-progs

2020-06-24 Thread Marek Behún
From: Qu Wenruo This patch copies the core function, btrfs_search_slot(), from btrfs-progs. This version has the following functionality removed: - The ability to COW tree block Related code is commented out, and can be enabled in the future. - The readahead functionality This is abused in

[PATCH U-BOOT v3 08/30] fs: btrfs: Crossport volumes.[ch] from btrfs-progs

2020-06-24 Thread Marek Behún
From: Qu Wenruo This patch crossports volumes.[ch] from btrfs-progs, including: - btrfs_map_block() The core mechanism to map btrfs logical address to physical address. This version includes multi-device support, along with RAID56 support. - btrfs_scan_one_device() This is the function to

[PATCH U-BOOT v3 10/30] fs: btrfs: Rename struct btrfs_path to struct __btrfs_path

2020-06-24 Thread Marek Behún
From: Qu Wenruo To avoid name conflicting between the extent buffer based btrfs_path from btrfs-progs, rename struct btrfs_path to struct __btrfs_path. Also rename btrfs_free_path() to __btrfs_free_path() to avoid conflicts. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún ---

[PATCH U-BOOT v3 11/30] fs: btrfs: Rename btrfs_root to __btrfs_root

2020-06-24 Thread Marek Behún
From: Qu Wenruo This is to avoid naming conflicts between extent buffer based btrfs_root. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/btrfs.c | 10 +- fs/btrfs/btrfs.h | 26 +- fs/btrfs/ctree.c | 2 +- fs/btrfs/ctree.h | 6

[PATCH U-BOOT v3 01/30] fs: btrfs: Sync btrfs_btree.h from kernel

2020-06-24 Thread Marek Behún
From: Qu Wenruo This version includes all needed on-disk format from kernel. Only need to modify the include headers for U-Boot, everything else is untouched. Also, since U-Boot btrfs is using a different endian convert timing (at tree block read time), it needs some forced type conversion

[PATCH U-BOOT v3 12/30] fs: btrfs: Crossport struct btrfs_root to ctree.h

2020-06-24 Thread Marek Behún
From: Qu Wenruo Crossport struct btrfs_root to ctree.h from btrfs-progs, with write related members deleted. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/ctree.h | 17 + 1 file changed, 17 insertions(+) diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h index

[PATCH U-BOOT v3 09/30] fs: btrfs: Crossport read_tree_block() from btrfs-progs

2020-06-24 Thread Marek Behún
From: Qu Wenruo This is the one of the basic stone function for btrfs, which: - Resolves the chunk mappings - Reads data from disk - Does various sanity check With read_tree_block(), we can finally crossport needed btrfs btree operations to U-Boot. Signed-off-by: Qu Wenruo Reviewed-by: Marek

[PATCH U-BOOT v3 06/30] fs: btrfs: Crossport extent-io.[ch] from btrfs-progs

2020-06-24 Thread Marek Behún
From: Qu Wenruo This brings the extent_io_tree infrastructure, with which we can finally bring in proper btrfs_fs_info structure to ctree.h. With read/write_extent_buffer() implemented we also backport read/write_eb_member() to ctree.h. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún ---

[PATCH U-BOOT v3 02/30] fs: btrfs: Add more checksum algorithms

2020-06-24 Thread Marek Behún
From: Qu Wenruo This mostly crossports crypto/hash.[ch] from btrfs-progs. The differences are: - No blake2 support No blake2 related library in U-Boot yet. - Use uboot xxhash/sha256 directly No need to implement the code as U-Boot has already provided the interface. This adds the

[PATCH U-BOOT v3 03/30] fs: btrfs: Crossport btrfs_read_dev_super() from btrfs-progs

2020-06-24 Thread Marek Behún
From: Qu Wenruo This patch uses generic code from btrfs-progs to read one super block from block device. To support the btrfs-progs coding style, the following is also crossported: - BTRFS_SETGET_FUNC for btrfs_super_block - btrfs_check_super() function - Move btrfs_read_superblock() to

[PATCH U-BOOT v3 05/30] fs: btrfs: Crossport extent-cache.[ch] from btrfs-progs

2020-06-24 Thread Marek Behún
From: Qu Wenruo This patch implements an infrastructure to insert/search/merge an extent range (with variable length). This provides the basis for later extent buffer cache used in btrfs. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/extent-cache.c | 318

[PATCH U-BOOT v3 04/30] fs: btrfs: Crossport rbtree-utils from btrfs-progs

2020-06-24 Thread Marek Behún
From: Qu Wenruo This is needed for incoming extent-cache infrastructure. Signed-off-by: Qu Wenruo Reviewed-by: Marek Behún --- fs/btrfs/Makefile | 3 +- fs/btrfs/common/rbtree-utils.c | 83 ++ fs/btrfs/common/rbtree-utils.h | 53

[PATCH U-BOOT v3 00/30] PLEASE TEST fs: btrfs: Re-implement btrfs support using code from btrfs-progs

2020-06-24 Thread Marek Behún
Hello, this is a cleaned up version of Qu's patches that reimplements U-Boot's btrfs driver with code from btrfs-progs. I have tested this series, found and corrected one bug (failure when accesing files via symlinks), and it looks it is working now, but I would like more people to do some

Re: [PATCH 1/1] gitlab: show skipped Python tests

2020-06-24 Thread Tom Rini
On Wed, Jun 24, 2020 at 09:17:51AM -0600, Simon Glass wrote: > Hi Heinrich, > > On Wed, 24 Jun 2020 at 07:56, Heinrich Schuchardt wrote: > > > > On 24.06.20 15:49, Simon Glass wrote: > > > Hi, > > > > > > On Mon, 22 Jun 2020 at 12:46, Tom Rini wrote: > > >> > > >> On Mon, Jun 22, 2020 at

Re: PINE64 Rock64 - How to get SPI driver working

2020-06-24 Thread Johannes Krottmayer
Sorry for my late response. Thanks for the suggestions. On 22.05.20 at 16:28, Peter Robinson wrote: > Make sure you enable the driver for the actual flash chip too, some of > the popular ones are: > CONFIG_SPI_FLASH_WINBOND=y > CONFIG_SPI_FLASH_MACRONIX=y > CONFIG_SPI_FLASH_SPANSION=y >

Re: [PATCH 1/1] gitlab: show skipped Python tests

2020-06-24 Thread Heinrich Schuchardt
On 24.06.20 17:17, Simon Glass wrote: > Hi Heinrich, > > On Wed, 24 Jun 2020 at 07:56, Heinrich Schuchardt wrote: >> >> On 24.06.20 15:49, Simon Glass wrote: >>> Hi, >>> >>> On Mon, 22 Jun 2020 at 12:46, Tom Rini wrote: On Mon, Jun 22, 2020 at 12:23:35PM -0600, Simon Glass wrote: >

[PATCH 2/2] efi_loader: size of secure boot variables

2020-06-24 Thread Heinrich Schuchardt
The variables SetupMode, AuditMode, DeployedMode are explicitly defined as UINT8 in the UEFI specification. The type of SecureBoot is UINT8 in EDK2. Use variable name secure_boot instead of sec_boot for the value of the UEFI variable SecureBoot. Avoid abbreviations in function descriptions.

[PATCH 1/2] efi_loader: type of efi_secure_mode

2020-06-24 Thread Heinrich Schuchardt
Variable efi_secure_mode is meant to hold a value of enum efi_secure_mode. So it should not be defined as int but as enum efi_secure_mode. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_variable.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 0/2] efi_loader: fix secure boot variables

2020-06-24 Thread Heinrich Schuchardt
The size and type of some UEFI secure boot related variables is incorrect. This leads to incorrect UEFI variable values. For an internal variable holding an enum int is used as type instead of the enumeration. Heinrich Schuchardt (2): efi_loader: type of efi_secure_mode efi_loader: size of

RE: [PATCH v1 0/4] rework DSI characteristics

2020-06-24 Thread Patrick DELAUNAY
Hi Anatolij > From: Anatolij Gustschin > Sent: mercredi 24 juin 2020 15:32 > > Hi Yannick, > > On Wed, 24 Jun 2020 10:45:38 +0200 > Yannick Fertre yannick.fer...@st.com wrote: > > > Fill characteristics of DSI data link to platform data instead of mipi > > device to avoid memory corruption. >

  1   2   >