* [net-next] pci_regs.h: Add PCI bus link speed and width defines
@ 2013-12-28 12:36 Jeff Kirsher
2014-01-03 0:21 ` David Miller
0 siblings, 1 reply; 10+ messages in thread
From: Jeff Kirsher @ 2013-12-28 12:36 UTC (permalink / raw)
To: davem; +Cc: Jeff Kirsher, netdev, gospo, sassmann, linux-kernel
Add missing PCI bus link speed 8.0 GT/s and bus link widths of
x1, x2, x4 and x8.
CC: <linux-kernel@vger.kernel.org>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
---
include/uapi/linux/pci_regs.h | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h
index 4a98e85..c870c2a 100644
--- a/include/uapi/linux/pci_regs.h
+++ b/include/uapi/linux/pci_regs.h
@@ -489,7 +489,12 @@
#define PCI_EXP_LNKSTA_CLS 0x000f /* Current Link Speed */
#define PCI_EXP_LNKSTA_CLS_2_5GB 0x0001 /* Current Link Speed 2.5GT/s */
#define PCI_EXP_LNKSTA_CLS_5_0GB 0x0002 /* Current Link Speed 5.0GT/s */
+#define PCI_EXP_LNKSTA_CLS_8_0GB 0x0003 /* Current Link Speed 8.0GT/s */
#define PCI_EXP_LNKSTA_NLW 0x03f0 /* Negotiated Link Width */
+#define PCI_EXP_LNKSTA_NLW_X1 0x0010 /* Current Link Width x1 */
+#define PCI_EXP_LNKSTA_NLW_X2 0x0020 /* Current Link Width x2 */
+#define PCI_EXP_LNKSTA_NLW_X4 0x0040 /* Current Link Width x4 */
+#define PCI_EXP_LNKSTA_NLW_X8 0x0080 /* Current Link Width x8 */
#define PCI_EXP_LNKSTA_NLW_SHIFT 4 /* start of NLW mask in link status */
#define PCI_EXP_LNKSTA_LT 0x0800 /* Link Training */
#define PCI_EXP_LNKSTA_SLC 0x1000 /* Slot Clock Configuration */
--
1.8.3.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines
2013-12-28 12:36 [net-next] pci_regs.h: Add PCI bus link speed and width defines Jeff Kirsher
@ 2014-01-03 0:21 ` David Miller
2014-01-03 21:53 ` Jesse Brandeburg
0 siblings, 1 reply; 10+ messages in thread
From: David Miller @ 2014-01-03 0:21 UTC (permalink / raw)
To: jeffrey.t.kirsher; +Cc: netdev, gospo, sassmann, linux-kernel
From: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Date: Sat, 28 Dec 2013 04:36:13 -0800
> Add missing PCI bus link speed 8.0 GT/s and bus link widths of
> x1, x2, x4 and x8.
>
> CC: <linux-kernel@vger.kernel.org>
> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Can a PCI person please ACK this? This is a prerequisite for some
networking driver changes Jeff would like to push to me.
Thanks.
> ---
> include/uapi/linux/pci_regs.h | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h
> index 4a98e85..c870c2a 100644
> --- a/include/uapi/linux/pci_regs.h
> +++ b/include/uapi/linux/pci_regs.h
> @@ -489,7 +489,12 @@
> #define PCI_EXP_LNKSTA_CLS 0x000f /* Current Link Speed */
> #define PCI_EXP_LNKSTA_CLS_2_5GB 0x0001 /* Current Link Speed 2.5GT/s */
> #define PCI_EXP_LNKSTA_CLS_5_0GB 0x0002 /* Current Link Speed 5.0GT/s */
> +#define PCI_EXP_LNKSTA_CLS_8_0GB 0x0003 /* Current Link Speed 8.0GT/s */
> #define PCI_EXP_LNKSTA_NLW 0x03f0 /* Negotiated Link Width */
> +#define PCI_EXP_LNKSTA_NLW_X1 0x0010 /* Current Link Width x1 */
> +#define PCI_EXP_LNKSTA_NLW_X2 0x0020 /* Current Link Width x2 */
> +#define PCI_EXP_LNKSTA_NLW_X4 0x0040 /* Current Link Width x4 */
> +#define PCI_EXP_LNKSTA_NLW_X8 0x0080 /* Current Link Width x8 */
> #define PCI_EXP_LNKSTA_NLW_SHIFT 4 /* start of NLW mask in link status */
> #define PCI_EXP_LNKSTA_LT 0x0800 /* Link Training */
> #define PCI_EXP_LNKSTA_SLC 0x1000 /* Slot Clock Configuration */
> --
> 1.8.3.1
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines
2014-01-03 0:21 ` David Miller
@ 2014-01-03 21:53 ` Jesse Brandeburg
2014-01-03 22:15 ` Bjorn Helgaas
0 siblings, 1 reply; 10+ messages in thread
From: Jesse Brandeburg @ 2014-01-03 21:53 UTC (permalink / raw)
To: David Miller, linux-pci
Cc: Jeff Kirsher, NetDEV list, gospo, sassmann, linux-kernel, Bjorn Helgaas
+linux-pci
<and see note below>
On Thu, Jan 2, 2014 at 4:21 PM, David Miller <davem@davemloft.net> wrote:
> From: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
> Date: Sat, 28 Dec 2013 04:36:13 -0800
>
>> Add missing PCI bus link speed 8.0 GT/s and bus link widths of
>> x1, x2, x4 and x8.
>>
>> CC: <linux-kernel@vger.kernel.org>
>> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
>
> Can a PCI person please ACK this? This is a prerequisite for some
> networking driver changes Jeff would like to push to me.
>
> Thanks.
>
>> ---
>> include/uapi/linux/pci_regs.h | 5 +++++
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h
>> index 4a98e85..c870c2a 100644
>> --- a/include/uapi/linux/pci_regs.h
>> +++ b/include/uapi/linux/pci_regs.h
>> @@ -489,7 +489,12 @@
>> #define PCI_EXP_LNKSTA_CLS 0x000f /* Current Link Speed */
>> #define PCI_EXP_LNKSTA_CLS_2_5GB 0x0001 /* Current Link Speed 2.5GT/s */
>> #define PCI_EXP_LNKSTA_CLS_5_0GB 0x0002 /* Current Link Speed 5.0GT/s */
>> +#define PCI_EXP_LNKSTA_CLS_8_0GB 0x0003 /* Current Link Speed 8.0GT/s */
>> #define PCI_EXP_LNKSTA_NLW 0x03f0 /* Negotiated Link Width */
>> +#define PCI_EXP_LNKSTA_NLW_X1 0x0010 /* Current Link Width x1 */
>> +#define PCI_EXP_LNKSTA_NLW_X2 0x0020 /* Current Link Width x2 */
>> +#define PCI_EXP_LNKSTA_NLW_X4 0x0040 /* Current Link Width x4 */
>> +#define PCI_EXP_LNKSTA_NLW_X8 0x0080 /* Current Link Width x8 */
>> #define PCI_EXP_LNKSTA_NLW_SHIFT 4 /* start of NLW mask in link status */
>> #define PCI_EXP_LNKSTA_LT 0x0800 /* Link Training */
>> #define PCI_EXP_LNKSTA_SLC 0x1000 /* Slot Clock Configuration */
also note that these defines from this patch are already partially
defined in drivers/pci/probe.c but likely need to move into a header
file and be consolidated throughout the kernel. That said I don't see
any issue with adding these as a separate set of defines in
uapi/linux/pci_regs.h
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines
2014-01-03 21:53 ` Jesse Brandeburg
@ 2014-01-03 22:15 ` Bjorn Helgaas
2014-01-03 23:56 ` David Miller
0 siblings, 1 reply; 10+ messages in thread
From: Bjorn Helgaas @ 2014-01-03 22:15 UTC (permalink / raw)
To: Jesse Brandeburg
Cc: David Miller, linux-pci, Jeff Kirsher, NetDEV list, gospo,
sassmann, linux-kernel, Bjorn Helgaas
On Fri, Jan 3, 2014 at 2:53 PM, Jesse Brandeburg
<jesse.brandeburg@gmail.com> wrote:
> +linux-pci
Thanks, Jesse.
> <and see note below>
>
> On Thu, Jan 2, 2014 at 4:21 PM, David Miller <davem@davemloft.net> wrote:
>> From: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
>> Date: Sat, 28 Dec 2013 04:36:13 -0800
>>
>>> Add missing PCI bus link speed 8.0 GT/s and bus link widths of
>>> x1, x2, x4 and x8.
>>>
>>> CC: <linux-kernel@vger.kernel.org>
>>> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
>>
>> Can a PCI person please ACK this? This is a prerequisite for some
>> networking driver changes Jeff would like to push to me.
The additions look correct to me, and I don't object to them per se, so:
Acked-by: Bjorn Helgaas <bhelgaas@google.com>
However, I do raise my eyebrows a bit at drivers that poke around in
the PCIe capability. I would prefer to have PCI core interfaces that
handle that instead. But I haven't seen Jeff's changes yet.
>>> ---
>>> include/uapi/linux/pci_regs.h | 5 +++++
>>> 1 file changed, 5 insertions(+)
>>>
>>> diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h
>>> index 4a98e85..c870c2a 100644
>>> --- a/include/uapi/linux/pci_regs.h
>>> +++ b/include/uapi/linux/pci_regs.h
>>> @@ -489,7 +489,12 @@
>>> #define PCI_EXP_LNKSTA_CLS 0x000f /* Current Link Speed */
>>> #define PCI_EXP_LNKSTA_CLS_2_5GB 0x0001 /* Current Link Speed 2.5GT/s */
>>> #define PCI_EXP_LNKSTA_CLS_5_0GB 0x0002 /* Current Link Speed 5.0GT/s */
>>> +#define PCI_EXP_LNKSTA_CLS_8_0GB 0x0003 /* Current Link Speed 8.0GT/s */
>>> #define PCI_EXP_LNKSTA_NLW 0x03f0 /* Negotiated Link Width */
>>> +#define PCI_EXP_LNKSTA_NLW_X1 0x0010 /* Current Link Width x1 */
>>> +#define PCI_EXP_LNKSTA_NLW_X2 0x0020 /* Current Link Width x2 */
>>> +#define PCI_EXP_LNKSTA_NLW_X4 0x0040 /* Current Link Width x4 */
>>> +#define PCI_EXP_LNKSTA_NLW_X8 0x0080 /* Current Link Width x8 */
>>> #define PCI_EXP_LNKSTA_NLW_SHIFT 4 /* start of NLW mask in link status */
>>> #define PCI_EXP_LNKSTA_LT 0x0800 /* Link Training */
>>> #define PCI_EXP_LNKSTA_SLC 0x1000 /* Slot Clock Configuration */
>
> also note that these defines from this patch are already partially
> defined in drivers/pci/probe.c but likely need to move into a header
> file and be consolidated throughout the kernel. That said I don't see
> any issue with adding these as a separate set of defines in
> uapi/linux/pci_regs.h
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines
2014-01-03 22:15 ` Bjorn Helgaas
@ 2014-01-03 23:56 ` David Miller
2014-01-04 0:00 ` Bjorn Helgaas
0 siblings, 1 reply; 10+ messages in thread
From: David Miller @ 2014-01-03 23:56 UTC (permalink / raw)
To: bhelgaas
Cc: jesse.brandeburg, linux-pci, jeffrey.t.kirsher, netdev, gospo,
sassmann, linux-kernel, bjorn.helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
Date: Fri, 3 Jan 2014 15:15:57 -0700
> However, I do raise my eyebrows a bit at drivers that poke around in
> the PCIe capability. I would prefer to have PCI core interfaces that
> handle that instead. But I haven't seen Jeff's changes yet.
The changes just read the link status to interpret the speed at which
the PCI-E link is running at.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines
2014-01-03 23:56 ` David Miller
@ 2014-01-04 0:00 ` Bjorn Helgaas
2014-01-04 0:30 ` David Miller
0 siblings, 1 reply; 10+ messages in thread
From: Bjorn Helgaas @ 2014-01-04 0:00 UTC (permalink / raw)
To: David Miller
Cc: jesse.brandeburg, linux-pci, Kirsher, Jeffrey T, netdev, gospo,
sassmann, linux-kernel, bjorn.helgaas
On Fri, Jan 3, 2014 at 4:56 PM, David Miller <davem@davemloft.net> wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
> Date: Fri, 3 Jan 2014 15:15:57 -0700
>
>> However, I do raise my eyebrows a bit at drivers that poke around in
>> the PCIe capability. I would prefer to have PCI core interfaces that
>> handle that instead. But I haven't seen Jeff's changes yet.
>
> The changes just read the link status to interpret the speed at which
> the PCI-E link is running at.
Several drivers want to do that. It'd be nice if somebody abstracted
that out somehow. Jacob added pcie_get_minimum_link() which is
similar. But maybe Jeff needs something more in this case.
In any case, it's not a blocker for this patch.
Bjorn
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines
2014-01-04 0:00 ` Bjorn Helgaas
@ 2014-01-04 0:30 ` David Miller
2014-01-04 1:27 ` Jeff Kirsher
0 siblings, 1 reply; 10+ messages in thread
From: David Miller @ 2014-01-04 0:30 UTC (permalink / raw)
To: bhelgaas
Cc: jesse.brandeburg, linux-pci, jeffrey.t.kirsher, netdev, gospo,
sassmann, linux-kernel, bjorn.helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
Date: Fri, 3 Jan 2014 17:00:42 -0700
> On Fri, Jan 3, 2014 at 4:56 PM, David Miller <davem@davemloft.net> wrote:
>> From: Bjorn Helgaas <bhelgaas@google.com>
>> Date: Fri, 3 Jan 2014 15:15:57 -0700
>>
>>> However, I do raise my eyebrows a bit at drivers that poke around in
>>> the PCIe capability. I would prefer to have PCI core interfaces that
>>> handle that instead. But I haven't seen Jeff's changes yet.
>>
>> The changes just read the link status to interpret the speed at which
>> the PCI-E link is running at.
>
> Several drivers want to do that. It'd be nice if somebody abstracted
> that out somehow. Jacob added pcie_get_minimum_link() which is
> similar. But maybe Jeff needs something more in this case.
>
> In any case, it's not a blocker for this patch.
Ok.
Jeff, please merge this via the Intel submission process and don't forget
to add Bjorn's ACK.
Thanks.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines
2014-01-04 0:30 ` David Miller
@ 2014-01-04 1:27 ` Jeff Kirsher
2014-01-04 1:48 ` David Miller
0 siblings, 1 reply; 10+ messages in thread
From: Jeff Kirsher @ 2014-01-04 1:27 UTC (permalink / raw)
To: David Miller
Cc: bhelgaas, jesse.brandeburg, linux-pci, netdev, gospo, sassmann,
linux-kernel, bjorn.helgaas
[-- Attachment #1: Type: text/plain, Size: 1495 bytes --]
On Fri, 2014-01-03 at 19:30 -0500, David Miller wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
> Date: Fri, 3 Jan 2014 17:00:42 -0700
>
> > On Fri, Jan 3, 2014 at 4:56 PM, David Miller <davem@davemloft.net> wrote:
> >> From: Bjorn Helgaas <bhelgaas@google.com>
> >> Date: Fri, 3 Jan 2014 15:15:57 -0700
> >>
> >>> However, I do raise my eyebrows a bit at drivers that poke around in
> >>> the PCIe capability. I would prefer to have PCI core interfaces that
> >>> handle that instead. But I haven't seen Jeff's changes yet.
> >>
> >> The changes just read the link status to interpret the speed at which
> >> the PCI-E link is running at.
> >
> > Several drivers want to do that. It'd be nice if somebody abstracted
> > that out somehow. Jacob added pcie_get_minimum_link() which is
> > similar. But maybe Jeff needs something more in this case.
> >
> > In any case, it's not a blocker for this patch.
>
> Ok.
>
> Jeff, please merge this via the Intel submission process and don't forget
> to add Bjorn's ACK.
>
> Thanks.
Ok, will do. What do you want to do with the current series of patches
(mainly the ixgbe LER patches) on my tree? I know that there is a
thread going on between Joe Perches and Mark, but it sounded like Mark
is working on a follow-on patch to address Joe's concerns.
Guess I want to know if I leave my tree alone for you to pull or toss
the current pull request series and get the i40e series applied.
Cheers,
Jeff
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines
2014-01-04 1:27 ` Jeff Kirsher
@ 2014-01-04 1:48 ` David Miller
2014-01-04 1:51 ` Jeff Kirsher
0 siblings, 1 reply; 10+ messages in thread
From: David Miller @ 2014-01-04 1:48 UTC (permalink / raw)
To: jeffrey.t.kirsher
Cc: bhelgaas, jesse.brandeburg, linux-pci, netdev, gospo, sassmann,
linux-kernel, bjorn.helgaas
From: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Date: Fri, 03 Jan 2014 17:27:39 -0800
> What do you want to do with the current series of patches
> (mainly the ixgbe LER patches) on my tree? I know that there is a
> thread going on between Joe Perches and Mark, but it sounded like Mark
> is working on a follow-on patch to address Joe's concerns.
I want the LER config option removed.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [net-next] pci_regs.h: Add PCI bus link speed and width defines
2014-01-04 1:48 ` David Miller
@ 2014-01-04 1:51 ` Jeff Kirsher
0 siblings, 0 replies; 10+ messages in thread
From: Jeff Kirsher @ 2014-01-04 1:51 UTC (permalink / raw)
To: David Miller
Cc: bhelgaas, jesse.brandeburg, linux-pci, netdev, gospo, sassmann,
linux-kernel, bjorn.helgaas
[-- Attachment #1: Type: text/plain, Size: 480 bytes --]
On Fri, 2014-01-03 at 20:48 -0500, David Miller wrote:
> From: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
> Date: Fri, 03 Jan 2014 17:27:39 -0800
>
> > What do you want to do with the current series of patches
> > (mainly the ixgbe LER patches) on my tree? I know that there is a
> > thread going on between Joe Perches and Mark, but it sounded like Mark
> > is working on a follow-on patch to address Joe's concerns.
>
> I want the LER config option removed.
Ok.
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2014-01-04 1:51 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-12-28 12:36 [net-next] pci_regs.h: Add PCI bus link speed and width defines Jeff Kirsher
2014-01-03 0:21 ` David Miller
2014-01-03 21:53 ` Jesse Brandeburg
2014-01-03 22:15 ` Bjorn Helgaas
2014-01-03 23:56 ` David Miller
2014-01-04 0:00 ` Bjorn Helgaas
2014-01-04 0:30 ` David Miller
2014-01-04 1:27 ` Jeff Kirsher
2014-01-04 1:48 ` David Miller
2014-01-04 1:51 ` Jeff Kirsher
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).