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

[Dario Binacchi]

Hi Lokesh,

> Il 04/02/2021 19:02 Lokesh Vutla <lokeshvutla at ti.com> ha scritto:
> 
>  
> On 29/01/21 3:54 am, Dario Binacchi wrote:
> > 
> > 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 be cleanly applied to the HEAD of the master which
> > at the time of release points to 8b195f4b71
> > ("Merge https://gitlab.denx.de/u-boot/custodians/u-boot-marvell") commit
> 
> Sandbox tests are failing with this series. Can you take a look?
> https://gitlab.denx.de/u-boot/custodians/u-boot-ti/-/jobs/217033
> 

I applied the series to both the "master" branch of u-boot and the "for-rc" 
branch of u-boot-ti. 
In both cases, no test fails.

Here's how I ran the tests (u-boot-ti case):

git clone https://gitlab.denx.de/u-boot/custodians/u-boot-ti.git
cd u-boot-ti/
git checkout -b for-rc origin/for-rc
git am ~/Downloads/Add-support-for-pinmux-status-command-on-beaglebone.patch
virtualenv -p /usr/bin/python3 /tmp/venv
. /tmp/venv/bin/activate
pip install -r test/py/requirements.txt
./test/py/test.py -ra --bd sandbox --build

...
=========== 719 passed, 81 skipped in 117.37s (0:01:57) =========


What am I doing wrong? 
Can you help me ?

Thanks and regards,
Dario

> Thanks and regards,
> Lokesh
> 
> > 
> > Changes in v2:
> > - Added Simon Glass review tag.
> > - Updated commit message.
> > - Added Simon Glass review tag.
> > - Added Pratyush Yadav review tag.
> > - Updated commit message
> > - Split in 2 commits
> > - Check dev_read_addr_size return value
> > - Updated commit message.
> > - Added Simon Glass review tag.
> > - Updated commit message.
> > - Remove pointer to access functions.
> > - Added Simon Glass review tag.
> > - Added Simon Glass review tag.
> > - Added Simon Glass review tag.
> > - Added error checking when the 'width' property is missing.
> > - Fix coding style.
> > 
> > Dario Binacchi (12):
> >   pinctrl: single: fix format of structure documentation
> >   pinctrl: single: fix the loop counter variable type
> >   pinctrl: single: fix offset management
> >   pinctrl: single: fix debug messages formatting
> >   pinctrl: single: get register area size by device API
> >   pinctrl: single: check "register-width" DT property
> >   pinctrl: single: change function mask default value
> >   pinctrl: single: add register access functions
> >   pinctrl: single: add get_pins_count operation
> >   pinctrl: single: add get_pin_name operation
> >   pinctrl: single: add get_pin_muxing operation
> >   test: pinmux: add test for 'pinctrl-single' driver
> > 
> >  arch/sandbox/dts/test.dts        |  72 +++++
> >  configs/sandbox_defconfig        |   1 +
> >  drivers/pinctrl/pinctrl-single.c | 474 +++++++++++++++++++++++++++----
> >  test/dm/pinmux.c                 |  91 +++++-
> >  4 files changed, 573 insertions(+), 65 deletions(-)
> >