All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeremy Sowden <jeremy@azazel.net>
To: Linux Driver Project Developer List
	<driverdev-devel@linuxdriverproject.org>
Subject: [PATCH v2 0/5] staging: kpc2000: assorted fixes
Date: Wed, 15 May 2019 12:14:32 +0100	[thread overview]
Message-ID: <20190515111437.18828-1-jeremy@azazel.net> (raw)
In-Reply-To: <20190515103454.18456-1-jeremy@azazel.net>

The first patch contains some white-space and formatting fixes that I
made while I was looking at the following two TODO items:

  * the loop in kp2000_probe_cores() that uses probe_core_uio() also
    probably needs error handling.
  * probe_core_uio() probably needs error handling

The second and third patches contain fixes for some sparse warnings.

The last two patches contain the actual error-handling fixes.

Jeremy Sowden (5):
  staging: kpc2000: inverted conditional in order to reduce indentation.
  staging: kpc2000: declare two functions as static.
  staging: kpc2000: added designated initializers to two structs.
  staging: kpc2000: added missing clean-up to probe_core_uio.
  staging: kpc2000: clean up after probe failure.

 drivers/staging/kpc2000/TODO                 |  2 -
 drivers/staging/kpc2000/kpc2000/cell_probe.c | 66 ++++++++++++--------
 2 files changed, 39 insertions(+), 29 deletions(-)

Since v1:

  * in patch 3 removed the assignment of the struct-members initialized
    by the designated initializers.

-- 
2.20.1

_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

  parent reply	other threads:[~2019-05-15 11:14 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-15 10:34 [PATCH 0/5] staging: kpc2000: assorted fixes Jeremy Sowden
2019-05-15 10:34 ` [PATCH 1/5] staging: kpc2000: inverted conditional in order to reduce indentation Jeremy Sowden
2019-05-15 10:34 ` [PATCH 2/5] staging: kpc2000: declare two functions as static Jeremy Sowden
2019-05-15 10:34 ` [PATCH 3/5] staging: kpc2000: added designated initializers to two structs Jeremy Sowden
2019-05-15 11:09   ` Dan Carpenter
2019-05-15 11:11     ` Dan Carpenter
2019-05-15 11:19       ` Jeremy Sowden
2019-05-15 10:34 ` [PATCH 4/5] staging: kpc2000: added missing clean-up to probe_core_uio Jeremy Sowden
2019-05-15 10:34 ` [PATCH 5/5] staging: kpc2000: clean up after probe failure Jeremy Sowden
2019-05-15 11:14 ` Jeremy Sowden [this message]
2019-05-15 11:14   ` [PATCH v2 1/5] staging: kpc2000: inverted conditional in order to reduce indentation Jeremy Sowden
2019-05-15 13:14     ` Greg KH
2019-05-15 13:19       ` Jeremy Sowden
2019-05-15 11:14   ` [PATCH v2 2/5] staging: kpc2000: declare two functions as static Jeremy Sowden
2019-05-15 11:14   ` [PATCH v2 3/5] staging: kpc2000: added designated initializers to two structs Jeremy Sowden
2019-05-15 11:14   ` [PATCH v2 4/5] staging: kpc2000: added missing clean-up to probe_core_uio Jeremy Sowden
2019-05-15 11:14   ` [PATCH v2 5/5] staging: kpc2000: clean up after probe failure Jeremy Sowden
2019-05-15 11:28   ` [PATCH v2 0/5] staging: kpc2000: assorted fixes Dan Carpenter

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190515111437.18828-1-jeremy@azazel.net \
    --to=jeremy@azazel.net \
    --cc=driverdev-devel@linuxdriverproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.