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

[Joel Stanley]

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?

> +       /*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
>