Dear Feng Kan,

In message <[email protected]> you wrote:
> This adds support for the Designware SATA controller.
> 
> Signed-off-by: Feng Kan <[email protected]>
> Signed-off-by: Mark Miesfeld <[email protected]>
> ---
>  drivers/ata/Kconfig    |   10 +
>  drivers/ata/Makefile   |    1 +
>  drivers/ata/sata_dwc.c | 2053 
> ++++++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 2064 insertions(+), 0 deletions(-)
>  create mode 100644 drivers/ata/sata_dwc.c

...
> +
> +MODULE_LICENSE("GPL");
> +MODULE_AUTHOR("Mark Miesfeld <[email protected]>");
> +MODULE_DESCRIPTION("DesignWare Cores SATA controller low lever driver");
------------------------------------------------------------^^^^^

Guess that's a typo?

Hmm... that's 2000+ lines of code in a single file. Maybe this should
be split?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [email protected]
On a clear disk you can seek forever.
_______________________________________________
Linuxppc-dev mailing list
[email protected]
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to