Hi, On Mon, Mar 09, 2015 at 08:40:10AM +0800, Ken Xue wrote: > tty:8250:add AMD UART support based on 8250_dw. > AMD CZ integrates Designware's UART IP with 32bit width registers. > APD(similar with LPSS) creates UART platform device based on ACPI > device AMD0020, and registers clk for this UART device. What exactly is AMD CZ? Is it a SoC? It would probable be good to explain it in the commit message. Maybe you could just say in your subject something like: "serial: 8250_dw: add support for AMD CZ" And the commit message would say this: "Add ACPI identifier for UART on AMD <what ever CZ means>." > Signed-off-by: Ken Xue <Ken.Xue@xxxxxxx> > --- > 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 06933e8..2ad76ba 100644 > --- a/drivers/tty/serial/8250/8250_dw.c > +++ b/drivers/tty/serial/8250/8250_dw.c > @@ -594,6 +594,7 @@ static const struct acpi_device_id dw8250_acpi_match[] = { > { "80860F0A", 0 }, > { "8086228A", 0 }, > { "APMC0D08", 0}, > + { "AMD0020", 0 }, > { }, > }; > MODULE_DEVICE_TABLE(acpi, dw8250_acpi_match); > -- > 1.9.1 -- heikki -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html