On Tue, May 24, 2016 at 05:32:10PM +0800, Shawn Lin wrote: > Without supporting clock PM capable, if we want to disable > clkpm, we don't need this extra check as it already be zero for > the enable argument. And it's the same for enabling clkpm here. > So let's remove this check. > > Signed-off-by: Shawn Lin <shawn.lin@xxxxxxxxxxxxxx> Applied to pci/aspm for v4.8, thanks, Shawn. > --- > > drivers/pci/pcie/aspm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/pcie/aspm.c b/drivers/pci/pcie/aspm.c > index 2dfe7fd..0ec649d 100644 > --- a/drivers/pci/pcie/aspm.c > +++ b/drivers/pci/pcie/aspm.c > @@ -139,7 +139,7 @@ static void pcie_set_clkpm_nocheck(struct pcie_link_state *link, int enable) > static void pcie_set_clkpm(struct pcie_link_state *link, int enable) > { > /* Don't enable Clock PM if the link is not Clock PM capable */ > - if (!link->clkpm_capable && enable) > + if (!link->clkpm_capable) > enable = 0; > /* Need nothing if the specified equals to current state */ > if (link->clkpm_enabled == enable) > -- > 2.3.7 > > > -- > 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 -- 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