* [PATCH v2 01/12] pinctrl: single: fix format of structure documentation
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
@ 2021-01-28 22:24 ` Dario Binacchi
2021-01-28 22:24 ` [PATCH v2 02/12] pinctrl: single: fix the loop counter variable type Dario Binacchi
` (11 subsequent siblings)
12 siblings, 0 replies; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:24 UTC (permalink / raw)
To: u-boot
U-Boot adopted the kernel-doc annotation style.
Signed-off-by: Dario Binacchi <dariobin@libero.it>
Reviewed-by: Simon Glass <sjg@chromium.org>
---
Changes in v2:
- Added Simon Glass review tag.
drivers/pinctrl/pinctrl-single.c | 45 +++++++++++++++++++++++++-------
1 file changed, 36 insertions(+), 9 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index 20c3c82aa9..c9a6c272bf 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -10,23 +10,50 @@
#include <linux/libfdt.h>
#include <asm/io.h>
+/**
+ * struct single_pdata - platform data
+ * @base: first configuration register
+ * @offset: index of last configuration register
+ * @mask: configuration-value mask bits
+ * @width: configuration register bit width
+ * @bits_per_mux: true if one register controls more than one pin
+ */
struct single_pdata {
- fdt_addr_t base; /* first configuration register */
- int offset; /* index of last configuration register */
- u32 mask; /* configuration-value mask bits */
- int width; /* configuration register bit width */
+ fdt_addr_t base;
+ int offset;
+ u32 mask;
+ int width;
bool bits_per_mux;
};
+/**
+ * struct single_fdt_pin_cfg - pin configuration
+ *
+ * This structure is used for the pin configuration parameters in case
+ * the register controls only one pin.
+ *
+ * @reg: configuration register offset
+ * @val: configuration register value
+ */
struct single_fdt_pin_cfg {
- fdt32_t reg; /* configuration register offset */
- fdt32_t val; /* configuration register value */
+ fdt32_t reg;
+ fdt32_t val;
};
+/**
+ * struct single_fdt_bits_cfg - pin configuration
+ *
+ * This structure is used for the pin configuration parameters in case
+ * the register controls more than one pin.
+ *
+ * @reg: configuration register offset
+ * @val: configuration register value
+ * @mask: configuration register mask
+ */
struct single_fdt_bits_cfg {
- fdt32_t reg; /* configuration register offset */
- fdt32_t val; /* configuration register value */
- fdt32_t mask; /* configuration register mask */
+ fdt32_t reg;
+ fdt32_t val;
+ fdt32_t mask;
};
/**
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 02/12] pinctrl: single: fix the loop counter variable type
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
2021-01-28 22:24 ` [PATCH v2 01/12] pinctrl: single: fix format of structure documentation Dario Binacchi
@ 2021-01-28 22:24 ` Dario Binacchi
2021-01-28 22:24 ` [PATCH v2 03/12] pinctrl: single: fix offset management Dario Binacchi
` (10 subsequent siblings)
12 siblings, 0 replies; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:24 UTC (permalink / raw)
To: u-boot
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 <dariobin@libero.it>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Pratyush Yadav <p.yadav@ti.com>
---
Changes in v2:
- Updated commit message.
- Added Simon Glass review tag.
- Added Pratyush Yadav review tag.
drivers/pinctrl/pinctrl-single.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index c9a6c272bf..49ed15211d 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -75,8 +75,8 @@ static int single_configure_pins(struct udevice *dev,
int size)
{
struct single_pdata *pdata = dev_get_plat(dev);
- int count = size / sizeof(struct single_fdt_pin_cfg);
- phys_addr_t n, reg;
+ int n, count = size / sizeof(struct single_fdt_pin_cfg);
+ phys_addr_t reg;
u32 val;
for (n = 0; n < count; n++, pins++) {
@@ -109,8 +109,8 @@ static int single_configure_bits(struct udevice *dev,
int size)
{
struct single_pdata *pdata = dev_get_plat(dev);
- int count = size / sizeof(struct single_fdt_bits_cfg);
- phys_addr_t n, reg;
+ int n, count = size / sizeof(struct single_fdt_bits_cfg);
+ phys_addr_t reg;
u32 val, mask;
for (n = 0; n < count; n++, pins++) {
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 03/12] pinctrl: single: fix offset management
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
2021-01-28 22:24 ` [PATCH v2 01/12] pinctrl: single: fix format of structure documentation Dario Binacchi
2021-01-28 22:24 ` [PATCH v2 02/12] pinctrl: single: fix the loop counter variable type Dario Binacchi
@ 2021-01-28 22:24 ` Dario Binacchi
2021-01-29 16:50 ` Pratyush Yadav
2021-01-28 22:24 ` [PATCH v2 04/12] pinctrl: single: fix debug messages formatting Dario Binacchi
` (9 subsequent siblings)
12 siblings, 1 reply; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:24 UTC (permalink / raw)
To: u-boot
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 option than a physical address.
Signed-off-by: Dario Binacchi <dariobin@libero.it>
---
(no changes since v1)
drivers/pinctrl/pinctrl-single.c | 24 ++++++++++++++----------
1 file changed, 14 insertions(+), 10 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index 49ed15211d..935b5e920d 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -77,15 +77,17 @@ static int single_configure_pins(struct udevice *dev,
struct single_pdata *pdata = dev_get_plat(dev);
int n, count = size / sizeof(struct single_fdt_pin_cfg);
phys_addr_t reg;
- u32 val;
+ u32 offset, val;
for (n = 0; n < count; n++, pins++) {
- reg = fdt32_to_cpu(pins->reg);
- if ((reg < 0) || (reg > pdata->offset)) {
- dev_dbg(dev, " invalid register offset 0x%pa\n", ®);
+ offset = fdt32_to_cpu(pins->reg);
+ if (offset < 0 || offset > pdata->offset) {
+ dev_dbg(dev, " invalid register offset 0x%x\n",
+ offset);
continue;
}
- reg += pdata->base;
+
+ reg = pdata->base + offset;
val = fdt32_to_cpu(pins->val) & pdata->mask;
switch (pdata->width) {
case 16:
@@ -111,15 +113,17 @@ static int single_configure_bits(struct udevice *dev,
struct single_pdata *pdata = dev_get_plat(dev);
int n, count = size / sizeof(struct single_fdt_bits_cfg);
phys_addr_t reg;
- u32 val, mask;
+ u32 offset, val, mask;
for (n = 0; n < count; n++, pins++) {
- reg = fdt32_to_cpu(pins->reg);
- if ((reg < 0) || (reg > pdata->offset)) {
- dev_dbg(dev, " invalid register offset 0x%pa\n", ®);
+ offset = fdt32_to_cpu(pins->reg);
+ if (offset < 0 || offset > pdata->offset) {
+ dev_dbg(dev, " invalid register offset 0x%x\n",
+ offset);
continue;
}
- reg += pdata->base;
+
+ reg = pdata->base + offset;
mask = fdt32_to_cpu(pins->mask);
val = fdt32_to_cpu(pins->val) & mask;
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 03/12] pinctrl: single: fix offset management
2021-01-28 22:24 ` [PATCH v2 03/12] pinctrl: single: fix offset management Dario Binacchi
@ 2021-01-29 16:50 ` Pratyush Yadav
0 siblings, 0 replies; 23+ messages in thread
From: Pratyush Yadav @ 2021-01-29 16:50 UTC (permalink / raw)
To: u-boot
On 28/01/21 11:24PM, Dario Binacchi wrote:
> 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 option than a physical address.
>
> Signed-off-by: Dario Binacchi <dariobin@libero.it>
Reviewed-by: Pratyush Yadav <p.yadav@ti.com>
--
Regards,
Pratyush Yadav
Texas Instruments India
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v2 04/12] pinctrl: single: fix debug messages formatting
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
` (2 preceding siblings ...)
2021-01-28 22:24 ` [PATCH v2 03/12] pinctrl: single: fix offset management Dario Binacchi
@ 2021-01-28 22:24 ` Dario Binacchi
2021-01-29 16:53 ` Pratyush Yadav
2021-01-28 22:24 ` [PATCH v2 05/12] pinctrl: single: get register area size by device API Dario Binacchi
` (8 subsequent siblings)
12 siblings, 1 reply; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:24 UTC (permalink / raw)
To: u-boot
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 appends the prefix '0x' to the
address displayed.
Signed-off-by: Dario Binacchi <dariobin@libero.it>
---
Changes in v2:
- Updated commit message
- Split in 2 commits
drivers/pinctrl/pinctrl-single.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index 935b5e920d..cec00e289c 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -101,7 +101,7 @@ static int single_configure_pins(struct udevice *dev,
pdata->width);
continue;
}
- dev_dbg(dev, " reg/val 0x%pa/0x%08x\n", ®, val);
+ dev_dbg(dev, " reg/val %pa/0x%08x\n", ®, val);
}
return 0;
}
@@ -140,7 +140,7 @@ static int single_configure_bits(struct udevice *dev,
pdata->width);
continue;
}
- dev_dbg(dev, " reg/val 0x%pa/0x%08x\n", ®, val);
+ dev_dbg(dev, " reg/val %pa/0x%08x\n", ®, val);
}
return 0;
}
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 04/12] pinctrl: single: fix debug messages formatting
2021-01-28 22:24 ` [PATCH v2 04/12] pinctrl: single: fix debug messages formatting Dario Binacchi
@ 2021-01-29 16:53 ` Pratyush Yadav
0 siblings, 0 replies; 23+ messages in thread
From: Pratyush Yadav @ 2021-01-29 16:53 UTC (permalink / raw)
To: u-boot
On 28/01/21 11:24PM, Dario Binacchi wrote:
> 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 appends the prefix '0x' to the
s/appends/prepends/
Other than that,
Reviewed-by: Pratyush Yadav <p.yadav@ti.com>
> address displayed.
>
> Signed-off-by: Dario Binacchi <dariobin@libero.it>
>
> ---
>
> Changes in v2:
> - Updated commit message
> - Split in 2 commits
>
> drivers/pinctrl/pinctrl-single.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
> index 935b5e920d..cec00e289c 100644
> --- a/drivers/pinctrl/pinctrl-single.c
> +++ b/drivers/pinctrl/pinctrl-single.c
> @@ -101,7 +101,7 @@ static int single_configure_pins(struct udevice *dev,
> pdata->width);
> continue;
> }
> - dev_dbg(dev, " reg/val 0x%pa/0x%08x\n", ®, val);
> + dev_dbg(dev, " reg/val %pa/0x%08x\n", ®, val);
> }
> return 0;
> }
> @@ -140,7 +140,7 @@ static int single_configure_bits(struct udevice *dev,
> pdata->width);
> continue;
> }
> - dev_dbg(dev, " reg/val 0x%pa/0x%08x\n", ®, val);
> + dev_dbg(dev, " reg/val %pa/0x%08x\n", ®, val);
> }
> return 0;
> }
> --
> 2.17.1
>
--
Regards,
Pratyush Yadav
Texas Instruments India
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v2 05/12] pinctrl: single: get register area size by device API
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
` (3 preceding siblings ...)
2021-01-28 22:24 ` [PATCH v2 04/12] pinctrl: single: fix debug messages formatting Dario Binacchi
@ 2021-01-28 22:24 ` Dario Binacchi
2021-01-29 17:47 ` Pratyush Yadav
2021-01-28 22:24 ` [PATCH v2 06/12] pinctrl: single: check "register-width" DT property Dario Binacchi
` (7 subsequent siblings)
12 siblings, 1 reply; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:24 UTC (permalink / raw)
To: u-boot
Use dev_read_addr_size to get size of the controller's register area.
Signed-off-by: Dario Binacchi <dariobin@libero.it>
---
Changes in v2:
- Check dev_read_addr_size return value
drivers/pinctrl/pinctrl-single.c | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index cec00e289c..d5656de8e8 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -182,17 +182,19 @@ static int single_set_state(struct udevice *dev,
static int single_of_to_plat(struct udevice *dev)
{
fdt_addr_t addr;
- u32 of_reg[2];
- int res;
+ fdt_size_t size;
struct single_pdata *pdata = dev_get_plat(dev);
pdata->width =
dev_read_u32_default(dev, "pinctrl-single,register-width", 0);
- res = dev_read_u32_array(dev, "reg", of_reg, 2);
- if (res)
- return res;
- pdata->offset = of_reg[1] - pdata->width / 8;
+ addr = dev_read_addr_size(dev, "reg", &size);
+ if (addr == FDT_ADDR_T_NONE) {
+ dev_err(dev, "failed to get base register size\n");
+ return -EINVAL;
+ }
+
+ pdata->offset = size - pdata->width / BITS_PER_BYTE;
addr = dev_read_addr(dev);
if (addr == FDT_ADDR_T_NONE) {
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 06/12] pinctrl: single: check "register-width" DT property
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
` (4 preceding siblings ...)
2021-01-28 22:24 ` [PATCH v2 05/12] pinctrl: single: get register area size by device API Dario Binacchi
@ 2021-01-28 22:24 ` Dario Binacchi
2021-01-28 22:24 ` [PATCH v2 07/12] pinctrl: single: change function mask default value Dario Binacchi
` (6 subsequent siblings)
12 siblings, 0 replies; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:24 UTC (permalink / raw)
To: u-boot
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: Dario Binacchi <dariobin@libero.it>
Reviewed-by: Simon Glass <sjg@chromium.org>
---
Changes in v2:
- Updated commit message.
- Added Simon Glass review tag.
drivers/pinctrl/pinctrl-single.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index d5656de8e8..5ade108875 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -22,7 +22,7 @@ struct single_pdata {
fdt_addr_t base;
int offset;
u32 mask;
- int width;
+ u32 width;
bool bits_per_mux;
};
@@ -184,9 +184,13 @@ static int single_of_to_plat(struct udevice *dev)
fdt_addr_t addr;
fdt_size_t size;
struct single_pdata *pdata = dev_get_plat(dev);
+ int ret;
- pdata->width =
- dev_read_u32_default(dev, "pinctrl-single,register-width", 0);
+ ret = dev_read_u32(dev, "pinctrl-single,register-width", &pdata->width);
+ if (ret) {
+ dev_err(dev, "missing register width\n");
+ return ret;
+ }
addr = dev_read_addr_size(dev, "reg", &size);
if (addr == FDT_ADDR_T_NONE) {
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 07/12] pinctrl: single: change function mask default value
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
` (5 preceding siblings ...)
2021-01-28 22:24 ` [PATCH v2 06/12] pinctrl: single: check "register-width" DT property Dario Binacchi
@ 2021-01-28 22:24 ` Dario Binacchi
2021-02-01 20:44 ` Simon Glass
2021-01-28 22:24 ` [PATCH v2 08/12] pinctrl: single: add register access functions Dario Binacchi
` (5 subsequent siblings)
12 siblings, 1 reply; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:24 UTC (permalink / raw)
To: u-boot
The patch is inspired by more recent versions of the Linux driver.
Replacing the default value 0xffffffff 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 <dariobin@libero.it>
---
(no changes since v1)
drivers/pinctrl/pinctrl-single.c | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index 5ade108875..630a6c08b8 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -79,6 +79,10 @@ static int single_configure_pins(struct udevice *dev,
phys_addr_t reg;
u32 offset, val;
+ /* If function mask is null, needn't enable it. */
+ if (!pdata->mask)
+ return 0;
+
for (n = 0; n < count; n++, pins++) {
offset = fdt32_to_cpu(pins->reg);
if (offset < 0 || offset > pdata->offset) {
@@ -207,8 +211,12 @@ static int single_of_to_plat(struct udevice *dev)
}
pdata->base = addr;
- pdata->mask = dev_read_u32_default(dev, "pinctrl-single,function-mask",
- 0xffffffff);
+ ret = dev_read_u32(dev, "pinctrl-single,function-mask", &pdata->mask);
+ if (ret) {
+ pdata->mask = 0;
+ dev_warn(dev, "missing function register mask\n");
+ }
+
pdata->bits_per_mux = dev_read_bool(dev, "pinctrl-single,bit-per-mux");
return 0;
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 07/12] pinctrl: single: change function mask default value
2021-01-28 22:24 ` [PATCH v2 07/12] pinctrl: single: change function mask default value Dario Binacchi
@ 2021-02-01 20:44 ` Simon Glass
0 siblings, 0 replies; 23+ messages in thread
From: Simon Glass @ 2021-02-01 20:44 UTC (permalink / raw)
To: u-boot
On Thu, 28 Jan 2021 at 15:25, Dario Binacchi <dariobin@libero.it> wrote:
>
> The patch is inspired by more recent versions of the Linux driver.
> Replacing the default value 0xffffffff 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 <dariobin@libero.it>
> ---
>
> (no changes since v1)
>
> drivers/pinctrl/pinctrl-single.c | 12 ++++++++++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v2 08/12] pinctrl: single: add register access functions
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
` (6 preceding siblings ...)
2021-01-28 22:24 ` [PATCH v2 07/12] pinctrl: single: change function mask default value Dario Binacchi
@ 2021-01-28 22:24 ` Dario Binacchi
2021-02-01 20:44 ` Simon Glass
2021-01-28 22:24 ` [PATCH v2 09/12] pinctrl: single: add get_pins_count operation Dario Binacchi
` (4 subsequent siblings)
12 siblings, 1 reply; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:24 UTC (permalink / raw)
To: u-boot
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 <dariobin@libero.it>
---
Changes in v2:
- Updated commit message.
- Remove pointer to access functions.
drivers/pinctrl/pinctrl-single.c | 71 +++++++++++++++++++++-----------
1 file changed, 46 insertions(+), 25 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index 630a6c08b8..5b3ccc2281 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -56,6 +56,38 @@ struct single_fdt_bits_cfg {
fdt32_t mask;
};
+static unsigned int single_read(struct udevice *dev, fdt_addr_t reg)
+{
+ struct single_pdata *pdata = dev_get_plat(dev);
+
+ switch (pdata->width) {
+ case 8:
+ return readb(reg);
+ case 16:
+ return readw(reg);
+ default: /* 32 bits */
+ return readl(reg);
+ }
+
+ return readb(reg);
+}
+
+static void single_write(struct udevice *dev, unsigned int val, fdt_addr_t reg)
+{
+ struct single_pdata *pdata = dev_get_plat(dev);
+
+ switch (pdata->width) {
+ case 8:
+ writeb(val, reg);
+ break;
+ case 16:
+ writew(val, reg);
+ break;
+ default: /* 32 bits */
+ writel(val, reg);
+ }
+}
+
/**
* single_configure_pins() - Configure pins based on FDT data
*
@@ -93,19 +125,10 @@ static int single_configure_pins(struct udevice *dev,
reg = pdata->base + offset;
val = fdt32_to_cpu(pins->val) & pdata->mask;
- switch (pdata->width) {
- case 16:
- writew((readw(reg) & ~pdata->mask) | val, reg);
- break;
- case 32:
- writel((readl(reg) & ~pdata->mask) | val, reg);
- break;
- default:
- dev_warn(dev, "unsupported register width %i\n",
- pdata->width);
- continue;
- }
+ single_write(dev, (single_read(dev, reg) & ~pdata->mask) | val,
+ reg);
dev_dbg(dev, " reg/val %pa/0x%08x\n", ®, val);
+
}
return 0;
}
@@ -131,19 +154,7 @@ static int single_configure_bits(struct udevice *dev,
mask = fdt32_to_cpu(pins->mask);
val = fdt32_to_cpu(pins->val) & mask;
-
- switch (pdata->width) {
- case 16:
- writew((readw(reg) & ~mask) | val, reg);
- break;
- case 32:
- writel((readl(reg) & ~mask) | val, reg);
- break;
- default:
- dev_warn(dev, "unsupported register width %i\n",
- pdata->width);
- continue;
- }
+ single_write(dev, (single_read(dev, reg) & ~mask) | val, reg);
dev_dbg(dev, " reg/val %pa/0x%08x\n", ®, val);
}
return 0;
@@ -196,6 +207,16 @@ static int single_of_to_plat(struct udevice *dev)
return ret;
}
+ switch (pdata->width) {
+ case 8:
+ case 16:
+ case 32:
+ break;
+ default:
+ dev_err(dev, "wrong register width\n");
+ return -EINVAL;
+ }
+
addr = dev_read_addr_size(dev, "reg", &size);
if (addr == FDT_ADDR_T_NONE) {
dev_err(dev, "failed to get base register size\n");
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 08/12] pinctrl: single: add register access functions
2021-01-28 22:24 ` [PATCH v2 08/12] pinctrl: single: add register access functions Dario Binacchi
@ 2021-02-01 20:44 ` Simon Glass
0 siblings, 0 replies; 23+ messages in thread
From: Simon Glass @ 2021-02-01 20:44 UTC (permalink / raw)
To: u-boot
On Thu, 28 Jan 2021 at 15:25, Dario Binacchi <dariobin@libero.it> wrote:
>
> 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 <dariobin@libero.it>
>
> ---
>
> Changes in v2:
> - Updated commit message.
> - Remove pointer to access functions.
>
> drivers/pinctrl/pinctrl-single.c | 71 +++++++++++++++++++++-----------
> 1 file changed, 46 insertions(+), 25 deletions(-)
>
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v2 09/12] pinctrl: single: add get_pins_count operation
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
` (7 preceding siblings ...)
2021-01-28 22:24 ` [PATCH v2 08/12] pinctrl: single: add register access functions Dario Binacchi
@ 2021-01-28 22:24 ` Dario Binacchi
2021-01-28 22:24 ` [PATCH v2 10/12] pinctrl: single: add get_pin_name operation Dario Binacchi
` (3 subsequent siblings)
12 siblings, 0 replies; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:24 UTC (permalink / raw)
To: u-boot
It returns the number of selectable pins.
Signed-off-by: Dario Binacchi <dariobin@libero.it>
Reviewed-by: Simon Glass <sjg@chromium.org>
---
Changes in v2:
- Added Simon Glass review tag.
drivers/pinctrl/pinctrl-single.c | 37 ++++++++++++++++++++++++++++++++
1 file changed, 37 insertions(+)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index 5b3ccc2281..e99eb552f5 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -26,6 +26,16 @@ struct single_pdata {
bool bits_per_mux;
};
+/**
+ * struct single_priv - private data
+ * @bits_per_pin: number of bits per pin
+ * @npins: number of selectable pins
+ */
+struct single_priv {
+ unsigned int bits_per_pin;
+ unsigned int npins;
+};
+
/**
* struct single_fdt_pin_cfg - pin configuration
*
@@ -194,6 +204,30 @@ static int single_set_state(struct udevice *dev,
return len;
}
+static int single_get_pins_count(struct udevice *dev)
+{
+ struct single_priv *priv = dev_get_priv(dev);
+
+ return priv->npins;
+}
+
+static int single_probe(struct udevice *dev)
+{
+ struct single_pdata *pdata = dev_get_plat(dev);
+ struct single_priv *priv = dev_get_priv(dev);
+ u32 size;
+
+ size = pdata->offset + pdata->width / BITS_PER_BYTE;
+ priv->npins = size / (pdata->width / BITS_PER_BYTE);
+ if (pdata->bits_per_mux) {
+ priv->bits_per_pin = fls(pdata->mask);
+ priv->npins *= (pdata->width / priv->bits_per_pin);
+ }
+
+ dev_dbg(dev, "%d pins\n", priv->npins);
+ return 0;
+}
+
static int single_of_to_plat(struct udevice *dev)
{
fdt_addr_t addr;
@@ -244,6 +278,7 @@ static int single_of_to_plat(struct udevice *dev)
}
const struct pinctrl_ops single_pinctrl_ops = {
+ .get_pins_count = single_get_pins_count,
.set_state = single_set_state,
};
@@ -258,5 +293,7 @@ U_BOOT_DRIVER(single_pinctrl) = {
.of_match = single_pinctrl_match,
.ops = &single_pinctrl_ops,
.plat_auto = sizeof(struct single_pdata),
+ .priv_auto = sizeof(struct single_priv),
.of_to_plat = single_of_to_plat,
+ .probe = single_probe,
};
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 10/12] pinctrl: single: add get_pin_name operation
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
` (8 preceding siblings ...)
2021-01-28 22:24 ` [PATCH v2 09/12] pinctrl: single: add get_pins_count operation Dario Binacchi
@ 2021-01-28 22:24 ` Dario Binacchi
2021-01-28 22:25 ` [PATCH v2 11/12] pinctrl: single: add get_pin_muxing operation Dario Binacchi
` (2 subsequent siblings)
12 siblings, 0 replies; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:24 UTC (permalink / raw)
To: u-boot
It returns the name of the requested pin.
Signed-off-by: Dario Binacchi <dariobin@libero.it>
Reviewed-by: Simon Glass <sjg@chromium.org>
---
Changes in v2:
- Added Simon Glass review tag.
drivers/pinctrl/pinctrl-single.c | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index e99eb552f5..cdb5040852 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -30,10 +30,12 @@ struct single_pdata {
* struct single_priv - private data
* @bits_per_pin: number of bits per pin
* @npins: number of selectable pins
+ * @pin_name: temporary buffer to store the pin name
*/
struct single_priv {
unsigned int bits_per_pin;
unsigned int npins;
+ char pin_name[PINNAME_SIZE];
};
/**
@@ -204,6 +206,19 @@ static int single_set_state(struct udevice *dev,
return len;
}
+static const char *single_get_pin_name(struct udevice *dev,
+ unsigned int selector)
+{
+ struct single_priv *priv = dev_get_priv(dev);
+
+ if (selector >= priv->npins)
+ snprintf(priv->pin_name, PINNAME_SIZE, "Error");
+ else
+ snprintf(priv->pin_name, PINNAME_SIZE, "PIN%u", selector);
+
+ return priv->pin_name;
+}
+
static int single_get_pins_count(struct udevice *dev)
{
struct single_priv *priv = dev_get_priv(dev);
@@ -279,6 +294,7 @@ static int single_of_to_plat(struct udevice *dev)
const struct pinctrl_ops single_pinctrl_ops = {
.get_pins_count = single_get_pins_count,
+ .get_pin_name = single_get_pin_name,
.set_state = single_set_state,
};
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 11/12] pinctrl: single: add get_pin_muxing operation
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
` (9 preceding siblings ...)
2021-01-28 22:24 ` [PATCH v2 10/12] pinctrl: single: add get_pin_name operation Dario Binacchi
@ 2021-01-28 22:25 ` Dario Binacchi
2021-02-01 20:44 ` Simon Glass
2021-01-28 22:25 ` [PATCH v2 12/12] test: pinmux: add test for 'pinctrl-single' driver Dario Binacchi
2021-02-04 18:02 ` [PATCH v2 00/12] Add support for pinmux status command on beaglebone Lokesh Vutla
12 siblings, 1 reply; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:25 UTC (permalink / raw)
To: u-boot
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 a generic type driver, and it can be useful for debug
purposes.
Signed-off-by: Dario Binacchi <dariobin@libero.it>
---
(no changes since v1)
drivers/pinctrl/pinctrl-single.c | 222 +++++++++++++++++++++++++++++--
1 file changed, 213 insertions(+), 9 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index cdb5040852..3ddb637ab7 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -1,14 +1,18 @@
// SPDX-License-Identifier: GPL-2.0+
/*
* Copyright (C) EETS GmbH, 2017, Felix Brack <f.brack@eets.ch>
+ * Copyright (C) 2021 Dario Binacchi <dariobin@libero.it>
*/
#include <common.h>
#include <dm.h>
#include <dm/device_compat.h>
+#include <dm/devres.h>
#include <dm/pinctrl.h>
#include <linux/libfdt.h>
+#include <linux/list.h>
#include <asm/io.h>
+#include <sort.h>
/**
* struct single_pdata - platform data
@@ -26,6 +30,20 @@ struct single_pdata {
bool bits_per_mux;
};
+/**
+ * struct single_func - pinctrl function
+ * @node: list node
+ * @name: pinctrl function name
+ * @npins: number of entries in pins array
+ * @pins: pins array
+ */
+struct single_func {
+ struct list_head node;
+ const char *name;
+ unsigned int npins;
+ unsigned int *pins;
+};
+
/**
* struct single_priv - private data
* @bits_per_pin: number of bits per pin
@@ -36,6 +54,7 @@ struct single_priv {
unsigned int bits_per_pin;
unsigned int npins;
char pin_name[PINNAME_SIZE];
+ struct list_head functions;
};
/**
@@ -100,6 +119,121 @@ static void single_write(struct udevice *dev, unsigned int val, fdt_addr_t reg)
}
}
+/**
+ * single_get_pin_by_offset() - get a pin based on the register offset
+ * @dev: single driver instance
+ * @offset: register offset from the base
+ */
+static int single_get_pin_by_offset(struct udevice *dev, unsigned int offset)
+{
+ struct single_pdata *pdata = dev_get_plat(dev);
+ struct single_priv *priv = dev_get_priv(dev);
+
+ if (offset > pdata->offset) {
+ dev_err(dev, "mux offset out of range: 0x%x (0x%x)\n",
+ offset, pdata->offset);
+ return -EINVAL;
+ }
+
+ if (pdata->bits_per_mux)
+ return (offset * BITS_PER_BYTE) / priv->bits_per_pin;
+
+ return offset / (pdata->width / BITS_PER_BYTE);
+}
+
+static int single_get_offset_by_pin(struct udevice *dev, unsigned int pin)
+{
+ struct single_pdata *pdata = dev_get_plat(dev);
+ struct single_priv *priv = dev_get_priv(dev);
+ unsigned int mux_bytes;
+
+ if (pin >= priv->npins)
+ return -EINVAL;
+
+ mux_bytes = pdata->width / BITS_PER_BYTE;
+ if (pdata->bits_per_mux) {
+ int byte_num;
+
+ byte_num = (priv->bits_per_pin * pin) / BITS_PER_BYTE;
+ return (byte_num / mux_bytes) * mux_bytes;
+ }
+
+ return pin * mux_bytes;
+}
+
+static const char *single_get_pin_function(struct udevice *dev,
+ unsigned int pin)
+{
+ struct single_priv *priv = dev_get_priv(dev);
+ struct single_func *func;
+ int i;
+
+ list_for_each_entry(func, &priv->functions, node) {
+ for (i = 0; i < func->npins; i++) {
+ if (pin == func->pins[i])
+ return func->name;
+
+ if (pin < func->pins[i])
+ break;
+ }
+ }
+
+ return NULL;
+}
+
+static int single_get_pin_muxing(struct udevice *dev, unsigned int pin,
+ char *buf, int size)
+{
+ struct single_pdata *pdata = dev_get_plat(dev);
+ struct single_priv *priv = dev_get_priv(dev);
+ fdt_addr_t reg;
+ const char *fname;
+ unsigned int val;
+ int offset, pin_shift = 0;
+
+ offset = single_get_offset_by_pin(dev, pin);
+ if (offset < 0)
+ return offset;
+
+ reg = pdata->base + offset;
+ val = single_read(dev, reg);
+
+ if (pdata->bits_per_mux)
+ pin_shift = pin % (pdata->width / priv->bits_per_pin) *
+ priv->bits_per_pin;
+
+ val &= (pdata->mask << pin_shift);
+ fname = single_get_pin_function(dev, pin);
+ snprintf(buf, size, "%pa 0x%08x %s", ®, val,
+ fname ? fname : "UNCLAIMED");
+ return 0;
+}
+
+static struct single_func *single_allocate_function(struct udevice *dev,
+ unsigned int group_pins)
+{
+ struct single_func *func;
+
+ func = devm_kmalloc(dev, sizeof(*func), GFP_KERNEL);
+ if (!func)
+ return ERR_PTR(-ENOMEM);
+
+ func->pins = devm_kmalloc(dev, sizeof(unsigned int) * group_pins,
+ GFP_KERNEL);
+ if (!func->pins)
+ return ERR_PTR(-ENOMEM);
+
+ return func;
+}
+
+static int single_pin_compare(const void *s1, const void *s2)
+{
+ int pin1 = *(const unsigned int *)s1;
+ int pin2 = *(const unsigned int *)s2;
+
+ return pin1 - pin2;
+}
+
/**
* single_configure_pins() - Configure pins based on FDT data
*
@@ -113,13 +247,16 @@ static void single_write(struct udevice *dev, unsigned int val, fdt_addr_t reg)
* @size: Size of the 'pins' array in bytes.
* The number of register/value pairs in the 'pins' array therefore
* equals to 'size / sizeof(struct single_fdt_pin_cfg)'.
+ * @fname: Function name.
*/
static int single_configure_pins(struct udevice *dev,
const struct single_fdt_pin_cfg *pins,
- int size)
+ int size, const char *fname)
{
struct single_pdata *pdata = dev_get_plat(dev);
- int n, count = size / sizeof(struct single_fdt_pin_cfg);
+ struct single_priv *priv = dev_get_priv(dev);
+ int n, pin, count = size / sizeof(struct single_fdt_pin_cfg);
+ struct single_func *func;
phys_addr_t reg;
u32 offset, val;
@@ -127,33 +264,61 @@ static int single_configure_pins(struct udevice *dev,
if (!pdata->mask)
return 0;
+ func = single_allocate_function(dev, count);
+ if (IS_ERR(func))
+ return PTR_ERR(func);
+
+ func->name = fname;
+ func->npins = 0;
for (n = 0; n < count; n++, pins++) {
offset = fdt32_to_cpu(pins->reg);
if (offset < 0 || offset > pdata->offset) {
- dev_dbg(dev, " invalid register offset 0x%x\n",
+ dev_err(dev, " invalid register offset 0x%x\n",
offset);
continue;
}
reg = pdata->base + offset;
val = fdt32_to_cpu(pins->val) & pdata->mask;
+ pin = single_get_pin_by_offset(dev, offset);
+ if (pin < 0) {
+ dev_err(dev, " failed to get pin by offset %x\n",
+ offset);
+ continue;
+ }
+
single_write(dev, (single_read(dev, reg) & ~pdata->mask) | val,
reg);
dev_dbg(dev, " reg/val %pa/0x%08x\n", ®, val);
-
+ func->pins[func->npins] = pin;
+ func->npins++;
}
+
+ qsort(func->pins, func->npins, sizeof(func->pins[0]),
+ single_pin_compare);
+ list_add(&func->node, &priv->functions);
return 0;
}
static int single_configure_bits(struct udevice *dev,
const struct single_fdt_bits_cfg *pins,
- int size)
+ int size, const char *fname)
{
struct single_pdata *pdata = dev_get_plat(dev);
- int n, count = size / sizeof(struct single_fdt_bits_cfg);
+ struct single_priv *priv = dev_get_priv(dev);
+ int n, pin, count = size / sizeof(struct single_fdt_bits_cfg);
+ int npins_in_reg, pin_num_from_lsb;
+ struct single_func *func;
phys_addr_t reg;
- u32 offset, val, mask;
+ u32 offset, val, mask, bit_pos, val_pos, mask_pos, submask;
+
+ npins_in_reg = pdata->width / priv->bits_per_pin;
+ func = single_allocate_function(dev, count * npins_in_reg);
+ if (IS_ERR(func))
+ return PTR_ERR(func);
+ func->name = fname;
+ func->npins = 0;
for (n = 0; n < count; n++, pins++) {
offset = fdt32_to_cpu(pins->reg);
if (offset < 0 || offset > pdata->offset) {
@@ -164,11 +329,47 @@ static int single_configure_bits(struct udevice *dev,
reg = pdata->base + offset;
+ pin = single_get_pin_by_offset(dev, offset);
+ if (pin < 0) {
+ dev_err(dev, " failed to get pin by offset 0x%pa\n",
+ ®);
+ continue;
+ }
+
mask = fdt32_to_cpu(pins->mask);
val = fdt32_to_cpu(pins->val) & mask;
single_write(dev, (single_read(dev, reg) & ~mask) | val, reg);
dev_dbg(dev, " reg/val %pa/0x%08x\n", ®, val);
+
+ while (mask) {
+ bit_pos = __ffs(mask);
+ pin_num_from_lsb = bit_pos / priv->bits_per_pin;
+ mask_pos = pdata->mask << bit_pos;
+ val_pos = val & mask_pos;
+ submask = mask & mask_pos;
+
+ if ((mask & mask_pos) == 0) {
+ dev_err(dev, "Invalid mask at 0x%x\n", offset);
+ break;
+ }
+
+ mask &= ~mask_pos;
+
+ if (submask != mask_pos) {
+ dev_warn(dev,
+ "Invalid submask 0x%x@0x%x\n",
+ submask, offset);
+ continue;
+ }
+
+ func->pins[func->npins] = pin + pin_num_from_lsb;
+ func->npins++;
+ }
}
+
+ qsort(func->pins, func->npins, sizeof(func->pins[0]),
+ single_pin_compare);
+ list_add(&func->node, &priv->functions);
return 0;
}
static int single_set_state(struct udevice *dev,
@@ -186,7 +387,7 @@ static int single_set_state(struct udevice *dev,
dev_dbg(dev, " invalid pin configuration in fdt\n");
return -FDT_ERR_BADSTRUCTURE;
}
- single_configure_pins(dev, prop, len);
+ single_configure_pins(dev, prop, len, config->name);
return 0;
}
@@ -198,7 +399,7 @@ static int single_set_state(struct udevice *dev,
dev_dbg(dev, " invalid bits configuration in fdt\n");
return -FDT_ERR_BADSTRUCTURE;
}
- single_configure_bits(dev, prop_bits, len);
+ single_configure_bits(dev, prop_bits, len, config->name);
return 0;
}
@@ -232,6 +433,8 @@ static int single_probe(struct udevice *dev)
struct single_priv *priv = dev_get_priv(dev);
u32 size;
+ INIT_LIST_HEAD(&priv->functions);
+
size = pdata->offset + pdata->width / BITS_PER_BYTE;
priv->npins = size / (pdata->width / BITS_PER_BYTE);
if (pdata->bits_per_mux) {
@@ -296,6 +499,7 @@ const struct pinctrl_ops single_pinctrl_ops = {
.get_pins_count = single_get_pins_count,
.get_pin_name = single_get_pin_name,
.set_state = single_set_state,
+ .get_pin_muxing = single_get_pin_muxing,
};
static const struct udevice_id single_pinctrl_match[] = {
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 11/12] pinctrl: single: add get_pin_muxing operation
2021-01-28 22:25 ` [PATCH v2 11/12] pinctrl: single: add get_pin_muxing operation Dario Binacchi
@ 2021-02-01 20:44 ` Simon Glass
0 siblings, 0 replies; 23+ messages in thread
From: Simon Glass @ 2021-02-01 20:44 UTC (permalink / raw)
To: u-boot
On Thu, 28 Jan 2021 at 15:25, Dario Binacchi <dariobin@libero.it> wrote:
>
> 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 a generic type driver, and it can be useful for debug
> purposes.
>
> Signed-off-by: Dario Binacchi <dariobin@libero.it>
> ---
>
> (no changes since v1)
>
> drivers/pinctrl/pinctrl-single.c | 222 +++++++++++++++++++++++++++++--
> 1 file changed, 213 insertions(+), 9 deletions(-)
>
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v2 12/12] test: pinmux: add test for 'pinctrl-single' driver
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
` (10 preceding siblings ...)
2021-01-28 22:25 ` [PATCH v2 11/12] pinctrl: single: add get_pin_muxing operation Dario Binacchi
@ 2021-01-28 22:25 ` Dario Binacchi
2021-02-04 18:02 ` [PATCH v2 00/12] Add support for pinmux status command on beaglebone Lokesh Vutla
12 siblings, 0 replies; 23+ messages in thread
From: Dario Binacchi @ 2021-01-28 22:25 UTC (permalink / raw)
To: u-boot
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 used on sandbox.
Signed-off-by: Dario Binacchi <dariobin@libero.it>
Reviewed-by: Simon Glass <sjg@chromium.org>
---
Changes in v2:
- Added Simon Glass review tag.
- Added error checking when the 'width' property is missing.
- Fix coding style.
arch/sandbox/dts/test.dts | 72 +++++++++++++++++++++++++
configs/sandbox_defconfig | 1 +
drivers/pinctrl/pinctrl-single.c | 31 +++++++++++
test/dm/pinmux.c | 91 ++++++++++++++++++++++++++++++--
4 files changed, 190 insertions(+), 5 deletions(-)
diff --git a/arch/sandbox/dts/test.dts b/arch/sandbox/dts/test.dts
index f86cd0d3b2..c9b0aad5ed 100644
--- a/arch/sandbox/dts/test.dts
+++ b/arch/sandbox/dts/test.dts
@@ -513,6 +513,9 @@
reg = <0 1>;
compatible = "sandbox,i2c";
clock-frequency = <100000>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinmux_i2c0_pins>;
+
eeprom at 2c {
reg = <0x2c>;
compatible = "i2c-eeprom";
@@ -592,6 +595,8 @@
lcd {
u-boot,dm-pre-reloc;
compatible = "sandbox,lcd-sdl";
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinmux_lcd_pins>;
xres = <1366>;
yres = <768>;
};
@@ -842,6 +847,8 @@
pwm: pwm {
compatible = "sandbox,pwm";
#pwm-cells = <2>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinmux_pwm_pins>;
};
pwm2 {
@@ -913,6 +920,9 @@
reg = <0 1>;
compatible = "sandbox,spi";
cs-gpios = <0>, <0>, <&gpio_a 0>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinmux_spi0_pins>;
+
spi.bin at 0 {
reg = <0>;
compatible = "spansion,m25p16", "jedec,spi-nor";
@@ -1002,6 +1012,8 @@
uart0: serial {
compatible = "sandbox,serial";
u-boot,dm-pre-reloc;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinmux_uart0_pins>;
};
usb_0: usb at 0 {
@@ -1268,6 +1280,66 @@
};
};
+ pinctrl-single-no-width {
+ compatible = "pinctrl-single";
+ reg = <0x0000 0x238>;
+ #pinctrl-cells = <1>;
+ pinctrl-single,function-mask = <0x7f>;
+ };
+
+ pinctrl-single-pins {
+ compatible = "pinctrl-single";
+ reg = <0x0000 0x238>;
+ #pinctrl-cells = <1>;
+ pinctrl-single,register-width = <32>;
+ pinctrl-single,function-mask = <0x7f>;
+
+ pinmux_pwm_pins: pinmux_pwm_pins {
+ pinctrl-single,pins = < 0x48 0x06 >;
+ };
+
+ pinmux_spi0_pins: pinmux_spi0_pins {
+ pinctrl-single,pins = <
+ 0x190 0x0c
+ 0x194 0x0c
+ 0x198 0x23
+ 0x19c 0x0c
+ >;
+ };
+
+ pinmux_uart0_pins: pinmux_uart0_pins {
+ pinctrl-single,pins = <
+ 0x70 0x30
+ 0x74 0x00
+ >;
+ };
+ };
+
+ pinctrl-single-bits {
+ compatible = "pinctrl-single";
+ reg = <0x0000 0x50>;
+ #pinctrl-cells = <2>;
+ pinctrl-single,bit-per-mux;
+ pinctrl-single,register-width = <32>;
+ pinctrl-single,function-mask = <0xf>;
+
+ pinmux_i2c0_pins: pinmux_i2c0_pins {
+ pinctrl-single,bits = <
+ 0x10 0x00002200 0x0000ff00
+ >;
+ };
+
+ pinmux_lcd_pins: pinmux_lcd_pins {
+ pinctrl-single,bits = <
+ 0x40 0x22222200 0xffffff00
+ 0x44 0x22222222 0xffffffff
+ 0x48 0x00000022 0x000000ff
+ 0x48 0x02000000 0x0f000000
+ 0x4c 0x02000022 0x0f0000ff
+ >;
+ };
+ };
+
hwspinlock at 0 {
compatible = "sandbox,hwspinlock";
};
diff --git a/configs/sandbox_defconfig b/configs/sandbox_defconfig
index 86dc603667..1c7d49f073 100644
--- a/configs/sandbox_defconfig
+++ b/configs/sandbox_defconfig
@@ -193,6 +193,7 @@ CONFIG_PHY_SANDBOX=y
CONFIG_PINCTRL=y
CONFIG_PINCONF=y
CONFIG_PINCTRL_SANDBOX=y
+CONFIG_PINCTRL_SINGLE=y
CONFIG_POWER_DOMAIN=y
CONFIG_SANDBOX_POWER_DOMAIN=y
CONFIG_DM_PMIC=y
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index 3ddb637ab7..48bdd0f6f5 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -51,6 +51,9 @@ struct single_func {
* @pin_name: temporary buffer to store the pin name
*/
struct single_priv {
+#if (IS_ENABLED(CONFIG_SANDBOX))
+ u32 *sandbox_regs;
+#endif
unsigned int bits_per_pin;
unsigned int npins;
char pin_name[PINNAME_SIZE];
@@ -87,6 +90,8 @@ struct single_fdt_bits_cfg {
fdt32_t mask;
};
+#if (!IS_ENABLED(CONFIG_SANDBOX))
+
static unsigned int single_read(struct udevice *dev, fdt_addr_t reg)
{
struct single_pdata *pdata = dev_get_plat(dev);
@@ -119,6 +124,24 @@ static void single_write(struct udevice *dev, unsigned int val, fdt_addr_t reg)
}
}
+#else /* CONFIG_SANDBOX */
+
+static unsigned int single_read(struct udevice *dev, fdt_addr_t reg)
+{
+ struct single_priv *priv = dev_get_priv(dev);
+
+ return priv->sandbox_regs[reg];
+}
+
+static void single_write(struct udevice *dev, unsigned int val, fdt_addr_t reg)
+{
+ struct single_priv *priv = dev_get_priv(dev);
+
+ priv->sandbox_regs[reg] = val;
+}
+
+#endif /* CONFIG_SANDBOX */
+
/**
* single_get_pin_by_offset() - get a pin based on the register offset
* @dev: single driver instance
@@ -436,6 +459,14 @@ static int single_probe(struct udevice *dev)
INIT_LIST_HEAD(&priv->functions);
size = pdata->offset + pdata->width / BITS_PER_BYTE;
+ #if (CONFIG_IS_ENABLED(SANDBOX))
+ priv->sandbox_regs =
+ devm_kzalloc(dev, size * sizeof(*priv->sandbox_regs),
+ GFP_KERNEL);
+ if (!priv->sandbox_regs)
+ return -ENOMEM;
+ #endif
+
priv->npins = size / (pdata->width / BITS_PER_BYTE);
if (pdata->bits_per_mux) {
priv->bits_per_pin = fls(pdata->mask);
diff --git a/test/dm/pinmux.c b/test/dm/pinmux.c
index 047184d4bc..265df4ccb9 100644
--- a/test/dm/pinmux.c
+++ b/test/dm/pinmux.c
@@ -9,16 +9,21 @@
#include <dm/test.h>
#include <test/ut.h>
-static int dm_test_pinmux(struct unit_test_state *uts)
-{
- char buf[64];
- struct udevice *dev;
-
+static char buf[64];
#define test_muxing(selector, expected) do { \
ut_assertok(pinctrl_get_pin_muxing(dev, selector, buf, sizeof(buf))); \
ut_asserteq_str(expected, (char *)&buf); \
} while (0)
+#define test_name(selector, expected) do { \
+ ut_assertok(pinctrl_get_pin_name(dev, selector, buf, sizeof(buf))); \
+ ut_asserteq_str(expected, (char *)&buf); \
+} while (0)
+
+static int dm_test_pinmux(struct unit_test_state *uts)
+{
+ struct udevice *dev;
+
ut_assertok(uclass_get_device_by_name(UCLASS_PINCTRL, "pinctrl", &dev));
test_muxing(0, "UART TX.");
test_muxing(1, "UART RX.");
@@ -54,4 +59,80 @@ static int dm_test_pinmux(struct unit_test_state *uts)
return 0;
}
+
DM_TEST(dm_test_pinmux, UT_TESTF_SCAN_FDT);
+
+static int dm_test_pinctrl_single(struct unit_test_state *uts)
+{
+ struct udevice *dev;
+ int ret;
+
+ ret = uclass_get_device_by_name(UCLASS_PINCTRL,
+ "pinctrl-single-no-width", &dev);
+ ut_asserteq(-EINVAL, ret);
+ ut_assertok(uclass_get_device_by_name(UCLASS_PWM, "pwm", &dev));
+ ut_assertok(uclass_get_device_by_name(UCLASS_SERIAL, "serial", &dev));
+ ut_assertok(uclass_get_device_by_name(UCLASS_SPI, "spi at 0", &dev));
+ ut_assertok(uclass_get_device_by_name(UCLASS_PINCTRL,
+ "pinctrl-single-pins", &dev));
+ ut_asserteq(142, pinctrl_get_pins_count(dev));
+ test_name(0, "PIN0");
+ test_name(141, "PIN141");
+ test_name(142, "Error");
+ test_muxing(0, "0x00000000 0x00000000 UNCLAIMED");
+ test_muxing(18, "0x00000048 0x00000006 pinmux_pwm_pins");
+ test_muxing(28, "0x00000070 0x00000030 pinmux_uart0_pins");
+ test_muxing(29, "0x00000074 0x00000000 pinmux_uart0_pins");
+ test_muxing(100, "0x00000190 0x0000000c pinmux_spi0_pins");
+ test_muxing(101, "0x00000194 0x0000000c pinmux_spi0_pins");
+ test_muxing(102, "0x00000198 0x00000023 pinmux_spi0_pins");
+ test_muxing(103, "0x0000019c 0x0000000c pinmux_spi0_pins");
+ ret = pinctrl_get_pin_muxing(dev, 142, buf, sizeof(buf));
+ ut_asserteq(-EINVAL, ret);
+ ut_assertok(uclass_get_device_by_name(UCLASS_I2C, "i2c at 0", &dev));
+ ut_assertok(uclass_get_device_by_name(UCLASS_VIDEO, "lcd", &dev));
+ ut_assertok(uclass_get_device_by_name(UCLASS_PINCTRL,
+ "pinctrl-single-bits", &dev));
+ ut_asserteq(160, pinctrl_get_pins_count(dev));
+ test_name(0, "PIN0");
+ test_name(159, "PIN159");
+ test_name(160, "Error");
+ test_muxing(0, "0x00000000 0x00000000 UNCLAIMED");
+ test_muxing(34, "0x00000010 0x00000200 pinmux_i2c0_pins");
+ test_muxing(35, "0x00000010 0x00002000 pinmux_i2c0_pins");
+ test_muxing(130, "0x00000040 0x00000200 pinmux_lcd_pins");
+ test_muxing(131, "0x00000040 0x00002000 pinmux_lcd_pins");
+ test_muxing(132, "0x00000040 0x00020000 pinmux_lcd_pins");
+ test_muxing(133, "0x00000040 0x00200000 pinmux_lcd_pins");
+ test_muxing(134, "0x00000040 0x02000000 pinmux_lcd_pins");
+ test_muxing(135, "0x00000040 0x20000000 pinmux_lcd_pins");
+ test_muxing(136, "0x00000044 0x00000002 pinmux_lcd_pins");
+ test_muxing(137, "0x00000044 0x00000020 pinmux_lcd_pins");
+ test_muxing(138, "0x00000044 0x00000200 pinmux_lcd_pins");
+ test_muxing(139, "0x00000044 0x00002000 pinmux_lcd_pins");
+ test_muxing(140, "0x00000044 0x00020000 pinmux_lcd_pins");
+ test_muxing(141, "0x00000044 0x00200000 pinmux_lcd_pins");
+ test_muxing(142, "0x00000044 0x02000000 pinmux_lcd_pins");
+ test_muxing(143, "0x00000044 0x20000000 pinmux_lcd_pins");
+ test_muxing(144, "0x00000048 0x00000002 pinmux_lcd_pins");
+ test_muxing(145, "0x00000048 0x00000020 pinmux_lcd_pins");
+ test_muxing(146, "0x00000048 0x00000000 UNCLAIMED");
+ test_muxing(147, "0x00000048 0x00000000 UNCLAIMED");
+ test_muxing(148, "0x00000048 0x00000000 UNCLAIMED");
+ test_muxing(149, "0x00000048 0x00000000 UNCLAIMED");
+ test_muxing(150, "0x00000048 0x02000000 pinmux_lcd_pins");
+ test_muxing(151, "0x00000048 0x00000000 UNCLAIMED");
+ test_muxing(152, "0x0000004c 0x00000002 pinmux_lcd_pins");
+ test_muxing(153, "0x0000004c 0x00000020 pinmux_lcd_pins");
+ test_muxing(154, "0x0000004c 0x00000000 UNCLAIMED");
+ test_muxing(155, "0x0000004c 0x00000000 UNCLAIMED");
+ test_muxing(156, "0x0000004c 0x00000000 UNCLAIMED");
+ test_muxing(157, "0x0000004c 0x00000000 UNCLAIMED");
+ test_muxing(158, "0x0000004c 0x02000000 pinmux_lcd_pins");
+ test_muxing(159, "0x0000004c 0x00000000 UNCLAIMED");
+ ret = pinctrl_get_pin_muxing(dev, 160, buf, sizeof(buf));
+ ut_asserteq(-EINVAL, ret);
+ return 0;
+}
+
+DM_TEST(dm_test_pinctrl_single, UT_TESTF_SCAN_FDT);
--
2.17.1
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v2 00/12] Add support for pinmux status command on beaglebone
2021-01-28 22:24 [PATCH v2 00/12] Add support for pinmux status command on beaglebone Dario Binacchi
` (11 preceding siblings ...)
2021-01-28 22:25 ` [PATCH v2 12/12] test: pinmux: add test for 'pinctrl-single' driver Dario Binacchi
@ 2021-02-04 18:02 ` Lokesh Vutla
2021-02-05 10:54 ` Dario Binacchi
12 siblings, 1 reply; 23+ messages in thread
From: Lokesh Vutla @ 2021-02-04 18:02 UTC (permalink / raw)
To: u-boot
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
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(-)
>
^ permalink raw reply [flat|nested] 23+ messages in thread* [PATCH v2 00/12] Add support for pinmux status command on beaglebone
2021-02-04 18:02 ` [PATCH v2 00/12] Add support for pinmux status command on beaglebone Lokesh Vutla
@ 2021-02-05 10:54 ` Dario Binacchi
2021-02-05 14:17 ` Lokesh Vutla
0 siblings, 1 reply; 23+ messages in thread
From: Dario Binacchi @ 2021-02-05 10:54 UTC (permalink / raw)
To: u-boot
Hi Lokesh,
> Il 04/02/2021 19:02 Lokesh Vutla <lokeshvutla@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(-)
> >
^ permalink raw reply [flat|nested] 23+ messages in thread* [PATCH v2 00/12] Add support for pinmux status command on beaglebone
2021-02-05 10:54 ` Dario Binacchi
@ 2021-02-05 14:17 ` Lokesh Vutla
2021-02-05 18:37 ` Tom Rini
0 siblings, 1 reply; 23+ messages in thread
From: Lokesh Vutla @ 2021-02-05 14:17 UTC (permalink / raw)
To: u-boot
+Tom
On 05/02/21 4:24 pm, Dario Binacchi wrote:
> Hi Lokesh,
>
>> Il 04/02/2021 19:02 Lokesh Vutla <lokeshvutla@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) =========
You are right. I tried these locally and I could not reproduce. But the
gitlab-ci is throwing the same error anytime I push this series.
Simon, Tom,
Any idea why this could happen?
Thanks and regards,
Lokesh
>
>
> 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(-)
>>>
^ permalink raw reply [flat|nested] 23+ messages in thread* [PATCH v2 00/12] Add support for pinmux status command on beaglebone
2021-02-05 14:17 ` Lokesh Vutla
@ 2021-02-05 18:37 ` Tom Rini
0 siblings, 0 replies; 23+ messages in thread
From: Tom Rini @ 2021-02-05 18:37 UTC (permalink / raw)
To: u-boot
On Fri, Feb 05, 2021 at 07:47:19PM +0530, Lokesh Vutla wrote:
> +Tom
>
> On 05/02/21 4:24 pm, Dario Binacchi wrote:
> > Hi Lokesh,
> >
> >> Il 04/02/2021 19:02 Lokesh Vutla <lokeshvutla@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) =========
>
> You are right. I tried these locally and I could not reproduce. But the
> gitlab-ci is throwing the same error anytime I push this series.
>
> Simon, Tom,
> Any idea why this could happen?
Since GitLab/Azure are just running it in Docker, what happens if you
run it locally that way?
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210205/a80e698d/attachment.sig>
^ permalink raw reply [flat|nested] 23+ messages in thread