Hi Ilpo, On Mon, Sep 18, 2023 at 03:17:41PM +0300, Ilpo Järvinen wrote: > > > > + */ > > > > + val = cap[1] & PCI_EXP_LNKCTL_ASPMC; > > Given this line, it just felt pretty obvious because why would the code > & PCI_EXP_LNKCTL_* with some random register (value) that isn't LNKCTL :-). That's a fair point :) Okay I can drop the comment in the next version.