Re: [PATCH V2] tty:8250:add AMD UART support based on 8250_dw

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux