2010-11-26 10:57:39

by Samuel Ortiz

[permalink] [raw]
Subject: Re: [PATCH 1/4] MFD: add cs5535-mfd driver for CS5535/CS5536 support

Hi Andres,

On Sat, Oct 23, 2010 at 12:41:02AM -0700, Andres Salomon wrote:
>
> Add an MFD driver to handle the ISA device on CS5535 and CS5536 southbridges.
> This ISA bridge is actually multiple devices; GPIOs, MFGPTs, etc.
The patch looks good, and I applied it.
I just have one minor nitpick though, that you may want to address with a
further patch:

> +config MFD_CS5535
> + tristate "Support for CS5535 and CS5536 southbridge core functions"
An AMD Geode mention should appear here. It might not be obvious to everyone
where those devices can be found.

Cheers,
Samuel.

--
Intel Open Source Technology Centre
http://oss.intel.com/