[PATCH v2 05/11] ahci: mediatek: add ahci driver

Sam Shih sam.shih at mediatek.com
Wed Aug 19 10:38:36 CEST 2020


Hi Frank,
Thanks your for your patches.
some suggestions for coding style.O

> From: Frank Wunderlich <frank-w at public-files.de>

> diff --git a/drivers/ata/Makefile b/drivers/ata/Makefile
> index a69edb10f7..98fb480700 100644
> --- a/drivers/ata/Makefile
> +++ b/drivers/ata/Makefile
> @@ -19,3 +19,4 @@ obj-$(CONFIG_SATA_SIL) += sata_sil.o
>  obj-$(CONFIG_SANDBOX) += sata_sandbox.o
>  obj-$(CONFIG_AHCI_MVEBU) += ahci_mvebu.o
>  obj-$(CONFIG_SUNXI_AHCI) += ahci_sunxi.o
> +obj-$(CONFIG_MTK_AHCI) += mtk_ahci.o
> diff --git a/drivers/ata/mtk_ahci.c b/drivers/ata/mtk_ahci.c
> new file mode 100644
> index 0000000000..8e09c5f9e6
> --- /dev/null
> +++ b/drivers/ata/mtk_ahci.c
> @@ -0,0 +1,130 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * MTK SATA platform driver
> + *
> + * (C) Copyright 2020
> + *     Mediatek

Please make copyright in the same line.
Copyright (C) 2020 MediaTek Inc.

> + *
> + * Author: Frank Wunderlich <frank-w at public-files.de>
> + * based on https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/ata/ahci_mtk.c

Don't place this line between the Author tags

> + * Author: Ryder Lee <ryder.lee at mediatek.com>
> + */



Thanks,
Best Regards,
Sam Shih



More information about the U-Boot mailing list