Re: [PATCH v3 04/38] ppc: qemu: Support non-identity PCI bus address

2021-02-28 Thread Bin Meng
Hi Priyanka, On Mon, Mar 1, 2021 at 3:15 PM Priyanka Jain (OSS) wrote: > > > > >-Original Message- > >From: Bin Meng > >Sent: Friday, February 26, 2021 7:46 PM > >To: Priyanka Jain (OSS) > >Cc: Simon Glass ; Alexander Graf ; U- > >Boot Mailing List ; Tom Rini > >Subject: Re: [PATCH v3

RE: [PATCH v3 04/38] ppc: qemu: Support non-identity PCI bus address

2021-02-28 Thread Priyanka Jain (OSS)
>-Original Message- >From: Bin Meng >Sent: Friday, February 26, 2021 7:46 PM >To: Priyanka Jain (OSS) >Cc: Simon Glass ; Alexander Graf ; U- >Boot Mailing List ; Tom Rini >Subject: Re: [PATCH v3 04/38] ppc: qemu: Support non-identity PCI bus address > >Hi Priyanka, > >On Fri, Feb 26, 2

RE: [v4 2/7] arm: socfpga: soc64: Support Vendor Authorized Boot (VAB)

2021-02-28 Thread Tan, Ley Foon
> -Original Message- > From: Lim, Elly Siew Chin > Sent: Monday, March 1, 2021 2:43 PM > To: Tan, Ley Foon ; u-boot@lists.denx.de > Cc: Marek Vasut ; See, Chin Liang > ; Simon Goldschmidt > ; Chee, Tien Fong > ; Westergreen, Dalon > ; Simon Glass ; Gan, > Yau Wai > Subject: RE: [v4 2/7

RE: [v4 2/7] arm: socfpga: soc64: Support Vendor Authorized Boot (VAB)

2021-02-28 Thread Lim, Elly Siew Chin
Hi Ley Foon, > -Original Message- > From: Tan, Ley Foon > Sent: Monday, March 1, 2021 9:56 AM > To: Lim, Elly Siew Chin ; u-boot@lists.denx.de > Cc: Marek Vasut ; See, Chin Liang > ; Simon Goldschmidt > ; Chee, Tien Fong > ; Westergreen, Dalon > ; Simon Glass ; Gan, > Yau Wai > Subject:

[PATCH v5 3/3] qemu: arm: select QFW, MMIO on qemu-arm

2021-02-28 Thread Asherah Connor
Select CMD_QFW and QFW_MMIO in the qemu-arm board (covers arm and arm64). Signed-off-by: Asherah Connor --- Changes in v5: * Split adding MMIO driver to QEMU arm/64 into own commit. * Use the generic qemu-arm board config rather than adding to multiple defconfigs. board/emulation/qemu-arm/K

[PATCH v5 2/3] qemu: add MMIO driver for QFW

2021-02-28 Thread Asherah Connor
Add MMIO driver for QFW. Note that there is no consumer as of this patch. Signed-off-by: Asherah Connor --- Changes in v5: * Split MMIO driver into its own commit. * Add CONFIG_QFW_MMIO for selection by arch/board. drivers/misc/Kconfig| 7 +++ drivers/misc/Makefile | 1 + drivers/mi

[PATCH v5 1/3] x86: qemu: move QFW to its own uclass

2021-02-28 Thread Asherah Connor
We move qfw into its own uclass and split the PIO functions into a specific driver for that uclass. The PIO driver is selected in the qemu-x86 board config (this covers x86 and x86_64). A sandbox driver and test are added for the uclass, and a test in QEMU added for qfw functionality to confirm i

[PATCH v5 0/3] Move qfw to DM, add Arm support

2021-02-28 Thread Asherah Connor
This series moves the QFW driver into a uclass, UCLASS_QFW, and splits the driver into qfw_pio and qfw_mmio. Each driver is selected on the appropriate QEMU board. A sandbox driver is also added, and a DM unit test against that driver. The qfw command is tested in QEMU, and documentation added.

RE: [RESEND v4 7/7] Makefile: socfpga: Add target to generate hex output for combined spl and dtb

2021-02-28 Thread Tan, Ley Foon
> -Original Message- > From: Lim, Elly Siew Chin > Sent: Saturday, February 27, 2021 12:58 AM > To: u-boot@lists.denx.de > Cc: Marek Vasut ; Tan, Ley Foon > ; See, Chin Liang ; > Simon Goldschmidt ; Chee, Tien Fong > ; Westergreen, Dalon > ; Simon Glass ; Gan, > Yau Wai ; Lim, Elly Siew

RE: [v4 2/7] arm: socfpga: soc64: Support Vendor Authorized Boot (VAB)

2021-02-28 Thread Tan, Ley Foon
> -Original Message- > From: Lim, Elly Siew Chin > Sent: Saturday, February 27, 2021 12:11 AM > To: u-boot@lists.denx.de > Cc: Marek Vasut ; Tan, Ley Foon > ; See, Chin Liang ; > Simon Goldschmidt ; Chee, Tien Fong > ; Westergreen, Dalon > ; Simon Glass ; Gan, > Yau Wai ; Lim, Elly Siew

Re: [PATCH 2/2 v4] efi: ESRT creation tests

2021-02-28 Thread AKASHI Takahiro
Heinrich, On Fri, Feb 26, 2021 at 03:17:42PM +0100, Heinrich Schuchardt wrote: > On 26.02.21 14:55, Heinrich Schuchardt wrote: > > On 23.02.21 17:50, Jose Marinho wrote: > >> This commmit exercises the ESRT creation -- introduced in the previous > >> commit -- in two tests. > >> > >> test 1: > >>

Re: [PATCH 5/5] cmd: Convert part uuid to use cmd_result

2021-02-28 Thread Sean Anderson
On 2/28/21 7:07 PM, Heinrich Schuchardt wrote: Am 1. März 2021 01:03:43 MEZ schrieb Heinrich Schuchardt : Am 1. März 2021 00:47:18 MEZ schrieb Sean Anderson : This is fairly straightforward. This allows part uuid mmc 0 foo To be rewritten as env set foo $(part uuid mmc 0) or eve

Re: [PATCH 5/5] cmd: Convert part uuid to use cmd_result

2021-02-28 Thread Heinrich Schuchardt
Am 1. März 2021 01:03:43 MEZ schrieb Heinrich Schuchardt : >Am 1. März 2021 00:47:18 MEZ schrieb Sean Anderson : >>This is fairly straightforward. This allows >> part uuid mmc 0 foo >>To be rewritten as >> env set foo $(part uuid mmc 0) >>or even (if the variable is not required to be env

Re: [PATCH 2/5] cmd: Add support for (limited) command substitution

2021-02-28 Thread Sean Anderson
On 2/28/21 6:59 PM, Heinrich Schuchardt wrote: Am 1. März 2021 00:47:15 MEZ schrieb Sean Anderson : The traditional way to grab the output from a shell script is to use command substitution. Unfortunately, we don't really have the concept of pipes in U-Boot (at least not without console recordin

Re: [PATCH 5/5] cmd: Convert part uuid to use cmd_result

2021-02-28 Thread Heinrich Schuchardt
Am 1. März 2021 00:47:18 MEZ schrieb Sean Anderson : >This is fairly straightforward. This allows > part uuid mmc 0 foo >To be rewritten as > env set foo $(part uuid mmc 0) >or even (if the variable is not required to be environmental) > foo=$(part uuid mmc 0) Who needs this? Why

Re: [PATCH 3/5] cmd: Convert echo to use cmd_result

2021-02-28 Thread Heinrich Schuchardt
Am 1. März 2021 00:47:16 MEZ schrieb Sean Anderson : >This adds some complexity, since we need to know how big the arguments >are >ahead of time instead of finding out when we print them. Why do you want to add complexity? Where is the documentation update? Best regards Heinrich > >Signed-off

Re: [PATCH 2/5] cmd: Add support for (limited) command substitution

2021-02-28 Thread Heinrich Schuchardt
Am 1. März 2021 00:47:15 MEZ schrieb Sean Anderson : >The traditional way to grab the output from a shell script is to use >command substitution. Unfortunately, we don't really have the concept >of >pipes in U-Boot (at least not without console recording). Even if we >did, >stdout is severely pollu

Re: [PATCH 1/5] hush: Print syntax error line with DEBUG_SHELL

2021-02-28 Thread Sean Anderson
On 2/28/21 6:51 PM, Heinrich Schuchardt wrote: Am 1. März 2021 00:47:14 MEZ schrieb Sean Anderson : This prints the filename (rather useless) and line (very useful) whenever a syntax error occurs if DEBUG_SHELL is enabled. Please, use log_error() instead. The rest of this file uses DEBUG_SHE

Re: [PATCH] hush: Fix assignments being misinterpreted as commands

2021-02-28 Thread Sean Anderson
On 2/28/21 6:40 PM, Heinrich Schuchardt wrote: Am 28. Februar 2021 22:29:51 MEZ schrieb Sean Anderson : If there were no variable substitutions in a command, then initial assignments would be misinterpreted as commands, instead of being skipped over. This is demonstrated by the following example

Re: [PATCH 1/5] hush: Print syntax error line with DEBUG_SHELL

2021-02-28 Thread Heinrich Schuchardt
Am 1. März 2021 00:47:14 MEZ schrieb Sean Anderson : >This prints the filename (rather useless) and line (very useful) >whenever a >syntax error occurs if DEBUG_SHELL is enabled. Please, use log_error() instead. Best regards Heinrich > >Signed-off-by: Sean Anderson >--- > > common/cli_hush.c |

[PATCH 2/5] cmd: Add support for (limited) command substitution

2021-02-28 Thread Sean Anderson
The traditional way to grab the output from a shell script is to use command substitution. Unfortunately, we don't really have the concept of pipes in U-Boot (at least not without console recording). Even if we did, stdout is severely polluted by informational printfs. Instead of redirecting stdou

[PATCH 4/5] test: hush: Add test for command substitution

2021-02-28 Thread Sean Anderson
This adds a basic test for command substitution. A more advanced test currently fails, and is left commented-out for now. A basic example of the difference is that in bash $ a=b; echo a=$(echo $a) b but in hush => a=b; echo a=$(echo $a) a= This is because command

[PATCH 3/5] cmd: Convert echo to use cmd_result

2021-02-28 Thread Sean Anderson
This adds some complexity, since we need to know how big the arguments are ahead of time instead of finding out when we print them. Signed-off-by: Sean Anderson --- cmd/echo.c | 33 ++--- 1 file changed, 26 insertions(+), 7 deletions(-) diff --git a/cmd/echo.c b/cmd

[PATCH 5/5] cmd: Convert part uuid to use cmd_result

2021-02-28 Thread Sean Anderson
This is fairly straightforward. This allows part uuid mmc 0 foo To be rewritten as env set foo $(part uuid mmc 0) or even (if the variable is not required to be environmental) foo=$(part uuid mmc 0) Signed-off-by: Sean Anderson --- cmd/part.c | 18 ++ 1 f

[PATCH 1/5] hush: Print syntax error line with DEBUG_SHELL

2021-02-28 Thread Sean Anderson
This prints the filename (rather useless) and line (very useful) whenever a syntax error occurs if DEBUG_SHELL is enabled. Signed-off-by: Sean Anderson --- common/cli_hush.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/common/cli_hush.c b/common/cli_hush.c

[PATCH 0/5] cmd: Add support for command substitution

2021-02-28 Thread Sean Anderson
This series adds support for command substitution using $(). The motivation for this is that we have many commands which have a version which prints some information, and one which sets an environmental variable with that info. This is a bit clunky, since every time someone wants to grab info from

Re: [PATCH v2 1/3] cmd: mvebu: Implement the Marvell hw_info command

2021-02-28 Thread Heinrich Schuchardt
Am 27. Februar 2021 14:05:50 MEZ schrieb Marek Behun : >Luka, Tom, > >I am very against this. > >there already is `mac` command in cmd/mac.c with following help > display and program the system ID and MAC addresses in EEPROM > [read|save|id|num|errata|date|ports|port_number] > mac read >

Re: [PATCH] hush: Fix assignments being misinterpreted as commands

2021-02-28 Thread Heinrich Schuchardt
Am 28. Februar 2021 22:29:51 MEZ schrieb Sean Anderson : >If there were no variable substitutions in a command, then initial >assignments would be misinterpreted as commands, instead of being >skipped >over. This is demonstrated by the following example: > > => foo=bar echo baz The commit me

Re: [PATCH v2 1/3] cmd: mvebu: Implement the Marvell hw_info command

2021-02-28 Thread Luka Kovacic
Hello Marek, On Sat, Feb 27, 2021 at 2:05 PM Marek Behun wrote: > > Luka, Tom, > > I am very against this. > > there already is `mac` command in cmd/mac.c with following help >display and program the system ID and MAC addresses in EEPROM >[read|save|id|num|errata|date|ports|port_number] >

Re: [PATCH v2 1/3] cmd: mvebu: Implement the Marvell hw_info command

2021-02-28 Thread Luka Kovacic
Hello Pali, On Sat, Feb 27, 2021 at 2:09 AM Pali Rohár wrote: > > On Monday 15 February 2021 20:59:32 Luka Kovacic wrote: > > The hw_info command is implemented to enable parsing Marvell hw_info > > formatted environments. This format is often used on Marvell Armada A37XX > > based devices to sto

Re: [PATCH 1/1] Correct U-Boot upstream repository

2021-02-28 Thread Tom Rini
On Wed, Feb 24, 2021 at 01:19:04PM +0100, Heinrich Schuchardt wrote: > The U-Boot source moves to https://source.denx.de/u-boot/u-boot.git > effective 2021-02-28. > > Signed-off-by: Heinrich Schuchardt Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCHv2] uboot-test-hooks: Switch to our GitLab instance

2021-02-28 Thread Tom Rini
On Wed, Feb 24, 2021 at 05:05:04PM -0500, Tom Rini wrote: > As Stephen is no longer actively maintaining the uboot-test-hooks > repository, switch to using the instance on our GitLab. > > Acked-by: Stephen Warren > Signed-off-by: Tom Rini Applied to u-boot/master, thanks! -- Tom signature.

Re: [PATCH v2 3/3] arm: mvebu: Initial ESPRESSOBin-Ultra board support

2021-02-28 Thread Luka Kovacic
Hello Marek, On Sat, Feb 27, 2021 at 2:24 PM Marek Behun wrote: > > In Linux the DTS for espressobin-ultra includes > armada-3720-espressobin.dtsi and just adds changes. > > If possible, please do this as is done in Linux. In fact we want to > slowly move in the direction to have the dts files ju

[PATCH] hush: Fix assignments being misinterpreted as commands

2021-02-28 Thread Sean Anderson
If there were no variable substitutions in a command, then initial assignments would be misinterpreted as commands, instead of being skipped over. This is demonstrated by the following example: => foo=bar echo baz Unknown command 'foo=bar' - try 'help' Signed-off-by: Sean Anderson

Re: [PATCH v2 2/3] arm: mvebu: mvebu_armada-37xx: Define the loadaddr environment variable

2021-02-28 Thread Luka Kovacic
Hello Pali, On Sat, Feb 27, 2021 at 1:38 AM Pali Rohár wrote: > > On Monday 15 February 2021 20:59:33 Luka Kovacic wrote: > > Add the loadaddr U-Boot environment variable, as this is available in > > the stock Marvell U-Boot by default on Marvell Armada A37XX platforms. > > Hello Luka! Why is thi

[PATCH 1/1] video: buffer overrun in TrueType console

2021-02-28 Thread Heinrich Schuchardt
When scrolling the TrueType console a buffer overrun occurs. Fixes: a29b012037cc ("video: Add a console driver that uses TrueType fonts") Signed-off-by: Heinrich Schuchardt --- drivers/video/console_truetype.c | 26 -- 1 file changed, 12 insertions(+), 14 deletions(-) di

[PATCH 0/1] rk3399 pinebook-pro enable display

2021-02-28 Thread Jesper Schmitz Mouridsen
This enables display for pinebook-pro Jesper Schmitz Mouridsen (1): rk3399: pinebook-pro enable display arch/arm/dts/rk3399-pinebook-pro-u-boot.dtsi | 4 arch/arm/dts/rk3399.dtsi | 4 ++-- drivers/video/rockchip/rk_edp.c | 24 ++-- 3 fil

[PATCH 1/1] rk3399: pinebook-pro enable display

2021-02-28 Thread Jesper Schmitz Mouridsen
Signed-off-by: Jesper Schmitz Mouridsen --- arch/arm/dts/rk3399-pinebook-pro-u-boot.dtsi | 4 arch/arm/dts/rk3399.dtsi | 4 ++-- drivers/video/rockchip/rk_edp.c | 24 ++-- 3 files changed, 28 insertions(+), 4 deletions(-) diff --git a/arch/

[PATCH 1/1] video: fix positioning in TrueType console

2021-02-28 Thread Heinrich Schuchardt
With the patch accurate positioning is possible for mono-typed fonts: Fix the return value of console_truetype_putc_xy(). The current position is passed as parameter x. Some part of x represents a fractional pixel. The return value represents how much the character position must be advanced. This

Re: Pull request for efi-2021-04-rc3-2

2021-02-28 Thread Tom Rini
On Sat, Feb 27, 2021 at 05:13:17AM +0100, Heinrich Schuchardt wrote: > Dear Tom, > > The following changes since commit c28d5d704d3347fcbe5e49ab561973c00bf9337f: > > Merge branch '2021-02-25-remove-platforms' (2021-02-25 13:18:27 -0500) > > are available in the Git repository at: > > https

[PATCH 1/1] efi_loader: console size of vidconsole

2021-02-28 Thread Heinrich Schuchardt
If stdout is 'vidconsole' we correctly set the console size. If stdout is 'vidconsole,serial' the video console is ignored. We should always evaluate the size of vidconsole if it is the primary console. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_console.c | 2 +- 1 file changed,

[PATCH 9/9] board: stm32: Add Engicam MicroGEA STM32MP1 MicroDev 2.0 7" OF

2021-02-28 Thread Jagan Teki
7" OF is a capacitive touch 7" Open Frame panel solutions with - 7" AUO B101AW03 LVDS panel - EDT, FT5526 Touch MicroGEA STM32MP1 is a STM32MP157A based Micro SoM. MicroDev 2.0 is a general purpose miniature carrier board with CAN, LTE and LVDS panel interfaces. MicroGEA STM32MP1 needs to mount

[PATCH 7/9] ARM: dts: stm32: Add Engicam MicroGEA STM32MP1 1X4Gb DDR3

2021-02-28 Thread Jagan Teki
Engicam MicroGEA STM32MP1 Micro SOM has mounted 1x4Gb DDR3 which has 16bits width 533Mhz frequency. Add DDR configurations via dtsi. Signed-off-by: Jagan Teki --- ...m32mp15-ddr3-microgea-1x4Gb-1066-binG.dtsi | 121 ++ 1 file changed, 121 insertions(+) create mode 100644 arch/a

[PATCH 8/9] board: stm32: Add Engicam MicroGEA STM32MP1 MicroDev 2.0 board

2021-02-28 Thread Jagan Teki
MicroDev 2.0 is a general purpose miniature carrier board with CAN, LTE and LVDS panel interfaces. Genaral features: - Ethernet 10/100 - USB Type A - Audio Out - microSD - LVDS panel connector - Wifi/BT (option) - UMTS LTE with sim connector (option) MicroGEA STM32MP1 is a STM32MP157A based Micro

[PATCH 5/9] board: stm32: Add Engicam i.Core STM32MP1 C.TOUCH 2.0

2021-02-28 Thread Jagan Teki
Engicam C.TOUCH 2.0 is an EDIMM compliant general purpose Carrier board. Genaral features: - Ethernet 10/100 - Wifi/BT - USB Type A/OTG - Audio Out - CAN - LVDS panel connector i.Core STM32MP1 is an EDIMM SoM based on STM32MP157A from Engicam. i.Core STM32MP1 needs to mount on top of this Carrie

[PATCH 4/9] board: stm32: Add Engicam i.Core STM32MP1 EDIMM2.2 Starter Kit

2021-02-28 Thread Jagan Teki
Engicam EDIMM2.2 Starter Kit is an EDIMM 2.2 Form Factor Capacitive Evaluation Board. Genaral features: - LCD 7" C.Touch - microSD slot - Ethernet 1Gb - Wifi/BT - 2x LVDS Full HD interfaces - 3x USB 2.0 - 1x USB 3.0 - HDMI Out - Mini PCIe - MIPI CSI - 2x CAN - Audio Out i.Core STM32MP1 is an EDIM

[PATCH 6/9] ARM: dts: stm32: Add Engicam MicroGEA STM32MP1 Micro SoM

2021-02-28 Thread Jagan Teki
MicroGEA STM32MP1 is a STM32MP157A based Micro SoM. General features: - STM32MP157AAC - Up to 1GB DDR3L-800 - 512MB Nand flash - I2S MicroGEA STM32MP1 needs to mount on top of Engicam MicroDev carrier boards for creating complete platform solutions. Add support for it. Signed-off-by: Jagan Teki

[PATCH 3/9] ARM: stm32: Imply SPL_SPI_LOAD

2021-02-28 Thread Jagan Teki
SPI Load isn't mandatory for STM32 builds. Let's imply instead of select it to get rid of build issues for non-SPI defconfigs. Signed-off-by: Jagan Teki --- arch/arm/mach-stm32mp/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/mach-stm32mp/Kconfig b/arch/arm

[PATCH 2/9] ARM: dts: stm32: Add Engicam i.Core STM32MP1 1X4Gb DDR3

2021-02-28 Thread Jagan Teki
Engicam i.Core STM32MP1 SODIMM SoM has mounted 1x4Gb DDR3 which has 32bits width 528000Khz frequency. Add DDR configuration via dtsi. Signed-off-by: Jagan Teki --- .../stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi | 119 ++ 1 file changed, 119 insertions(+) create mode 100644 arch/

[PATCH 1/9] ARM: dts: stm32: Add Engicam i.Core STM32MP1 SoM

2021-02-28 Thread Jagan Teki
i.Core STM32MP1 is an EDIMM SoM based on STM32MP157A from Engicam. General features: - STM32MP157A - Up to 1GB DDR3L - 4GB eMMC - 10/100 Ethernet - USB 2.0 Host/OTG - I2S - MIPI DSI to LVDS - rest of STM32MP157A features i.Core STM32MP1 needs to mount on top of Engicam baseboards for creating com

[PATCH 0/9] stm32: Add Engicam STM32MP1 SoM

2021-02-28 Thread Jagan Teki
Patch series for Engicam i.Core and MicroGEA SoM and it's associated carrier board dts(i) support. patch 1: for i.Core STM32MP1 SoM dtsi patch 2: i.Core DDR3 config patch 3: for SPL_SPI_LOAD build fix patch 4 - 5: i.Core STM32MP1 with EDIMM2.2 and C.Touch 2.0 carriers. patch 6: for MicroGEA ST

Re: [PATCH 2/4] pinephone_defconfig: reduce boot delay

2021-02-28 Thread Andre Przywara
On Thu, 25 Feb 2021 21:58:38 +0100 Maxime Ripard wrote: Hi, > On Thu, Feb 25, 2021 at 05:02:40PM +, André Przywara wrote: > > On 20/02/2021 12:14, Nicolas Boulenguez wrote: > > > From: Arnaud Ferraris > > > > > > On a cellular phone, the vast majority of users can be expected to > > > ha

Re: [IMPORTANT] gitlab relocation / rename

2021-02-28 Thread Wolfgang Denk
Hi all, In message <621240.1614155...@gemini.denx.de> I wrote: > > We have to move all public projects off gitlab.denx.de to a new > gitlab instance; the new host name is then source.denx.de . > We are fully aware that this will cause inconveniences, and I can > only assure you that we try hard

Re: [PATCH 1/1] usb: USB keyboard requires DM_KEYBOARD

2021-02-28 Thread Marek Vasut
On 2/28/21 12:50 PM, Heinrich Schuchardt wrote: If CONFIG_DM_USB=y, the USB keyboard only works if CONFIG_DM_KEYBOARD=y. Applied, thanks

[PATCH v3 12/12] test: pinmux: add test for 'pinctrl-single' driver

2021-02-28 Thread Dario Binacchi
The test adds two pinmux nodes to the device tree, one to test when a register changes only one pin's mux (pinctrl-single,pins), and the other to test when more than one pin's mux is changed (pinctrl-single,bits). This required replacing the controller's register access functions when the driver is

[PATCH v3 11/12] pinctrl: single: add get_pin_muxing operation

2021-02-28 Thread Dario Binacchi
It allows to display the muxing of a given pin. Inspired by more recent versions of the Linux driver, in addition to the address and the value of the configuration register I added the pin function retrieved from the DT. In doing so, the information displayed does not depend on the platform, being

[PATCH v3 08/12] pinctrl: single: add register access functions

2021-02-28 Thread Dario Binacchi
The configuration of pinmux registers was implemented with duplicate code which can be removed by adding two functions for read/write access. Access to 8-bit registers has also been added. Signed-off-by: Dario Binacchi Reviewed-by: Simon Glass --- Changes in v3: - Added Simon Glass review tag.

[PATCH v3 09/12] pinctrl: single: add get_pins_count operation

2021-02-28 Thread Dario Binacchi
It returns the number of selectable pins. Signed-off-by: Dario Binacchi Reviewed-by: Simon Glass --- (no changes since v2) Changes in v2: - Added Simon Glass review tag. drivers/pinctrl/pinctrl-single.c | 37 1 file changed, 37 insertions(+) diff --git a/dr

[PATCH v3 07/12] pinctrl: single: change function mask default value

2021-02-28 Thread Dario Binacchi
The patch is inspired by more recent versions of the Linux driver. Replacing the default value 0x of the function mask with 0 is certainly more conservative in case the "pinctrl-single,function-mask" DT property is missing. Signed-off-by: Dario Binacchi Reviewed-by: Simon Glass --- Cha

[PATCH v3 10/12] pinctrl: single: add get_pin_name operation

2021-02-28 Thread Dario Binacchi
It returns the name of the requested pin. Signed-off-by: Dario Binacchi Reviewed-by: Simon Glass --- (no changes since v2) Changes in v2: - Added Simon Glass review tag. drivers/pinctrl/pinctrl-single.c | 16 1 file changed, 16 insertions(+) diff --git a/drivers/pinctrl/pi

[PATCH v3 05/12] pinctrl: single: get register area size by device API

2021-02-28 Thread Dario Binacchi
Use dev_read_addr_size to get size of the controller's register area. Signed-off-by: Dario Binacchi Reviewed-by: Pratyush Yadav --- Changes in v3: - Added Pratyush Yadav review tag. Changes in v2: - Check dev_read_addr_size return value. drivers/pinctrl/pinctrl-single.c | 14 --

[PATCH v3 04/12] pinctrl: single: fix debug messages formatting

2021-02-28 Thread Dario Binacchi
The dev_dbg(dev, " reg/val 0x%pa/0x%08x\n", ®, val); prints the 'reg' address preceded by the prefix 0x0x instead of 0x. This because the printf '%pa' format specifier already prepends the prefix '0x' to the address displayed. Signed-off-by: Dario Binacchi Reviewed-by: Pratyush Yadav --- Chan

[PATCH v3 03/12] pinctrl: single: fix offset management

2021-02-28 Thread Dario Binacchi
The pinmux configuration DT node of a peripheral does not define a physical address but an offset. Only by adding it to the base address of the controller it is possible to calculate the physical address of the register to be configured. Printing an offset also requires a different formatting optio

[PATCH v3 02/12] pinctrl: single: fix the loop counter variable type

2021-02-28 Thread Dario Binacchi
The 'n' variable is used as a loop counter, not as a physical address, and is used in a comparison with an int. So it makes sense to change its type from phys_addr_t to int. Signed-off-by: Dario Binacchi Reviewed-by: Simon Glass Reviewed-by: Pratyush Yadav --- (no changes since v2) Changes i

[PATCH v3 06/12] pinctrl: single: check "register-width" DT property

2021-02-28 Thread Dario Binacchi
In more recent versions of the Linux kernel the driver's probe function returns an error if the "pinctrl-single,register-width" DT property is missing. The lack of this information, in fact, does not allow to know whether to access the registers of the controller at 8, 16, ... bits. Signed-off-by:

[PATCH v3 01/12] pinctrl: single: fix format of structure documentation

2021-02-28 Thread Dario Binacchi
U-Boot adopted the kernel-doc annotation style. Signed-off-by: Dario Binacchi Reviewed-by: Simon Glass --- (no changes since v2) Changes in v2: - Added Simon Glass review tag. drivers/pinctrl/pinctrl-single.c | 45 +--- 1 file changed, 36 insertions(+), 9 deletio

[PATCH v3 00/12] Add support for pinmux status command on beaglebone

2021-02-28 Thread Dario Binacchi
The series was born from the need to check the pinmux setting of a peripheral on a beaglebone board. I then ran the 'pinmux status -a' command but it failed because some operations (get_pin_muxing, get_pin_name and get_pins_count) were missing in the 'pinctrl-single' driver. The patch series can

Re: [PATCH 1/1] pinephone_defconfig: lower DRAM clock to 528

2021-02-28 Thread André Przywara
On 22/02/2021 22:11, Bob wrote: Hi Bob, > Lower DRAM clock speed to the official speed for the pinephone design: 528 Can you elaborate why this is necessary? Are there reports with the existing data rate causing problems? Please keep in mind that the whole DRAM timings we use do not confirm to

[PATCH 1/1] usb: USB keyboard requires DM_KEYBOARD

2021-02-28 Thread Heinrich Schuchardt
If CONFIG_DM_USB=y, the USB keyboard only works if CONFIG_DM_KEYBOARD=y. Signed-off-by: Heinrich Schuchardt --- drivers/usb/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/Kconfig b/drivers/usb/Kconfig index 6e291198ab..f6975730bf 100644 --- a/drivers/usb/Kconfig +++ b/dr

Re: [PATCH] RFC: spl: fit: Use libfdt functions to read stringlist

2021-02-28 Thread Andre Przywara
On Thu, 25 Feb 2021 14:31:23 -0500 Simon Glass wrote: Hi Simon, > At present the code here reimplements a few libfdt functions and does not > always respect the property length. The !str check is unlikely to fire > since 1 is added to the string address. If strchr() returns NULL then the > code