Hi Tokunori,
On Thu, May 31, 2018 at 10:02:40AM +0900, Tokunori Ikegami wrote:
> diff --git a/arch/mips/bcm47xx/setup.c b/arch/mips/bcm47xx/setup.c
> index 6054d49e608e..8fec219e1160 100644
> --- a/arch/mips/bcm47xx/setup.c
> +++ b/arch/mips/bcm47xx/setup.c
> @@ -212,6 +212,13 @@ static int __init bcm47xx_cpu_fixes(void)
> */
> if (bcm47xx_bus.bcma.bus.chipinfo.id == BCMA_CHIP_ID_BCM4706)
> cpu_wait = NULL;
> +
> + /*
> + * BCM47XX Erratum "R10: PCIe Transactions Periodically Fail"
> + * Enable ExternalSync for sync instruction to take effect
> + */
> + pr_info("ExternalSync has been enabled\n");
> + set_c0_config7(MIPS_CONF7_ES);
Great - this looks better placed than v2, and so long as this erratum
only applies to systems using BCMA this looks good to me.
My only other niggle would be questioning whether we really need the
pr_info() - I'd probably go without it, but it's not a strong opinion so
either way:
Reviewed-by: Paul Burton <paul.burton@xxxxxxxx>
Thanks,
Paul
