Skip to content

more cleanups - #632

Merged
plbossart merged 5 commits into
thesofproject:topic/sof-devfrom
plbossart:fix/more-cleanups
Feb 8, 2019
Merged

plbossart merged 5 commits into
thesofproject:topic/sof-devfrom
plbossart:fix/more-cleanups

Conversation

@plbossart

Copy link
Copy Markdown
Member

remove unused fields, remove pci field (use to_pci_dev() instead, add const when we missed them.

Remove unused prototypes, cleanup comments and remove id field (not
sure what it was used for, if ever)

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
use to_pci_dev() as needed, no need to have a dedicated field in the core

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
copy/pastes likely, fix missing const

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
again missing const, add

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
we set the .ignore_machine field dynamically to the machine driver
name, so add note to say we cannot use const (reported by checkpatch)

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
@plbossart
plbossart requested a review from ranj063 February 7, 2019 23:31
@plbossart
plbossart merged commit 14d15ff into thesofproject:topic/sof-dev Feb 8, 2019
oder-chiou pushed a commit to oder-chiou/linux-soundwire that referenced this pull request May 5, 2026
Fix checkpatch code style warnings:

  WARNING: quoted string split across lines
  thesofproject#480: FILE: net/core/pktgen.c:480:
  +       "Packet Generator for packet performance testing. "
  +       "Version: " VERSION "\n";

  WARNING: quoted string split across lines
  thesofproject#632: FILE: net/core/pktgen.c:632:
  +                  "     udp_src_min: %d  udp_src_max: %d"
  +                  "  udp_dst_min: %d  udp_dst_max: %d\n",

Signed-off-by: Peter Seiderer <ps.report@gmx.net>
Reviewed-by: Toke Høiland-Jørgensen <toke@redhat.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants