[PATCH v2] sandbox: Fix pinmux warnings with non-test devicetrees

Simon Glass sjg at chromium.org
Fri Aug 16 03:34:34 CEST 2024


Hi Sean,

On Thu, 15 Aug 2024 at 18:48, Sean Anderson <seanga2 at gmail.com> wrote:
>
> The sandbox pinmux driver is used in the non-test devicetree as well as
> the test one. I didn't realize this when I modified the driver for
> tests, and so broke the regular use case (which only resulted in
> warnings). First, making the pinmux and the UART group available in all
> boot phases to avoid ENODEV errors. Then, convert the pin groups and
> functions to the new style, adding onewire group as well.
>
> Fixes: 7f0f1806e3a ("test: pinmux: Add test for pin muxing")
> Closes: https://source.denx.de/u-boot/u-boot/-/issues/2 Please enter the commit message for your changes. Lines starting

Something strange there ^

> Signed-off-by: Sean Anderson <seanga2 at gmail.com>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes in v2:
> - Only add the pinctrl node pre-relocation, as SPL/VPL try to run dtoc on all
>   nodes included in their device trees.
>
>  arch/sandbox/dts/sandbox.dtsi                | 14 ++++++++------
>  drivers/pinctrl/pinctrl-sandbox.c            |  4 +++-
>  include/dt-bindings/pinctrl/sandbox-pinmux.h |  1 +
>  3 files changed, 12 insertions(+), 7 deletions(-)

Regards,
Simon


More information about the U-Boot mailing list