Re: [PATCH linux dev-4.10] drivers/misc: (aspeed-lpc-snoop): Add ast2400 to compat

[Rob Lippert]

I checked the datasheets when I wrote this and ast2400 does not have
the (undocumented) HICRB register bits 14,15 that enables the BMC to
actually respond to the snoop'ed address.

Without setting that bit in the ast2500 the transactions to that I/O
port would timeout on the host side (although the BMC snoop logic
would still see it and log it).
Probably not an issue for x86 systems that don't have any LPC I/O
error handling anyways but LPC timeouts causes issues with POWER
systems since it sets a hardware FIR bit which can cause boot
failures.

-Rob

On Tue, Jul 4, 2017 at 8:45 AM, Patrick Venture <venture@xxxxxxxxxx> wrote:
> This driver can be used on the aspeed ast2400.
>
> Tested: ast2400 on quanta-q71l
>
> Signed-off-by: Patrick Venture <venture@xxxxxxxxxx>
> ---
>  drivers/misc/aspeed-lpc-snoop.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/misc/aspeed-lpc-snoop.c b/drivers/misc/aspeed-lpc-snoop.c
> index 593905565b74..0647cff6280a 100644
> --- a/drivers/misc/aspeed-lpc-snoop.c
> +++ b/drivers/misc/aspeed-lpc-snoop.c
> @@ -241,6 +241,7 @@ static int aspeed_lpc_snoop_remove(struct platform_device *pdev)
>
>  static const struct of_device_id aspeed_lpc_snoop_match[] = {
>         { .compatible = "aspeed,ast2500-lpc-snoop" },
> +       { .compatible = "aspeed,ast2400-lpc-snoop" },
>         { },
>  };
>
> --
> 2.13.2.725.g09c95d1e9-goog
>