public inbox for linux-staging@lists.linux.dev
 help / color / mirror / Atom feed
From: Dave Penkler <dpenkler@gmail.com>
To: gregkh@linuxfoundation.org, linux-staging@lists.linux.dev,
	linux-kernel@vger.kernel.org
Cc: Dave Penkler <dpenkler@gmail.com>
Subject: [PATCH 00/17] staging: gpib: Console messaging cleanup
Date: Fri, 14 Feb 2025 12:46:51 +0100	[thread overview]
Message-ID: <20250214114708.28947-1-dpenkler@gmail.com> (raw)

The GPIB drivers printed a lot of spurious console messages. This
was linked to the level of code maturity, often using console
messages for debug.

This patch set cleans up the console messaging in the spirit of Greg's
recommendation: "When drivers are successful, they should be quiet"

All pr_info's have been removed except for one which is in the module
init of the common core driver indicating that the GPIB subsystem is
initialized.

All dev_info's have been removed or changed to dev_dbg except for the
attach and probe messages in the agilent and ni usb drivers. This is
to facilitate the creation of config and udev scripts to ensure that a
particular usb device is systematically attached to the same gpib
device file.

All custom debug and tty logging has been removed or replaced with
dev_dbg.

Error messages where the user can figure out what went wrong with
errno have also been removed, particularly timeouts and interrupts
during reads and writes which can occur quite frequently uneccessarily
cluttering up the console log.

The patches are 1 per driver.

Patch  1: agilent_82350b
Patch  2: agilent_82357a
Patch  3: cb7210
Patch  4: cec_gpib
Patch  5: common core
Patch  6: fluke
Patch  7: fmh fgpa
Patch  8: gpio bitbang
Patch  9: hp82335
Patch 10: hp82341
Patch 11: ines
Patch 12: lpvo
Patch 13: nec7210
Patch 14: ni_usb
Patch 15: pc2
Patch 16  tms9914
Patch 17: tnt4882

Dave Penkler (17):
  staging: gpib: agilent pci console messaging cleanup
  staging: gpib: agilent usb console messaging cleanup
  staging: gpib: cb7210 console messaging cleanup
  staging: gpib: cec_gpib console messaging cleanup
  staging: gpib: common core console messaging cleanup
  staging: gpib: fluke console messaging cleanup
  staging: gpib: fmh console messaging cleanup
  staging: gpib: gpio bitbang console messaging cleanup
  staging: gpib: hp82335 console messaging cleanup
  staging: gpib: hp82341 console messaging cleanup
  staging: gpib: ines console messaging cleanup
  staging: gpib: lpvo console messaging cleanup
  staging: gpib: nec7210 console messaging cleanup
  staging: gpib: ni_usb console messaging cleanup
  staging: gpib: pc2 console messaging cleanup
  staging: gpib: tms9914 console messaging cleanup
  staging: gpib: tnt4882 console messaging cleanup

 .../gpib/agilent_82350b/agilent_82350b.c      | 118 ++---
 .../gpib/agilent_82350b/agilent_82350b.h      |   3 -
 .../gpib/agilent_82357a/agilent_82357a.c      | 359 ++++++---------
 drivers/staging/gpib/cb7210/cb7210.c          | 133 +++---
 drivers/staging/gpib/cec/cec_gpib.c           |  32 +-
 drivers/staging/gpib/common/gpib_os.c         | 127 ++----
 drivers/staging/gpib/common/iblib.c           | 109 ++---
 drivers/staging/gpib/eastwood/fluke_gpib.c    |  90 ++--
 drivers/staging/gpib/fmh_gpib/fmh_gpib.c      | 140 +++---
 drivers/staging/gpib/gpio/gpib_bitbang.c      |  56 +--
 drivers/staging/gpib/hp_82335/hp82335.c       |  19 +-
 drivers/staging/gpib/hp_82341/hp_82341.c      |  70 ++-
 drivers/staging/gpib/ines/ines_gpib.c         | 101 ++---
 .../gpib/lpvo_usb_gpib/lpvo_usb_gpib.c        | 255 ++++-------
 drivers/staging/gpib/nec7210/nec7210.c        |  69 +--
 drivers/staging/gpib/ni_usb/ni_usb_gpib.c     | 425 ++++++++----------
 drivers/staging/gpib/pc2/pc2_gpib.c           |  51 ++-
 drivers/staging/gpib/tms9914/tms9914.c        |  33 +-
 drivers/staging/gpib/tnt4882/mite.c           |  17 -
 drivers/staging/gpib/tnt4882/tnt4882_gpib.c   | 186 +++-----
 20 files changed, 912 insertions(+), 1481 deletions(-)

-- 
2.48.1


             reply	other threads:[~2025-02-14 11:47 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-14 11:46 Dave Penkler [this message]
2025-02-14 11:46 ` [PATCH 01/17] staging: gpib: agilent pci console messaging cleanup Dave Penkler
2025-02-14 11:46 ` [PATCH 02/17] staging: gpib: agilent usb " Dave Penkler
2025-02-14 11:46 ` [PATCH 03/17] staging: gpib: cb7210 " Dave Penkler
2025-02-14 11:46 ` [PATCH 04/17] staging: gpib: cec_gpib " Dave Penkler
2025-02-14 11:46 ` [PATCH 05/17] staging: gpib: common core " Dave Penkler
2025-02-14 11:46 ` [PATCH 06/17] staging: gpib: fluke " Dave Penkler
2025-02-14 11:46 ` [PATCH 07/17] staging: gpib: fmh " Dave Penkler
2025-02-14 11:46 ` [PATCH 08/17] staging: gpib: gpio bitbang " Dave Penkler
2025-02-14 11:47 ` [PATCH 09/17] staging: gpib: hp82335 " Dave Penkler
2025-02-14 11:47 ` [PATCH 10/17] staging: gpib: hp82341 " Dave Penkler
2025-02-14 11:47 ` [PATCH 11/17] staging: gpib: ines " Dave Penkler
2025-02-14 11:47 ` [PATCH 12/17] staging: gpib: lpvo " Dave Penkler
2025-02-14 11:47 ` [PATCH 13/17] staging: gpib: nec7210 " Dave Penkler
2025-02-14 11:47 ` [PATCH 14/17] staging: gpib: ni_usb " Dave Penkler
2025-02-14 11:47 ` [PATCH 15/17] staging: gpib: pc2 " Dave Penkler
2025-02-14 11:47 ` [PATCH 16/17] staging: gpib: tms9914 " Dave Penkler
2025-02-14 11:47 ` [PATCH 17/17] staging: gpib: tnt4882 " Dave Penkler
2025-02-19 15:48 ` [PATCH 00/17] staging: gpib: Console " Greg KH
2025-02-20  8:55   ` Dave Penkler
2025-02-20  9:19     ` Greg KH

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=20250214114708.28947-1-dpenkler@gmail.com \
    --to=dpenkler@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-staging@lists.linux.dev \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox