On Thu, Mar 26, 2020 at 12:46 AM Andy Shevchenko
wrote:
>
> USB 3 host controller may be described in ACPI to allow users alter
> the properties or other features. Describe it for Intel Tangier SoC.
>
> Signed-off-by: Andy Shevchenko
> ---
> .../asm/arch-tangier/acpi/southcluster.asl| 38 +++
> 1 file changed, 38 insertions(+)
>
> diff --git a/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl
> b/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl
> index 6ccdc25136..953780a936 100644
> --- a/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl
> +++ b/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl
> @@ -321,6 +321,44 @@ Device (PCI0)
> }
> }
>
> +Device (DWC3)
> +{
> +Name (_ADR, 0x0011)
> +Name (_DEP, Package ()
> +{
> +^IPC1.PMIC
> +})
> +
> +Method (_STA, 0, NotSerialized)
> +{
> +Return (STA_VISIBLE)
> +}
> +
> +Device (RHUB)
> +{
> +Name (_ADR, Zero)
> +
> +// GPLD: Generate Port Location Data (PLD)
nits: please use /* */
> +Method (GPLD, 1, Serialized) {
> +Name (PCKG, Package () {
> +Buffer (0x10) {}
> +})
> +
> +// REV: Revision 0x02 for ACPI 5.0
> +CreateField (DerefOf (Index (PCKG, Zero)), Zero, 0x07, REV)
> +Store (0x02, REV)
> +
> +// VISI: Port visibility to user per port
> +CreateField (DerefOf (Index (PCKG, Zero)), 0x40, One, VISI)
> +Store (Arg0, VISI)
> +Return (PCKG)
> +}
> +
> +Device (HS01) { Name (_ADR, 1) }
> +Device (SS01) { Name (_ADR, 2) }
> +}
> +}
> +
> Device (PWM0)
> {
> Name (_ADR, 0x0017)
Otherwise, looks good to me
Reviewed-by: Bin Meng