diff mbox

[Linaro-uefi,v4,54/56] D03/USB: fix ehci interrupt pin number

Message ID 1479544691-59575-55-git-send-email-heyi.guo@linaro.org
State Superseded
Headers show

Commit Message

gary guo Nov. 19, 2016, 8:38 a.m. UTC 
The defination of OHCI and EHCI hardware pins are wrong,
the OHCI pin number is 640, and the EHCI hardware pin number
is 641, correct them.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Kefeng Wang <wangkefeng@huawei.com>
Signed-off-by: Heyi Guo <heyi.guo@linaro.org>
Review-by: Graeme Gregory <graeme.gregory@linaro.org>
---
 Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Mbig.asl | 2 +-
 Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Usb.asl  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Leif Lindholm Nov. 29, 2016, 7:19 p.m. UTC | #1 
On Sat, Nov 19, 2016 at 04:38:09PM +0800, Heyi Guo wrote:
> The defination of OHCI and EHCI hardware pins are wrong,
> the OHCI pin number is 640, and the EHCI hardware pin number
> is 641, correct them.
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Kefeng Wang <wangkefeng@huawei.com>
> Signed-off-by: Heyi Guo <heyi.guo@linaro.org>
> Review-by: Graeme Gregory <graeme.gregory@linaro.org>

Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>

> ---
>  Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Mbig.asl | 2 +-
>  Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Usb.asl  | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Mbig.asl b/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Mbig.asl
> index afd6b47..7265ac8 100644
> --- a/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Mbig.asl
> +++ b/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Mbig.asl
> @@ -23,7 +23,7 @@ Scope(_SB)
>      })
>  
>     Name(_PRS, ResourceTemplate() {
> -       Interrupt(ResourceProducer, Edge, ActiveHigh, Exclusive, 0,,) {0x41, 0x42}
> +       Interrupt(ResourceProducer, Edge, ActiveHigh, Exclusive, 0,,) {640, 641} //OHCI: 640, EHCI 641
>         })
>  
>     Name(_DSD, Package () {
> diff --git a/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Usb.asl b/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Usb.asl
> index 8429a4b..9132965 100644
> --- a/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Usb.asl
> +++ b/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Usb.asl
> @@ -34,7 +34,7 @@ Scope(_SB)
>                          )
>                      Interrupt (ResourceConsumer, Level, ActiveHigh, Exclusive, 0, "\\_SB.MBI0")
>                      {
> -                        0x00000041,
> +                        641, //EHCI
>                      }
>                  })
>                  Return (RBUF) /* \_SB_.USB0._CRS.RBUF */
> -- 
> 1.9.1
>
diff mbox

Patch

diff --git a/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Mbig.asl b/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Mbig.asl
index afd6b47..7265ac8 100644
--- a/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Mbig.asl
+++ b/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Mbig.asl
@@ -23,7 +23,7 @@  Scope(_SB)
     })
 
    Name(_PRS, ResourceTemplate() {
-       Interrupt(ResourceProducer, Edge, ActiveHigh, Exclusive, 0,,) {0x41, 0x42}
+       Interrupt(ResourceProducer, Edge, ActiveHigh, Exclusive, 0,,) {640, 641} //OHCI: 640, EHCI 641
        })
 
    Name(_DSD, Package () {
diff --git a/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Usb.asl b/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Usb.asl
index 8429a4b..9132965 100644
--- a/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Usb.asl
+++ b/Chips/Hisilicon/Hi1610/Hi1610AcpiTables/Dsdt/D03Usb.asl
@@ -34,7 +34,7 @@  Scope(_SB)
                         )
                     Interrupt (ResourceConsumer, Level, ActiveHigh, Exclusive, 0, "\\_SB.MBI0")
                     {
-                        0x00000041,
+                        641, //EHCI
                     }
                 })
                 Return (RBUF) /* \_SB_.USB0._CRS.RBUF */