2018-07-27 08:44:38

by Srinath Mannam

[permalink] [raw]
Subject: [PATCH v2] serial: 8250_dw: Add ACPI support for uart on Broadcom SoC

Add ACPI identifier HID for UART DW 8250 on Broadcom SoCs
to match the HID passed through ACPI tables to enable
UART controller.

Signed-off-by: Srinath Mannam <[email protected]>
Reviewed-by: Vladimir Olovyannikov <[email protected]>
Tested-by: Vladimir Olovyannikov <[email protected]>
Reviewed-by: Ray Jui <[email protected]>
Reviewed-by: Andy Shevchenko <[email protected]>
---
drivers/tty/serial/8250/8250_dw.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/tty/serial/8250/8250_dw.c b/drivers/tty/serial/8250/8250_dw.c
index aff04f1..40811df 100644
--- a/drivers/tty/serial/8250/8250_dw.c
+++ b/drivers/tty/serial/8250/8250_dw.c
@@ -707,6 +707,7 @@ static const struct acpi_device_id dw8250_acpi_match[] = {
{ "APMC0D08", 0},
{ "AMD0020", 0 },
{ "AMDI0020", 0 },
+ { "BRCM2032", 0 },
{ "HISI0031", 0 },
{ },
};
--
2.7.4



2018-07-28 14:56:09

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH v2] serial: 8250_dw: Add ACPI support for uart on Broadcom SoC

On Fri, Jul 27, 2018 at 02:12:50PM +0530, Srinath Mannam wrote:
> Add ACPI identifier HID for UART DW 8250 on Broadcom SoCs
> to match the HID passed through ACPI tables to enable
> UART controller.
>
> Signed-off-by: Srinath Mannam <[email protected]>
> Reviewed-by: Vladimir Olovyannikov <[email protected]>
> Tested-by: Vladimir Olovyannikov <[email protected]>
> Reviewed-by: Ray Jui <[email protected]>
> Reviewed-by: Andy Shevchenko <[email protected]>
> ---
> drivers/tty/serial/8250/8250_dw.c | 1 +
> 1 file changed, 1 insertion(+)

WHat changed from v1? ALways put that below the --- line.

v3?

thanks,

greg k-h

2018-07-28 15:14:05

by Srinath Mannam

[permalink] [raw]
Subject: Re: [PATCH v2] serial: 8250_dw: Add ACPI support for uart on Broadcom SoC

Hi Greg k-h

Sorry for my mistake, I will send v3 patchset with details of V1 and V2 changes.

Regards,
Srinath.

On Sat, Jul 28, 2018 at 8:24 PM, Greg Kroah-Hartman
<[email protected]> wrote:
> On Fri, Jul 27, 2018 at 02:12:50PM +0530, Srinath Mannam wrote:
>> Add ACPI identifier HID for UART DW 8250 on Broadcom SoCs
>> to match the HID passed through ACPI tables to enable
>> UART controller.
>>
>> Signed-off-by: Srinath Mannam <[email protected]>
>> Reviewed-by: Vladimir Olovyannikov <[email protected]>
>> Tested-by: Vladimir Olovyannikov <[email protected]>
>> Reviewed-by: Ray Jui <[email protected]>
>> Reviewed-by: Andy Shevchenko <[email protected]>
>> ---
>> drivers/tty/serial/8250/8250_dw.c | 1 +
>> 1 file changed, 1 insertion(+)
>
> WHat changed from v1? ALways put that below the --- line.
>
> v3?
>
> thanks,
>
> greg k-h