1. 15 Jan, 2018 40 commits
    • Greg Kroah-Hartman's avatar
      staging: dgnc: make dgnc_num_boards static · 197c3196
      Greg Kroah-Hartman authored
      There's no need for the world to know the number of dgnc boards in the
      system, so just make it a static variable.
      
      Cc: Lidza Louina <lidza.louina@gmail.com>
      Cc: Mark Hounschell <markh@compro.net>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      197c3196
    • Greg Kroah-Hartman's avatar
      staging: dgnc: remove crazy "management" device node · 37034c1b
      Greg Kroah-Hartman authored
      There's no need for a special character device just to get some random
      information out of a single serial port driver.  So remove the
      dgnc_mgmt.c file, and some structures and ioctl definitions that only it
      was using.
      
      Cc: Lidza Louina <lidza.louina@gmail.com>
      Cc: Mark Hounschell <markh@compro.net>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      37034c1b
    • Greg Kroah-Hartman's avatar
      staging: dgnc: delete dgnc_utils.c · f75c1b29
      Greg Kroah-Hartman authored
      There was just one function in it, and it duplicated what
      msleep_interruptable() was doing, which is pointless, so delete it and
      fix up the one calling site.
      
      Cc: Lidza Louina <lidza.louina@gmail.com>
      Cc: Mark Hounschell <markh@compro.net>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      f75c1b29
    • Greg Kroah-Hartman's avatar
      staging: fbtft: remove redundant license text · 78c00ccc
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in all fbtft files, that identifies the license
      in a specific and legally-defined manner.  So the extra GPL text wording
      can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      
      Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      78c00ccc
    • Greg Kroah-Hartman's avatar
      staging: fbtft: add SPDX identifiers · 783de57c
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the all of the staging fbtft drivers to have a proper SPDX
      identifier, based on the license text in the file itself.  The SPDX
      identifier is a legally binding shorthand, which can be used instead of
      the full boiler plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      783de57c
    • Greg Kroah-Hartman's avatar
      staging: emxx_udc: remove redundant license text · a04c7278
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in the emxx_udc driver files, that identifies
      the license in a specific and legally-defined manner.  So the extra GPL
      text wording can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      a04c7278
    • Greg Kroah-Hartman's avatar
      staging: emxx_udc: add SPDX identifier. · 38e08a45
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the staging emxx_udc driver to have a proper SPDX identifier,
      based on the license text in the file itself.  The SPDX identifier is a
      legally binding shorthand, which can be used instead of the full boiler
      plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      38e08a45
    • Greg Kroah-Hartman's avatar
      staging: clocking-wizard: remove redundant license text · 09956d59
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in the clocking-wizard driver files, that
      identifies the license in a specific and legally-defined manner.  So the
      extra GPL text wording can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      09956d59
    • Greg Kroah-Hartman's avatar
      staging: clocking-wizard: add SPDX identifier. · 1dbb3344
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the staging clocking-wizard driver to have a proper SPDX
      identifier, based on the license text in the file itself.  The SPDX
      identifier is a legally binding shorthand, which can be used instead of
      the full boiler plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      1dbb3344
    • Greg Kroah-Hartman's avatar
      staging: dgnc: remove redundant license text · ac1eedf2
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in all drivers/staging/dgnc files, that
      identifies the license in a specific and legally-defined manner.  So the
      extra GPL text wording can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      
      Cc: Lidza Louina <lidza.louina@gmail.com>
      Cc: Mark Hounschell <markh@compro.net>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      ac1eedf2
    • Greg Kroah-Hartman's avatar
      staging: dgnc: add SPDX identifier. · 0efdb6da
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the staging dgnc driver to have a proper SPDX identifier, based
      on the license text in the file itself.  The SPDX identifier is a
      legally binding shorthand, which can be used instead of the full boiler
      plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Cc: Lidza Louina <lidza.louina@gmail.com>
      Cc: Mark Hounschell <markh@compro.net>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      0efdb6da
    • Greg Kroah-Hartman's avatar
      staging: fwserial: remove redundant license text · d461cc8a
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in all drivers/staging/fwserial/ files, that
      identifies the license in a specific and legally-defined manner.  So the
      extra GPL text wording can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      d461cc8a
    • Greg Kroah-Hartman's avatar
      staging: fwserial: add SPDX identifier. · 980e27bd
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the staging fwserial driver to have a proper SPDX identifier,
      based on the license text in the file itself.  The SPDX identifier is a
      legally binding shorthand, which can be used instead of the full boiler
      plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      980e27bd
    • Greg Kroah-Hartman's avatar
      staging: octeon: remove redundant license text · 98a95b9c
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in all drivers/staging/octeon/ files, that
      identifies the license in a specific and legally-defined manner.  So the
      extra GPL text wording can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      98a95b9c
    • Greg Kroah-Hartman's avatar
      staging: octeon: add SPDX identifiers. · 30bdc499
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the staging octeon driver to have a proper SPDX identifier, based
      on the license text in the file itself.  The SPDX identifier is a
      legally binding shorthand, which can be used instead of the full boiler
      plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      30bdc499
    • Greg Kroah-Hartman's avatar
      staging: nvec: remove redundant license text · d6d69c82
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in all drivers/staging/nvec/ files, that
      identifies the license in a specific and legally-defined manner.  So the
      extra GPL text wording can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      
      Cc: Marc Dietrich <marvin24@gmx.de>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      d6d69c82
    • Greg Kroah-Hartman's avatar
      staging: nvec: add SPDX identifier. · 971bcfca
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the staging nvec driver to have a proper SPDX identifiers, based
      on the license text in the file itself.  The SPDX identifier is a
      legally binding shorthand, which can be used instead of the full boiler
      plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Cc: Marc Dietrich <marvin24@gmx.de>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      971bcfca
    • Greg Kroah-Hartman's avatar
      staging: speakup: remove redundant license text · a407235b
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in all drivers/staging/speakup/ files, that
      identifies the license in a specific and legally-defined manner.  So the
      extra GPL text wording can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      
      Cc: William Hubbs <w.d.hubbs@gmail.com>
      Cc: Chris Brannon <chris@the-brannons.com>
      Cc: Kirk Reiser <kirk@reisers.ca>
      Cc: Samuel Thibault <samuel.thibault@ens-lyon.org>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      a407235b
    • Greg Kroah-Hartman's avatar
      staging: speakup: add SPDX identifier. · 64969228
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the remaining staging speakup files to have a proper SPDX
      identifier, based on the license text in the file itself.  The SPDX
      identifier is a legally binding shorthand, which can be used instead of
      the full boiler plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Cc: William Hubbs <w.d.hubbs@gmail.com>
      Cc: Chris Brannon <chris@the-brannons.com>
      Cc: Kirk Reiser <kirk@reisers.ca>
      Cc: Samuel Thibault <samuel.thibault@ens-lyon.org>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      64969228
    • Greg Kroah-Hartman's avatar
      staging: vc04_services: bcm2835-camera: remove redundant license text · b4c95f03
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in all
      drivers/staging/vc04_services/bcm2835-camera/ files, that identifies the
      license in a specific and legally-defined manner.  So the extra GPL text
      wording can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      
      Cc: Eric Anholt <eric@anholt.net>
      Cc: Stefan Wahren <stefan.wahren@i2se.com>
      Cc: Florian Fainelli <f.fainelli@gmail.com>
      Cc: Ray Jui <rjui@broadcom.com>
      Cc: Scott Branden <sbranden@broadcom.com>
      Cc: <bcm-kernel-feedback-list@broadcom.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      b4c95f03
    • Greg Kroah-Hartman's avatar
      staging: vc04_services: bcm2835-camera: add SPDX identifiers · a42b7eb5
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the bcm2835-camera driver to have a proper SPDX identifier, based
      on the license text in the file itself.  The SPDX identifier is a
      legally binding shorthand, which can be used instead of the full boiler
      plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Cc: Eric Anholt <eric@anholt.net>
      Cc: Stefan Wahren <stefan.wahren@i2se.com>
      Cc: Florian Fainelli <f.fainelli@gmail.com>
      Cc: Ray Jui <rjui@broadcom.com>
      Cc: Scott Branden <sbranden@broadcom.com>
      Cc: <bcm-kernel-feedback-list@broadcom.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      a42b7eb5
    • Greg Kroah-Hartman's avatar
      staging: vc04_services: bcm2835-audio: remove redundant license text · f3ff723f
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in all
      drivers/staging/vc04_services/bcm2835-audio/ files, that identifies the
      license in a specific and legally-defined manner.  So the extra GPL text
      wording can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      
      Cc: Stefan Wahren <stefan.wahren@i2se.com>
      Cc: Florian Fainelli <f.fainelli@gmail.com>
      Cc: Ray Jui <rjui@broadcom.com>
      Cc: Scott Branden <sbranden@broadcom.com>
      Cc: <bcm-kernel-feedback-list@broadcom.com>
      Reviewed-by: default avatarEric Anholt <eric@anholt.net>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      f3ff723f
    • Greg Kroah-Hartman's avatar
      staging: vc04_services: bcm2835-audio: add SPDX identifiers · 598dce85
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the bcm2835-audio driver to have a proper SPDX identifier, based
      on the license text in the file itself.  The SPDX identifier is a
      legally binding shorthand, which can be used instead of the full boiler
      plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Cc: Eric Anholt <eric@anholt.net>
      Cc: Stefan Wahren <stefan.wahren@i2se.com>
      Cc: Florian Fainelli <f.fainelli@gmail.com>
      Cc: Ray Jui <rjui@broadcom.com>
      Cc: Scott Branden <sbranden@broadcom.com>
      Cc: <bcm-kernel-feedback-list@broadcom.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      598dce85
    • Greg Kroah-Hartman's avatar
      staging: typec: remove redundant license text · 0b79361c
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in all drivers/staging/typec/ files, that
      identifies the license in a specific and legally-defined manner.  So the
      extra GPL text wording can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      0b79361c
    • Greg Kroah-Hartman's avatar
      staging: typec: add SPDX identifier. · 2ded4527
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the drivers/staging/typec/ files to have a proper SPDX
      identifier, based on the license text in the file itself.  The SPDX
      identifier is a legally binding shorthand, which can be used instead of
      the full boiler plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      2ded4527
    • Greg Kroah-Hartman's avatar
      staging: vme: vme_user.c: remove redundant license text · 73f8a545
      Greg Kroah-Hartman authored
      Now that the SPDX tag is in all drivers/staging/vme/ files, that
      identifies the license in a specific and legally-defined manner.  So the
      extra GPL text wording can be removed as it is no longer needed at all.
      
      This is done on a quest to remove the 700+ different ways that files in
      the kernel describe the GPL license text.  And there's unneeded stuff
      like the address (sometimes incorrect) for the FSF which is never
      needed.
      
      No copyright headers or other non-license-description text was removed.
      
      Cc: Martyn Welch <martyn@welchs.me.uk>
      Cc: Manohar Vanga <manohar.vanga@gmail.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      73f8a545
    • Greg Kroah-Hartman's avatar
      staging: vme: vme_user: add SPDX identifier. · 0057cb5e
      Greg Kroah-Hartman authored
      It's good to have SPDX identifiers in all files to make it easier to
      audit the kernel tree for correct licenses.
      
      Fix up the remaining staging vme driver to have a proper SPDX
      identifier, based on the license text in the file itself.  The SPDX
      identifier is a legally binding shorthand, which can be used instead of
      the full boiler plate text.
      
      This work is based on a script and data from Thomas Gleixner, Philippe
      Ombredanne, and Kate Stewart.
      
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Cc: Kate Stewart <kstewart@linuxfoundation.org>
      Cc: Philippe Ombredanne <pombredanne@nexb.com>
      Cc: Martyn Welch <martyn@welchs.me.uk>
      Cc: Manohar Vanga <manohar.vanga@gmail.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      0057cb5e
    • Fabian Huegel's avatar
      staging: lustre: Align backslashes in multi-line macros · a7380cef
      Fabian Huegel authored
      This patch right aligns all backslashes in multi-line macros
      in obd_class.h for better readability.
      Signed-off-by: default avatarFabian Huegel <fabian_huegel@web.de>
      Signed-off-by: default avatarChristoph Volkert <linux@christoph-volkert.de>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      a7380cef
    • Fabian Huegel's avatar
      staging: lustre: Fix overlong lines · 6bf2270a
      Fabian Huegel authored
      Fixed four lines that went over the 80 character limit
      to reduce checkpatch warnings.
      Signed-off-by: default avatarFabian Huegel <fabian_huegel@web.de>
      Signed-off-by: default avatarChristoph Volkert <linux@christoph-volkert.de>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      6bf2270a
    • Fabian Huegel's avatar
      staging: lustre: Add identifier names to function declarations · 942381bf
      Fabian Huegel authored
      Checkpatch was complaining about missing identifier names in function
      declarations. So we added the missing names according to the names in
      the respective function implementation. *obd_import* was sometimes named
      *import* and sometimes *imp* (in genops.c), so to avoid confusion we just
      named it imp everywhere.
      Signed-off-by: default avatarFabian Huegel <fabian_huegel@web.de>
      Signed-off-by: default avatarChristoph Volkert <linux@christoph-volkert.de>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      942381bf
    • Fabian Huegel's avatar
      staging: lustre: Remove DECLARE_LU_VARS macro · a84b9900
      Fabian Huegel authored
      This macro was only used in four places to declare two variables.
      It saved one line of code, but in our opinion hurt readability.
      So we removed the macro, substituting every occurrence with the
      declaration of the two variables (like the preprocessor would have done).
      Signed-off-by: default avatarFabian Huegel <fabian_huegel@web.de>
      Signed-off-by: default avatarChristoph Volkert <linux@christoph-volkert.de>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      a84b9900
    • Fabian Huegel's avatar
      staging: lustre: Enclose complex macros in do-while loops · 6b1833a3
      Fabian Huegel authored
      Some complex multi-line macros were not enclosed by a do-while(0),
      so we fixed that.
      Signed-off-by: default avatarFabian Huegel <fabian_huegel@web.de>
      Signed-off-by: default avatarChristoph Volkert <linux@christoph-volkert.de>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      6b1833a3
    • Kenny Ballou's avatar
      staging: xgifb: vb_table: fix whitespacing issues · 454431c6
      Kenny Ballou authored
      Correct minor checkpatch issues.
      Signed-off-by: default avatarKenny Ballou <kballou@devnulllabs.io>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      454431c6
    • Ajay Singh's avatar
      staging: wilc1000: removed enum typedef BUS_RELEASE_T · a9ee75e8
      Ajay Singh authored
      This patch removes enum typedef BUS_RELEASE_T and define
      "enum bus_release" to use instead of typedef.
      
      checkpatch.pl not to add new typedef warning is fixed with this patch.
      Signed-off-by: default avatarAjay Singh <ajay.kathat@microchip.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      a9ee75e8
    • Ajay Singh's avatar
      staging: wilc1000: removed enum typedef BUS_ACQUIRE_T · 13ae5a7b
      Ajay Singh authored
      This patch removes enum typedef BUS_ACQUIRE_T and define
      enum bus_acquire to use instead of typedef.
      
      checkpatch.pl not to add new typedef warning is fixed with this patch.
      Signed-off-by: default avatarAjay Singh <ajay.kathat@microchip.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      13ae5a7b
    • Ajay Singh's avatar
      staging: wilc1000: removed enum typedef CHIP_PS_STATE_T · 888505e7
      Ajay Singh authored
      This patch removes enum typedef CHIP_PS_STATE_T and introduce
      enum chip_ps_states to use instead of typedef.
      
      checkpatch.pl not to add new typedef warning is fixed with this patch.
      Signed-off-by: default avatarAjay Singh <ajay.kathat@microchip.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      888505e7
    • Ajay Singh's avatar
      staging: wilc1000: removed enums typedef for BEACON_IE & TX_RATE_T · 98c899c9
      Ajay Singh authored
      This patch removed the unnecessary enum typedef for BEACON_IE &
      TX_RATE_T
      It fix "WARNING: do not add new typedefs" reported by checkpatch.pl
      Signed-off-by: default avatarAjay Singh <ajay.kathat@microchip.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      98c899c9
    • Ajay Singh's avatar
      staging: wilc1000: remove unnecessary typedef enum G_OPERATING_MODE_T · 57d6c22a
      Ajay Singh authored
      This patch has removed G_OPERATING_MODE_T typedef enum.
      Now, its used as anonymous-enums for constants.
      checkpatch.pl warning to avoid new typedef is fixes with this patch.
      Signed-off-by: default avatarAjay Singh <ajay.kathat@microchip.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      57d6c22a
    • Ajay Singh's avatar
      staging: wilc1000: removed typedef from enum BSSTYPE_T · 25bab0e1
      Ajay Singh authored
      This patch removes typedef from enum BSSTYPE_T and
      rename it to bss_types.
      
      It fixes "WARNING: do not add new typdefs" warning
      reported by checkpatch.pl.
      Signed-off-by: default avatarAjay Singh <ajay.kathat@microchip.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      25bab0e1
    • Colin Ian King's avatar
      staging: ccree: fix memory leaks in cc_ivgen_init · bd12b21e
      Colin Ian King authored
      The current error exit path in function cc_ivgen_init via label
      'out' free's resources from the drvdata->ivgen_handle context.
      However, drvdata->ivgen_handle has not been assigned to the
      context ivgen_ctx at this point, so the resources are not freed.
      Fix this by setting drvdata->ivgen_handle to ivgen_ctx as early
      as possible so that the clean up error exit return path can free
      the resources.
      
      Detected by CoveritScan, CID#1463795 ("Resource leak")
      Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
      Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
      bd12b21e