Hi Sean, On Thu, 15 Aug 2024 at 18:48, Sean Anderson <sean...@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 <sean...@gmail.com> > Reviewed-by: Simon Glass <s...@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