* [PATCH] PCI: Add defines for max read requests sizes
@ 2015-01-26 9:19 Rafał Miłecki
2015-01-26 16:01 ` Bjorn Helgaas
0 siblings, 1 reply; 4+ messages in thread
From: Rafał Miłecki @ 2015-01-26 9:19 UTC (permalink / raw)
To: Bjorn Helgaas, linux-pci; +Cc: Rafał Miłecki
There are few drivers using magic numbers when operating with PCIe
capabilities and PCI_EXP_DEVCTL_READRQ. Define known values to allow
cleaning their code a bit.
Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
---
Hi,
I'm new to the PCI subsystem, so let me know if there is anything wrong with my
patch. I hope you find these bits worth defining and that my coding style
matches rest of the file.
---
include/uapi/linux/pci_regs.h | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h
index 4a1d0cc..efe3443 100644
--- a/include/uapi/linux/pci_regs.h
+++ b/include/uapi/linux/pci_regs.h
@@ -451,6 +451,10 @@
#define PCI_EXP_DEVCTL_AUX_PME 0x0400 /* Auxiliary Power PM Enable */
#define PCI_EXP_DEVCTL_NOSNOOP_EN 0x0800 /* Enable No Snoop */
#define PCI_EXP_DEVCTL_READRQ 0x7000 /* Max_Read_Request_Size */
+#define PCI_EXP_DEVCTL_READRQ_128B 0x0000 /* 128 Bytes */
+#define PCI_EXP_DEVCTL_READRQ_256B 0x1000 /* 256 Bytes */
+#define PCI_EXP_DEVCTL_READRQ_512B 0x2000 /* 512 Bytes */
+#define PCI_EXP_DEVCTL_READRQ_1024B 0x3000 /* 1024 Bytes */
#define PCI_EXP_DEVCTL_BCR_FLR 0x8000 /* Bridge Configuration Retry / FLR */
#define PCI_EXP_DEVSTA 10 /* Device Status */
#define PCI_EXP_DEVSTA_CED 0x0001 /* Correctable Error Detected */
--
1.8.4.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] PCI: Add defines for max read requests sizes
2015-01-26 9:19 [PATCH] PCI: Add defines for max read requests sizes Rafał Miłecki
@ 2015-01-26 16:01 ` Bjorn Helgaas
2015-01-26 16:28 ` Rafał Miłecki
0 siblings, 1 reply; 4+ messages in thread
From: Bjorn Helgaas @ 2015-01-26 16:01 UTC (permalink / raw)
To: Rafał Miłecki; +Cc: linux-pci@vger.kernel.org
On Mon, Jan 26, 2015 at 3:19 AM, Rafał Miłecki <zajec5@gmail.com> wrote:
> There are few drivers using magic numbers when operating with PCIe
> capabilities and PCI_EXP_DEVCTL_READRQ. Define known values to allow
> cleaning their code a bit.
>
> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
> ---
> Hi,
>
> I'm new to the PCI subsystem, so let me know if there is anything wrong with my
> patch. I hope you find these bits worth defining and that my coding style
> matches rest of the file.
> ---
> include/uapi/linux/pci_regs.h | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h
> index 4a1d0cc..efe3443 100644
> --- a/include/uapi/linux/pci_regs.h
> +++ b/include/uapi/linux/pci_regs.h
> @@ -451,6 +451,10 @@
> #define PCI_EXP_DEVCTL_AUX_PME 0x0400 /* Auxiliary Power PM Enable */
> #define PCI_EXP_DEVCTL_NOSNOOP_EN 0x0800 /* Enable No Snoop */
> #define PCI_EXP_DEVCTL_READRQ 0x7000 /* Max_Read_Request_Size */
> +#define PCI_EXP_DEVCTL_READRQ_128B 0x0000 /* 128 Bytes */
> +#define PCI_EXP_DEVCTL_READRQ_256B 0x1000 /* 256 Bytes */
> +#define PCI_EXP_DEVCTL_READRQ_512B 0x2000 /* 512 Bytes */
> +#define PCI_EXP_DEVCTL_READRQ_1024B 0x3000 /* 1024 Bytes */
> #define PCI_EXP_DEVCTL_BCR_FLR 0x8000 /* Bridge Configuration Retry / FLR */
> #define PCI_EXP_DEVSTA 10 /* Device Status */
> #define PCI_EXP_DEVSTA_CED 0x0001 /* Correctable Error Detected */
This seems reasonable. Can you include patches for the drivers to
actually use these new symbols? It's ideal if we can do them at the
same time because we try to avoid adding new symbols unless they're
going to actually be used.
Bjorn
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] PCI: Add defines for max read requests sizes
2015-01-26 16:01 ` Bjorn Helgaas
@ 2015-01-26 16:28 ` Rafał Miłecki
2015-01-26 16:46 ` Bjorn Helgaas
0 siblings, 1 reply; 4+ messages in thread
From: Rafał Miłecki @ 2015-01-26 16:28 UTC (permalink / raw)
To: Bjorn Helgaas; +Cc: linux-pci@vger.kernel.org
On 26 January 2015 at 17:01, Bjorn Helgaas <bhelgaas@google.com> wrote:
> On Mon, Jan 26, 2015 at 3:19 AM, Rafał Miłecki <zajec5@gmail.com> wrote:
>> There are few drivers using magic numbers when operating with PCIe
>> capabilities and PCI_EXP_DEVCTL_READRQ. Define known values to allow
>> cleaning their code a bit.
>>
>> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
>> ---
>> Hi,
>>
>> I'm new to the PCI subsystem, so let me know if there is anything wrong with my
>> patch. I hope you find these bits worth defining and that my coding style
>> matches rest of the file.
>> ---
>> include/uapi/linux/pci_regs.h | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h
>> index 4a1d0cc..efe3443 100644
>> --- a/include/uapi/linux/pci_regs.h
>> +++ b/include/uapi/linux/pci_regs.h
>> @@ -451,6 +451,10 @@
>> #define PCI_EXP_DEVCTL_AUX_PME 0x0400 /* Auxiliary Power PM Enable */
>> #define PCI_EXP_DEVCTL_NOSNOOP_EN 0x0800 /* Enable No Snoop */
>> #define PCI_EXP_DEVCTL_READRQ 0x7000 /* Max_Read_Request_Size */
>> +#define PCI_EXP_DEVCTL_READRQ_128B 0x0000 /* 128 Bytes */
>> +#define PCI_EXP_DEVCTL_READRQ_256B 0x1000 /* 256 Bytes */
>> +#define PCI_EXP_DEVCTL_READRQ_512B 0x2000 /* 512 Bytes */
>> +#define PCI_EXP_DEVCTL_READRQ_1024B 0x3000 /* 1024 Bytes */
>> #define PCI_EXP_DEVCTL_BCR_FLR 0x8000 /* Bridge Configuration Retry / FLR */
>> #define PCI_EXP_DEVSTA 10 /* Device Status */
>> #define PCI_EXP_DEVSTA_CED 0x0001 /* Correctable Error Detected */
>
> This seems reasonable. Can you include patches for the drivers to
> actually use these new symbols? It's ideal if we can do them at the
> same time because we try to avoid adding new symbols unless they're
> going to actually be used.
Sure, but how to send them? Could the go through your (PCI) tree?
--
Rafał
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] PCI: Add defines for max read requests sizes
2015-01-26 16:28 ` Rafał Miłecki
@ 2015-01-26 16:46 ` Bjorn Helgaas
0 siblings, 0 replies; 4+ messages in thread
From: Bjorn Helgaas @ 2015-01-26 16:46 UTC (permalink / raw)
To: Rafał Miłecki; +Cc: linux-pci@vger.kernel.org
On Mon, Jan 26, 2015 at 10:28 AM, Rafał Miłecki <zajec5@gmail.com> wrote:
> On 26 January 2015 at 17:01, Bjorn Helgaas <bhelgaas@google.com> wrote:
>> On Mon, Jan 26, 2015 at 3:19 AM, Rafał Miłecki <zajec5@gmail.com> wrote:
>>> There are few drivers using magic numbers when operating with PCIe
>>> capabilities and PCI_EXP_DEVCTL_READRQ. Define known values to allow
>>> cleaning their code a bit.
>>>
>>> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
>>> ---
>>> Hi,
>>>
>>> I'm new to the PCI subsystem, so let me know if there is anything wrong with my
>>> patch. I hope you find these bits worth defining and that my coding style
>>> matches rest of the file.
>>> ---
>>> include/uapi/linux/pci_regs.h | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h
>>> index 4a1d0cc..efe3443 100644
>>> --- a/include/uapi/linux/pci_regs.h
>>> +++ b/include/uapi/linux/pci_regs.h
>>> @@ -451,6 +451,10 @@
>>> #define PCI_EXP_DEVCTL_AUX_PME 0x0400 /* Auxiliary Power PM Enable */
>>> #define PCI_EXP_DEVCTL_NOSNOOP_EN 0x0800 /* Enable No Snoop */
>>> #define PCI_EXP_DEVCTL_READRQ 0x7000 /* Max_Read_Request_Size */
>>> +#define PCI_EXP_DEVCTL_READRQ_128B 0x0000 /* 128 Bytes */
>>> +#define PCI_EXP_DEVCTL_READRQ_256B 0x1000 /* 256 Bytes */
>>> +#define PCI_EXP_DEVCTL_READRQ_512B 0x2000 /* 512 Bytes */
>>> +#define PCI_EXP_DEVCTL_READRQ_1024B 0x3000 /* 1024 Bytes */
>>> #define PCI_EXP_DEVCTL_BCR_FLR 0x8000 /* Bridge Configuration Retry / FLR */
>>> #define PCI_EXP_DEVSTA 10 /* Device Status */
>>> #define PCI_EXP_DEVSTA_CED 0x0001 /* Correctable Error Detected */
>>
>> This seems reasonable. Can you include patches for the drivers to
>> actually use these new symbols? It's ideal if we can do them at the
>> same time because we try to avoid adding new symbols unless they're
>> going to actually be used.
>
> Sure, but how to send them? Could the go through your (PCI) tree?
I think so. Something like is probably trivial, obviously-correct
patches, and we can probably get acks from the driver maintainers,
then merge them all at once via my tree.
Bjorn
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-01-26 16:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-26 9:19 [PATCH] PCI: Add defines for max read requests sizes Rafał Miłecki
2015-01-26 16:01 ` Bjorn Helgaas
2015-01-26 16:28 ` Rafał Miłecki
2015-01-26 16:46 ` Bjorn Helgaas
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).