diff mbox

[Linaro-uefi,06/28] D02/ACPI: Use HISI0031 HID for uart on Hip05 soc

Message ID 1479122995-50330-7-git-send-email-heyi.guo@linaro.org
State Superseded
Headers show

Commit Message

gary guo Nov. 14, 2016, 11:29 a.m. UTC
The UART on Hip05 soc is not 16550 compatible, use appropriate ACPI ID
for Hisi uart instead of APM one, and delete the wrong comments.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>
---
 Chips/Hisilicon/Pv660/Pv660AcpiTables/Dsdt/Com.asl | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

Comments

Graeme Gregory Nov. 16, 2016, 10:28 a.m. UTC | #1
On Mon, Nov 14, 2016 at 07:29:33PM +0800, Heyi Guo wrote:
> The UART on Hip05 soc is not 16550 compatible, use appropriate ACPI ID
> for Hisi uart instead of APM one, and delete the wrong comments.
> 

Review-by: Graeme Gregory <graeme.gregory@linaro.org>

> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>
> ---
>  Chips/Hisilicon/Pv660/Pv660AcpiTables/Dsdt/Com.asl | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/Chips/Hisilicon/Pv660/Pv660AcpiTables/Dsdt/Com.asl b/Chips/Hisilicon/Pv660/Pv660AcpiTables/Dsdt/Com.asl
> index 43027e4..3bcc5fb 100644
> --- a/Chips/Hisilicon/Pv660/Pv660AcpiTables/Dsdt/Com.asl
> +++ b/Chips/Hisilicon/Pv660/Pv660AcpiTables/Dsdt/Com.asl
> @@ -18,13 +18,12 @@
>  
>  Scope(_SB)
>  {
> -  // UART 8250
>    Device(COM0) {
> -    Name(_HID, "APMC0D08") //Or AMD0020, trick to use dw8250 serial driver
> +    Name(_HID, "HISI0031") //it is not 16550 compatible
>      Name(_CID, "8250dw")
>      Name(_UID, Zero)
>      Name(_CRS, ResourceTemplate() {
> -      Memory32Fixed(ReadWrite, 0x80300000, 0x1000) //0x7FF80000, 0x1000
> +      Memory32Fixed(ReadWrite, 0x80300000, 0x1000)
>        Interrupt(ResourceConsumer, Level, ActiveHigh, Exclusive) { 349 }
>      })
>      Name (_DSD, Package () {
> -- 
> 1.9.1
>
diff mbox

Patch

diff --git a/Chips/Hisilicon/Pv660/Pv660AcpiTables/Dsdt/Com.asl b/Chips/Hisilicon/Pv660/Pv660AcpiTables/Dsdt/Com.asl
index 43027e4..3bcc5fb 100644
--- a/Chips/Hisilicon/Pv660/Pv660AcpiTables/Dsdt/Com.asl
+++ b/Chips/Hisilicon/Pv660/Pv660AcpiTables/Dsdt/Com.asl
@@ -18,13 +18,12 @@ 
 
 Scope(_SB)
 {
-  // UART 8250
   Device(COM0) {
-    Name(_HID, "APMC0D08") //Or AMD0020, trick to use dw8250 serial driver
+    Name(_HID, "HISI0031") //it is not 16550 compatible
     Name(_CID, "8250dw")
     Name(_UID, Zero)
     Name(_CRS, ResourceTemplate() {
-      Memory32Fixed(ReadWrite, 0x80300000, 0x1000) //0x7FF80000, 0x1000
+      Memory32Fixed(ReadWrite, 0x80300000, 0x1000)
       Interrupt(ResourceConsumer, Level, ActiveHigh, Exclusive) { 349 }
     })
     Name (_DSD, Package () {