lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20140907180023.GA1849@greed>
Date:	Sun, 7 Sep 2014 20:00:23 +0200
From:	Quentin Lambert <lambert.quentin@...il.com>
To:	"Rafael J. Wysocki" <rjw@...ysocki.net>,
	Len Brown <lenb@...nel.org>,
	Bjorn Helgaas <bhelgaas@...gle.com>,
	Scott Murray <scott@...teful.org>
Cc:	linux-acpi@...r.kernel.org, linux-pci@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: [PATCH v2 0/4] Extending the first version of the patch to the
 entire directory

Patch 1 and 2 are straight forward extension from the previous version
of the patch.

Patch 3 introduces checkpatch warning due to lines surpassing 80
characters. Another warning concerning too many preceding tabs
is not fixed either.

Patch 4 are less straight forward similar modifications
the modification made to ibmphp_res.c change the value of len_tmp
when the first condition in if statement is not met.
I wasn't quite sure how to solve elegantly the checkpatch error for
slot.c, I chose to use a label and gotos since it was already used
and allowed for a slight code factorisation. This being my first patch I
understand that this may not be the best solution.



Quentin Lambert (4):
  PCI: fix a missing space coding style issue
  PCI: fix a simple if coding style issue
  PCI: remove assignement from if conditions
  PCI: remove assignement in non straight forward if condition

 drivers/pci/hotplug/acpi_pcihp.c        |  3 ++-
 drivers/pci/hotplug/acpiphp_ibm.c       |  2 +-
 drivers/pci/hotplug/cpci_hotplug_core.c | 13 ++++++----
 drivers/pci/hotplug/cpcihp_generic.c    | 28 ++++++++++-----------
 drivers/pci/hotplug/cpcihp_zt5550.c     | 44 ++++++++++++++++-----------------
 drivers/pci/hotplug/cpqphp.h            |  2 +-
 drivers/pci/hotplug/cpqphp_core.c       |  3 +--
 drivers/pci/hotplug/cpqphp_ctrl.c       | 19 ++++++--------
 drivers/pci/hotplug/cpqphp_nvram.c      | 13 ++++------
 drivers/pci/hotplug/ibmphp_core.c       | 19 ++++++++------
 drivers/pci/hotplug/ibmphp_ebda.c       |  3 +--
 drivers/pci/hotplug/ibmphp_hpc.c        |  3 +--
 drivers/pci/hotplug/ibmphp_pci.c        |  6 +++--
 drivers/pci/hotplug/ibmphp_res.c        | 43 ++++++++++++++++++++++----------
 drivers/pci/hotplug/shpchp_ctrl.c       | 12 ++++++---
 drivers/pci/hotplug/shpchp_hpc.c        |  3 ++-
 drivers/pci/pci.c                       | 19 +++++++++++---
 drivers/pci/slot.c                      | 12 +++++++--
 18 files changed, 143 insertions(+), 104 deletions(-)

-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ