[PATCH linux dev-5.15 1/2] ARM: dts: Aspeed: Rainier: Add GPIO hog for USB power control

Eddie James eajames at linux.ibm.com
Wed Dec 8 07:28:04 AEDT 2021


On 12/6/21 16:54, Joel Stanley wrote:
> On Mon, 6 Dec 2021 at 21:32, Eddie James <eajames at linux.ibm.com> wrote:
>> Set the USB power control to always on.
>>
>> Signed-off-by: Eddie James <eajames at linux.ibm.com>
>> ---
>>   arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts | 9 ++++++++-
>>   1 file changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
>> index 866f32cdccea..b2af8ad09d33 100644
>> --- a/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
>> +++ b/arch/arm/boot/dts/aspeed-bmc-ibm-rainier.dts
>> @@ -255,7 +255,7 @@ &gpio0 {
>>          /*L0-L7*/       "","","","","","","","",
>>          /*M0-M7*/       "","","","","","","","",
>>          /*N0-N7*/       "","","","","","","","",
>> -       /*O0-O7*/       "","","","usb-power","","","","",
> Do we want to remove the pin from the gpio-names? Can we leave it
> there as documentation, or does it conflict?


I don't know; The pin isn't documented so I thought it better to remove 
it from the line names. Then below, I named it according to the board 
spec, which somewhat fits with the i2c mux hog. I'm open to whatever we 
think is best here.


Thanks!

Eddie


>
>> +       /*O0-O7*/       "","","","","","","","",
>>          /*P0-P7*/       "","","","","pcieslot-power","","","",
>>          /*Q0-Q7*/       "cfam-reset","","regulator-standby-faulted","","","","","",
>>          /*R0-R7*/       "bmc-tpm-reset","power-chassis-control","power-chassis-good","","","","","",
>> @@ -275,6 +275,13 @@ i2c3_mux_oe_n {
>>                  output-high;
>>                  line-name = "I2C3_MUX_OE_N";
>>          };
>> +
>> +       usb_power_en_n {
>> +               gpio-hog;
>> +               gpios = <ASPEED_GPIO(O, 3) GPIO_ACTIVE_LOW>;
>> +               output-high;
>> +               line-name = "USB_POWER_EN_N";
> This contradicts what you called it above.
>
>> +       };
>>   };
>>
>>   &emmc_controller {
>> --
>> 2.27.0
>>


More information about the openbmc mailing list