Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
L
linux
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
0
Merge Requests
0
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
linux
Commits
79bf7fc5
Commit
79bf7fc5
authored
Dec 20, 2013
by
Bjorn Helgaas
Browse files
Options
Browse Files
Download
Plain Diff
Merge branch 'pci/misc' into next
* pci/misc: PCI/checkpatch: Deprecate DEFINE_PCI_DEVICE_TABLE
parents
47e0ab3f
92e112fd
Changes
3
Hide whitespace changes
Inline
Side-by-side
Showing
3 changed files
with
12 additions
and
8 deletions
+12
-8
Documentation/PCI/pci.txt
Documentation/PCI/pci.txt
+4
-2
include/linux/pci.h
include/linux/pci.h
+1
-2
scripts/checkpatch.pl
scripts/checkpatch.pl
+7
-4
No files found.
Documentation/PCI/pci.txt
View file @
79bf7fc5
...
...
@@ -123,8 +123,10 @@ initialization with a pointer to a structure describing the driver
The ID table is an array of struct pci_device_id entries ending with an
all-zero entry; use of the macro DEFINE_PCI_DEVICE_TABLE is the preferred
method of declaring the table. Each entry consists of:
all-zero entry. Definitions with static const are generally preferred.
Use of the deprecated macro DEFINE_PCI_DEVICE_TABLE should be avoided.
Each entry consists of:
vendor,device Vendor and device ID to match (or PCI_ANY_ID)
...
...
include/linux/pci.h
View file @
79bf7fc5
...
...
@@ -635,8 +635,7 @@ struct pci_driver {
* DEFINE_PCI_DEVICE_TABLE - macro used to describe a pci device table
* @_table: device table name
*
* This macro is used to create a struct pci_device_id array (a device table)
* in a generic manner.
* This macro is deprecated and should not be used in new code.
*/
#define DEFINE_PCI_DEVICE_TABLE(_table) \
const struct pci_device_id _table[]
...
...
scripts/checkpatch.pl
View file @
79bf7fc5
...
...
@@ -2634,10 +2634,13 @@ sub process {
$herecurr
);
}
# check for declarations of struct pci_device_id
if
(
$line
=~
/\bstruct\s+pci_device_id\s+\w+\s*\[\s*\]\s*\=\s*\{/
)
{
WARN
("
DEFINE_PCI_DEVICE_TABLE
",
"
Use DEFINE_PCI_DEVICE_TABLE for struct pci_device_id
\n
"
.
$herecurr
);
# check for uses of DEFINE_PCI_DEVICE_TABLE
if
(
$line
=~
/\bDEFINE_PCI_DEVICE_TABLE\s*\(\s*(\w+)\s*\)\s*=/
)
{
if
(
WARN
("
DEFINE_PCI_DEVICE_TABLE
",
"
Prefer struct pci_device_id over deprecated DEFINE_PCI_DEVICE_TABLE
\n
"
.
$herecurr
)
&&
$fix
)
{
$fixed
[
$linenr
-
1
]
=~
s/\b(?:static\s+|)DEFINE_PCI_DEVICE_TABLE\s*\(\s*(\w+)\s*\)\s*=\s*/static const struct pci_device_id $1\[\] = /
;
}
}
# check for new typedefs, only function parameters and sparse annotations
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment