* [PATCH 0/1] hw: aspeed_gpio: Fix pin I/O type declarations
@ 2021-09-28 3:24 pdel
2021-09-28 3:24 ` [PATCH 1/1] " pdel
0 siblings, 1 reply; 6+ messages in thread
From: pdel @ 2021-09-28 3:24 UTC (permalink / raw)
Cc: clg, joel, rashmica.g, patrick, qemu-devel, f4bug, Peter Delevoryas
From: Peter Delevoryas <pdel@fb.com>
In the Aspeed chips, the GPIO pins are mostly labeled in groups of 8,
but some of the groups only have 4 elements. Also, most pins have input
and output capabilities, but some are strictly input or strictly output
pins. We have some arrays that describe the I/O capabilities of each pin
for each chip.
A few of the declarations for the AST2500 and AST2600 in aspeed_gpio.c
don't seem to match the datasheet, probably because of confusion over
the association between big-endian uint32 literals (right-to-left) and
the array of group labels (left-to-right). I checked the AST2400 too,
but it was correct and didn't need any changes.
We might want to consider replacing the u32's with u8's and putting them
together into a u32 programmatically, or perhaps implicitly performing a
big-endian to little-endian conversion, something to make it easier to
read.
Peter Delevoryas (1):
hw: aspeed_gpio: Fix pin I/O type declarations
hw/gpio/aspeed_gpio.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
--
2.30.2
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/1] hw: aspeed_gpio: Fix pin I/O type declarations
2021-09-28 3:24 [PATCH 0/1] hw: aspeed_gpio: Fix pin I/O type declarations pdel
@ 2021-09-28 3:24 ` pdel
2021-09-28 10:53 ` Damien Hedde
0 siblings, 1 reply; 6+ messages in thread
From: pdel @ 2021-09-28 3:24 UTC (permalink / raw)
Cc: clg, joel, rashmica.g, patrick, qemu-devel, f4bug, Peter Delevoryas
From: Peter Delevoryas <pdel@fb.com>
Some of the pin declarations in the Aspeed GPIO module were incorrect,
probably because of confusion over which bits in the input and output
uint32_t's correspond to which groups in the label array. Since the
uint32_t literals are in big endian, it's sort of the opposite of what
would be intuitive. The least significant bit in ast2500_set_props[6]
corresponds to GPIOY0, not GPIOAB7.
GPIOxx indicates input and output capabilities, GPIxx indicates only
input, GPOxx indicates only output.
AST2500:
- Previously had GPIW0..GPIW7 and GPIX0..GPIX7, that's correct.
- Previously had GPIOY0..GPIOY3, should have been GPIOY0..GPIOY7.
- Previously had GPIOAB0..GPIOAB3 and GPIAB4..GPIAB7, should only have
been GPIOAB0..GPIOAB3.
AST2600:
- GPIOT0..GPIOT7 should have been GPIT0..GPIT7.
- GPIOU0..GPIOU7 should have been GPIU0..GPIU7.
- GPIW0..GPIW7 should have been GPIOW0..GPIOW7.
- GPIOY0..GPIOY7 and GPIOZ0...GPIOZ7 were disabled.
Fixes: 4b7f956862dc2db4c5c ("hw/gpio: Add basic Aspeed GPIO model for AST2400 and AST2500")
Fixes: 36d737ee82b2972167e ("hw/gpio: Add in AST2600 specific implementation")
Signed-off-by: Peter Delevoryas <pdel@fb.com>
---
hw/gpio/aspeed_gpio.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/gpio/aspeed_gpio.c b/hw/gpio/aspeed_gpio.c
index dfa6d6cb40..33a40a624a 100644
--- a/hw/gpio/aspeed_gpio.c
+++ b/hw/gpio/aspeed_gpio.c
@@ -796,7 +796,7 @@ static const GPIOSetProperties ast2500_set_props[] = {
[3] = {0xffffffff, 0xffffffff, {"M", "N", "O", "P"} },
[4] = {0xffffffff, 0xffffffff, {"Q", "R", "S", "T"} },
[5] = {0xffffffff, 0x0000ffff, {"U", "V", "W", "X"} },
- [6] = {0xffffff0f, 0x0fffff0f, {"Y", "Z", "AA", "AB"} },
+ [6] = {0x0fffffff, 0x0fffffff, {"Y", "Z", "AA", "AB"} },
[7] = {0x000000ff, 0x000000ff, {"AC"} },
};
@@ -805,9 +805,9 @@ static GPIOSetProperties ast2600_3_3v_set_props[] = {
[1] = {0xffffffff, 0xffffffff, {"E", "F", "G", "H"} },
[2] = {0xffffffff, 0xffffffff, {"I", "J", "K", "L"} },
[3] = {0xffffffff, 0xffffffff, {"M", "N", "O", "P"} },
- [4] = {0xffffffff, 0xffffffff, {"Q", "R", "S", "T"} },
- [5] = {0xffffffff, 0x0000ffff, {"U", "V", "W", "X"} },
- [6] = {0xffff0000, 0x0fff0000, {"Y", "Z", "", ""} },
+ [4] = {0xffffffff, 0x00ffffff, {"Q", "R", "S", "T"} },
+ [5] = {0xffffffff, 0xffffff00, {"U", "V", "W", "X"} },
+ [6] = {0x0000ffff, 0x0000ffff, {"Y", "Z"} },
};
static GPIOSetProperties ast2600_1_8v_set_props[] = {
--
2.30.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] hw: aspeed_gpio: Fix pin I/O type declarations
2021-09-28 3:24 ` [PATCH 1/1] " pdel
@ 2021-09-28 10:53 ` Damien Hedde
2021-09-30 0:45 ` Peter Delevoryas
0 siblings, 1 reply; 6+ messages in thread
From: Damien Hedde @ 2021-09-28 10:53 UTC (permalink / raw)
To: pdel; +Cc: qemu-devel, f4bug, patrick, clg, rashmica.g, joel
On 9/28/21 05:24, pdel@fb.com wrote:
> From: Peter Delevoryas <pdel@fb.com>
>
> Some of the pin declarations in the Aspeed GPIO module were incorrect,
> probably because of confusion over which bits in the input and output
> uint32_t's correspond to which groups in the label array. Since the
> uint32_t literals are in big endian, it's sort of the opposite of what
> would be intuitive. The least significant bit in ast2500_set_props[6]
> corresponds to GPIOY0, not GPIOAB7.
>
> GPIOxx indicates input and output capabilities, GPIxx indicates only
> input, GPOxx indicates only output.
>
> AST2500:
> - Previously had GPIW0..GPIW7 and GPIX0..GPIX7, that's correct.
> - Previously had GPIOY0..GPIOY3, should have been GPIOY0..GPIOY7.
> - Previously had GPIOAB0..GPIOAB3 and GPIAB4..GPIAB7, should only have
> been GPIOAB0..GPIOAB3.
>
> AST2600:
> - GPIOT0..GPIOT7 should have been GPIT0..GPIT7.
> - GPIOU0..GPIOU7 should have been GPIU0..GPIU7.
> - GPIW0..GPIW7 should have been GPIOW0..GPIOW7.
> - GPIOY0..GPIOY7 and GPIOZ0...GPIOZ7 were disabled.
>
> Fixes: 4b7f956862dc2db4c5c ("hw/gpio: Add basic Aspeed GPIO model for AST2400 and AST2500")
> Fixes: 36d737ee82b2972167e ("hw/gpio: Add in AST2600 specific implementation")
> Signed-off-by: Peter Delevoryas <pdel@fb.com>
Reviewed-by: Damien Hedde <damien.hedde@greensocs.com>
> ---
> hw/gpio/aspeed_gpio.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/hw/gpio/aspeed_gpio.c b/hw/gpio/aspeed_gpio.c
> index dfa6d6cb40..33a40a624a 100644
> --- a/hw/gpio/aspeed_gpio.c
> +++ b/hw/gpio/aspeed_gpio.c
> @@ -796,7 +796,7 @@ static const GPIOSetProperties ast2500_set_props[] = {
> [3] = {0xffffffff, 0xffffffff, {"M", "N", "O", "P"} },
> [4] = {0xffffffff, 0xffffffff, {"Q", "R", "S", "T"} },
> [5] = {0xffffffff, 0x0000ffff, {"U", "V", "W", "X"} },
> - [6] = {0xffffff0f, 0x0fffff0f, {"Y", "Z", "AA", "AB"} },
> + [6] = {0x0fffffff, 0x0fffffff, {"Y", "Z", "AA", "AB"} },
> [7] = {0x000000ff, 0x000000ff, {"AC"} },
> };
>
> @@ -805,9 +805,9 @@ static GPIOSetProperties ast2600_3_3v_set_props[] = {
> [1] = {0xffffffff, 0xffffffff, {"E", "F", "G", "H"} },
> [2] = {0xffffffff, 0xffffffff, {"I", "J", "K", "L"} },
> [3] = {0xffffffff, 0xffffffff, {"M", "N", "O", "P"} },
> - [4] = {0xffffffff, 0xffffffff, {"Q", "R", "S", "T"} },
> - [5] = {0xffffffff, 0x0000ffff, {"U", "V", "W", "X"} },
> - [6] = {0xffff0000, 0x0fff0000, {"Y", "Z", "", ""} },
> + [4] = {0xffffffff, 0x00ffffff, {"Q", "R", "S", "T"} },
> + [5] = {0xffffffff, 0xffffff00, {"U", "V", "W", "X"} },
> + [6] = {0x0000ffff, 0x0000ffff, {"Y", "Z"} },
> };
>
> static GPIOSetProperties ast2600_1_8v_set_props[] = {
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] hw: aspeed_gpio: Fix pin I/O type declarations
2021-09-28 10:53 ` Damien Hedde
@ 2021-09-30 0:45 ` Peter Delevoryas
2021-09-30 12:05 ` Rashmica Gupta
0 siblings, 1 reply; 6+ messages in thread
From: Peter Delevoryas @ 2021-09-30 0:45 UTC (permalink / raw)
Cc: Cédric Le Goater, Joel Stanley, rashmica.g, patrick,
Cameron Esfahani via, Philippe Mathieu-Daudé,
Dan Zhang, damien.hedde
> On Sep 28, 2021, at 3:53 AM, Damien Hedde <damien.hedde@greensocs.com> wrote:
>
>
>
> On 9/28/21 05:24, pdel@fb.com wrote:
>> From: Peter Delevoryas <pdel@fb.com>
>> Some of the pin declarations in the Aspeed GPIO module were incorrect,
>> probably because of confusion over which bits in the input and output
>> uint32_t's correspond to which groups in the label array. Since the
>> uint32_t literals are in big endian, it's sort of the opposite of what
>> would be intuitive. The least significant bit in ast2500_set_props[6]
>> corresponds to GPIOY0, not GPIOAB7.
>> GPIOxx indicates input and output capabilities, GPIxx indicates only
>> input, GPOxx indicates only output.
>> AST2500:
>> - Previously had GPIW0..GPIW7 and GPIX0..GPIX7, that's correct.
>> - Previously had GPIOY0..GPIOY3, should have been GPIOY0..GPIOY7.
>> - Previously had GPIOAB0..GPIOAB3 and GPIAB4..GPIAB7, should only have
>> been GPIOAB0..GPIOAB3.
>> AST2600:
>> - GPIOT0..GPIOT7 should have been GPIT0..GPIT7.
>> - GPIOU0..GPIOU7 should have been GPIU0..GPIU7.
>> - GPIW0..GPIW7 should have been GPIOW0..GPIOW7.
>> - GPIOY0..GPIOY7 and GPIOZ0...GPIOZ7 were disabled.
>> Fixes: 4b7f956862dc2db4c5c ("hw/gpio: Add basic Aspeed GPIO model for AST2400 and AST2500")
>> Fixes: 36d737ee82b2972167e ("hw/gpio: Add in AST2600 specific implementation")
>> Signed-off-by: Peter Delevoryas <pdel@fb.com>
>
> Reviewed-by: Damien Hedde <damien.hedde@greensocs.com>
cc’ing Dan
>
>> ---
>> hw/gpio/aspeed_gpio.c | 8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>> diff --git a/hw/gpio/aspeed_gpio.c b/hw/gpio/aspeed_gpio.c
>> index dfa6d6cb40..33a40a624a 100644
>> --- a/hw/gpio/aspeed_gpio.c
>> +++ b/hw/gpio/aspeed_gpio.c
>> @@ -796,7 +796,7 @@ static const GPIOSetProperties ast2500_set_props[] = {
>> [3] = {0xffffffff, 0xffffffff, {"M", "N", "O", "P"} },
>> [4] = {0xffffffff, 0xffffffff, {"Q", "R", "S", "T"} },
>> [5] = {0xffffffff, 0x0000ffff, {"U", "V", "W", "X"} },
>> - [6] = {0xffffff0f, 0x0fffff0f, {"Y", "Z", "AA", "AB"} },
>> + [6] = {0x0fffffff, 0x0fffffff, {"Y", "Z", "AA", "AB"} },
>> [7] = {0x000000ff, 0x000000ff, {"AC"} },
>> };
>> @@ -805,9 +805,9 @@ static GPIOSetProperties ast2600_3_3v_set_props[] = {
>> [1] = {0xffffffff, 0xffffffff, {"E", "F", "G", "H"} },
>> [2] = {0xffffffff, 0xffffffff, {"I", "J", "K", "L"} },
>> [3] = {0xffffffff, 0xffffffff, {"M", "N", "O", "P"} },
>> - [4] = {0xffffffff, 0xffffffff, {"Q", "R", "S", "T"} },
>> - [5] = {0xffffffff, 0x0000ffff, {"U", "V", "W", "X"} },
>> - [6] = {0xffff0000, 0x0fff0000, {"Y", "Z", "", ""} },
>> + [4] = {0xffffffff, 0x00ffffff, {"Q", "R", "S", "T"} },
>> + [5] = {0xffffffff, 0xffffff00, {"U", "V", "W", "X"} },
>> + [6] = {0x0000ffff, 0x0000ffff, {"Y", "Z"} },
>> };
>> static GPIOSetProperties ast2600_1_8v_set_props[] = {
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] hw: aspeed_gpio: Fix pin I/O type declarations
2021-09-30 0:45 ` Peter Delevoryas
@ 2021-09-30 12:05 ` Rashmica Gupta
2021-09-30 14:09 ` Peter Delevoryas
0 siblings, 1 reply; 6+ messages in thread
From: Rashmica Gupta @ 2021-09-30 12:05 UTC (permalink / raw)
To: Peter Delevoryas
Cc: damien.hedde, Cameron Esfahani via, Philippe Mathieu-Daudé,
patrick, Cédric Le Goater, Dan Zhang, Joel Stanley
On Thu, 2021-09-30 at 00:45 +0000, Peter Delevoryas wrote:
>
> > On Sep 28, 2021, at 3:53 AM, Damien Hedde
> > <damien.hedde@greensocs.com> wrote:
> >
> >
> >
> > On 9/28/21 05:24, pdel@fb.com wrote:
> > > From: Peter Delevoryas <pdel@fb.com>
> > > Some of the pin declarations in the Aspeed GPIO module were
> > > incorrect,
> > > probably because of confusion over which bits in the input and
> > > output
> > > uint32_t's correspond to which groups in the label array. Since
> > > the
> > > uint32_t literals are in big endian, it's sort of the opposite of
> > > what
> > > would be intuitive. The least significant bit in
> > > ast2500_set_props[6]
> > > corresponds to GPIOY0, not GPIOAB7.
Looks like I didn't think about endianness! I remember there was
conflicting information about which groups of GPIOs were input only -
the input/output table says groups W and X for ast2600 while the info
in direction registers says T and U. I don't recall why I went with the
former over the latter but the latter seems to be correct as this is
what is in the kernel driver.
> > > GPIOxx indicates input and output capabilities, GPIxx indicates
> > > only
> > > input, GPOxx indicates only output.
> > > AST2500:
> > > - Previously had GPIW0..GPIW7 and GPIX0..GPIX7, that's correct.
> > > - Previously had GPIOY0..GPIOY3, should have been GPIOY0..GPIOY7.
> > > - Previously had GPIOAB0..GPIOAB3 and GPIAB4..GPIAB7, should only
> > > have
> > > been GPIOAB0..GPIOAB3.
> > > AST2600:
> > > - GPIOT0..GPIOT7 should have been GPIT0..GPIT7.
> > > - GPIOU0..GPIOU7 should have been GPIU0..GPIU7.
> > > - GPIW0..GPIW7 should have been GPIOW0..GPIOW7.
> > > - GPIOY0..GPIOY7 and GPIOZ0...GPIOZ7 were disabled.
> > > Fixes: 4b7f956862dc2db4c5c ("hw/gpio: Add basic Aspeed GPIO model
> > > for AST2400 and AST2500")
> > > Fixes: 36d737ee82b2972167e ("hw/gpio: Add in AST2600 specific
> > > implementation")
> > > Signed-off-by: Peter Delevoryas <pdel@fb.com>
> >
> > Reviewed-by: Damien Hedde <damien.hedde@greensocs.com>
>
Reviewed-by: Rashmica Gupta <rashmica.g@gmail.com>
> cc’ing Dan
>
> >
> > > ---
> > > hw/gpio/aspeed_gpio.c | 8 ++++----
> > > 1 file changed, 4 insertions(+), 4 deletions(-)
> > > diff --git a/hw/gpio/aspeed_gpio.c b/hw/gpio/aspeed_gpio.c
> > > index dfa6d6cb40..33a40a624a 100644
> > > --- a/hw/gpio/aspeed_gpio.c
> > > +++ b/hw/gpio/aspeed_gpio.c
> > > @@ -796,7 +796,7 @@ static const GPIOSetProperties
> > > ast2500_set_props[] = {
> > > [3] = {0xffffffff, 0xffffffff, {"M", "N", "O", "P"} },
> > > [4] = {0xffffffff, 0xffffffff, {"Q", "R", "S", "T"} },
> > > [5] = {0xffffffff, 0x0000ffff, {"U", "V", "W", "X"} },
> > > - [6] = {0xffffff0f, 0x0fffff0f, {"Y", "Z", "AA", "AB"} },
> > > + [6] = {0x0fffffff, 0x0fffffff, {"Y", "Z", "AA", "AB"} },
> > > [7] = {0x000000ff, 0x000000ff, {"AC"} },
> > > };
> > > @@ -805,9 +805,9 @@ static GPIOSetProperties
> > > ast2600_3_3v_set_props[] = {
> > > [1] = {0xffffffff, 0xffffffff, {"E", "F", "G", "H"} },
> > > [2] = {0xffffffff, 0xffffffff, {"I", "J", "K", "L"} },
> > > [3] = {0xffffffff, 0xffffffff, {"M", "N", "O", "P"} },
> > > - [4] = {0xffffffff, 0xffffffff, {"Q", "R", "S", "T"} },
> > > - [5] = {0xffffffff, 0x0000ffff, {"U", "V", "W", "X"} },
> > > - [6] = {0xffff0000, 0x0fff0000, {"Y", "Z", "", ""} },
> > > + [4] = {0xffffffff, 0x00ffffff, {"Q", "R", "S", "T"} },
> > > + [5] = {0xffffffff, 0xffffff00, {"U", "V", "W", "X"} },
> > > + [6] = {0x0000ffff, 0x0000ffff, {"Y", "Z"} },
> > > };
> > > static GPIOSetProperties ast2600_1_8v_set_props[] = {
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] hw: aspeed_gpio: Fix pin I/O type declarations
2021-09-30 12:05 ` Rashmica Gupta
@ 2021-09-30 14:09 ` Peter Delevoryas
0 siblings, 0 replies; 6+ messages in thread
From: Peter Delevoryas @ 2021-09-30 14:09 UTC (permalink / raw)
Cc: Cédric Le Goater, Joel Stanley, patrick,
Cameron Esfahani via, Philippe Mathieu-Daudé,
Dan Zhang, damien.hedde, rashmica.g
> On Sep 30, 2021, at 5:05 AM, Rashmica Gupta <rashmica.g@gmail.com> wrote:
>
> On Thu, 2021-09-30 at 00:45 +0000, Peter Delevoryas wrote:
>>
>>> On Sep 28, 2021, at 3:53 AM, Damien Hedde
>>> <damien.hedde@greensocs.com> wrote:
>>>
>>>
>>>
>>> On 9/28/21 05:24, pdel@fb.com wrote:
>>>> From: Peter Delevoryas <pdel@fb.com>
>>>> Some of the pin declarations in the Aspeed GPIO module were
>>>> incorrect,
>>>> probably because of confusion over which bits in the input and
>>>> output
>>>> uint32_t's correspond to which groups in the label array. Since
>>>> the
>>>> uint32_t literals are in big endian, it's sort of the opposite of
>>>> what
>>>> would be intuitive. The least significant bit in
>>>> ast2500_set_props[6]
>>>> corresponds to GPIOY0, not GPIOAB7.
>
> Looks like I didn't think about endianness! I remember there was
> conflicting information about which groups of GPIOs were input only -
> the input/output table says groups W and X for ast2600 while the info
> in direction registers says T and U. I don't recall why I went with the
> former over the latter but the latter seems to be correct as this is
> what is in the kernel driver.
Oh I see, thanks for checking.
>
>>>> GPIOxx indicates input and output capabilities, GPIxx indicates
>>>> only
>>>> input, GPOxx indicates only output.
>>>> AST2500:
>>>> - Previously had GPIW0..GPIW7 and GPIX0..GPIX7, that's correct.
>>>> - Previously had GPIOY0..GPIOY3, should have been GPIOY0..GPIOY7.
>>>> - Previously had GPIOAB0..GPIOAB3 and GPIAB4..GPIAB7, should only
>>>> have
>>>> been GPIOAB0..GPIOAB3.
>>>> AST2600:
>>>> - GPIOT0..GPIOT7 should have been GPIT0..GPIT7.
>>>> - GPIOU0..GPIOU7 should have been GPIU0..GPIU7.
>>>> - GPIW0..GPIW7 should have been GPIOW0..GPIOW7.
>>>> - GPIOY0..GPIOY7 and GPIOZ0...GPIOZ7 were disabled.
>>>> Fixes: 4b7f956862dc2db4c5c ("hw/gpio: Add basic Aspeed GPIO model
>>>> for AST2400 and AST2500")
>>>> Fixes: 36d737ee82b2972167e ("hw/gpio: Add in AST2600 specific
>>>> implementation")
>>>> Signed-off-by: Peter Delevoryas <pdel@fb.com>
>>>
>>> Reviewed-by: Damien Hedde <damien.hedde@greensocs.com>
>>
>
> Reviewed-by: Rashmica Gupta <rashmica.g@gmail.com>
Thanks Rashmica!
Peter
>
>> cc’ing Dan
>>
>>>
>>>> ---
>>>> hw/gpio/aspeed_gpio.c | 8 ++++----
>>>> 1 file changed, 4 insertions(+), 4 deletions(-)
>>>> diff --git a/hw/gpio/aspeed_gpio.c b/hw/gpio/aspeed_gpio.c
>>>> index dfa6d6cb40..33a40a624a 100644
>>>> --- a/hw/gpio/aspeed_gpio.c
>>>> +++ b/hw/gpio/aspeed_gpio.c
>>>> @@ -796,7 +796,7 @@ static const GPIOSetProperties
>>>> ast2500_set_props[] = {
>>>> [3] = {0xffffffff, 0xffffffff, {"M", "N", "O", "P"} },
>>>> [4] = {0xffffffff, 0xffffffff, {"Q", "R", "S", "T"} },
>>>> [5] = {0xffffffff, 0x0000ffff, {"U", "V", "W", "X"} },
>>>> - [6] = {0xffffff0f, 0x0fffff0f, {"Y", "Z", "AA", "AB"} },
>>>> + [6] = {0x0fffffff, 0x0fffffff, {"Y", "Z", "AA", "AB"} },
>>>> [7] = {0x000000ff, 0x000000ff, {"AC"} },
>>>> };
>>>> @@ -805,9 +805,9 @@ static GPIOSetProperties
>>>> ast2600_3_3v_set_props[] = {
>>>> [1] = {0xffffffff, 0xffffffff, {"E", "F", "G", "H"} },
>>>> [2] = {0xffffffff, 0xffffffff, {"I", "J", "K", "L"} },
>>>> [3] = {0xffffffff, 0xffffffff, {"M", "N", "O", "P"} },
>>>> - [4] = {0xffffffff, 0xffffffff, {"Q", "R", "S", "T"} },
>>>> - [5] = {0xffffffff, 0x0000ffff, {"U", "V", "W", "X"} },
>>>> - [6] = {0xffff0000, 0x0fff0000, {"Y", "Z", "", ""} },
>>>> + [4] = {0xffffffff, 0x00ffffff, {"Q", "R", "S", "T"} },
>>>> + [5] = {0xffffffff, 0xffffff00, {"U", "V", "W", "X"} },
>>>> + [6] = {0x0000ffff, 0x0000ffff, {"Y", "Z"} },
>>>> };
>>>> static GPIOSetProperties ast2600_1_8v_set_props[] = {
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-09-30 14:12 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-28 3:24 [PATCH 0/1] hw: aspeed_gpio: Fix pin I/O type declarations pdel
2021-09-28 3:24 ` [PATCH 1/1] " pdel
2021-09-28 10:53 ` Damien Hedde
2021-09-30 0:45 ` Peter Delevoryas
2021-09-30 12:05 ` Rashmica Gupta
2021-09-30 14:09 ` Peter Delevoryas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).