On Wed, Oct 14, 2015 at 12:16 PM, YD Tseng <ltyu...@gmail.com> wrote:

> Dear all:
>
> This patch adds a new GPIO driver for AMD Promontory chip.  This GPIO 
> controller is enumerated by ACPI and the ACPI compliant hardware ID is 
> AMDF030.
>
> Signed-off-by: YD Tseng <yd_ts...@asmedia.com.tw>

Please fix subject line to
"gpio: driver for AMD Promontory"

Put a line-wrapped concise description in the driver commit
blurb.

> +#define TOTAL_GPIO_PINS 8

Call this PT_TOTAL_GPIO 8 or so.

> +config GPIO_AMDPT
> +       tristate "AMD Promontory GPIO support"
> +       depends on PCI && ACPI

Why does this depend on PCI when it's an ACPI platform device?

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to