Dear Jason Cooper, On Tue, 17 Mar 2015 15:15:01 +0000, Jason Cooper wrote: > > diff --git a/drivers/pci/host/pci-mvebu.c b/drivers/pci/host/pci-mvebu.c > > index 1309cfb..1ab8635 100644 > > --- a/drivers/pci/host/pci-mvebu.c > > +++ b/drivers/pci/host/pci-mvebu.c > > @@ -129,6 +129,7 @@ struct mvebu_pcie_port { > > size_t memwin_size; > > phys_addr_t iowin_base; > > size_t iowin_size; > > + u32 saved_pcie_stat; > > nit: It's already in the pcie port struct, why not just call it saved_state? Well the "stat" does not mean "state", but refers to the register being saved in this field. The register is called: PCIE_STAT_OFF. Hence the name saved_pcie_stat. But I'm fine with changing this to some other name, I really don't care :) > Either way, > > Acked-by: Jason Cooper <jason@xxxxxxxxxxxxxx> Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html