From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH] serial: 8250_dw: Add ACPI support for uart on Broadcom SoC Date: Fri, 27 Jul 2018 11:34:11 +0300 Message-ID: <3e17e519a59e2a23669be054a44ded9ea90de088.camel@linux.intel.com> References: <1532664066-30056-1-git-send-email-srinath.mannam@broadcom.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1532664066-30056-1-git-send-email-srinath.mannam@broadcom.com> Sender: linux-kernel-owner@vger.kernel.org To: Srinath Mannam , Greg Kroah-Hartman , Jiri Slaby , vikram.prakash@broadcom.com, Ray Jui , Scott Branden Cc: linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org, bcm-kernel-feedback-list@broadcom.com List-Id: linux-serial@vger.kernel.org On Fri, 2018-07-27 at 09:31 +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 > Reviewed-by: Vladimir Olovyannikov > > Tested-by: Vladimir Olovyannikov > Reviewed-by: Ray Jui One comment below, after addressing it, Reviewed-by: Andy Shevchenko > --- > 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..7ea94ec 100644 > --- a/drivers/tty/serial/8250/8250_dw.c > +++ b/drivers/tty/serial/8250/8250_dw.c > @@ -708,6 +708,7 @@ static const struct acpi_device_id > dw8250_acpi_match[] = { > { "AMD0020", 0 }, > { "AMDI0020", 0 }, > { "HISI0031", 0 }, > + { "BRCM2032", 0 }, Let's keep this in order (yes, I know about INT* ones above, but that is not related to this patch right now) > { }, > }; > MODULE_DEVICE_TABLE(acpi, dw8250_acpi_match); -- Andy Shevchenko Intel Finland Oy