[PATCH v1 00/23] staging: gpib: u8 over uint8_t and u64 over uint64_t

Michael Rubin posted 23 patches 1 month ago
.../gpib/agilent_82350b/agilent_82350b.c      | 22 +++++-----
.../gpib/agilent_82357a/agilent_82357a.c      | 20 +++++-----
drivers/staging/gpib/cb7210/cb7210.c          | 26 ++++++------
drivers/staging/gpib/cb7210/cb7210.h          | 12 +++---
drivers/staging/gpib/cec/cec_gpib.c           | 16 ++++----
drivers/staging/gpib/common/gpib_os.c         |  8 ++--
drivers/staging/gpib/common/iblib.c           | 12 +++---
drivers/staging/gpib/common/ibsys.h           |  6 +--
drivers/staging/gpib/eastwood/fluke_gpib.c    | 28 ++++++-------
drivers/staging/gpib/eastwood/fluke_gpib.h    | 12 +++---
drivers/staging/gpib/fmh_gpib/fmh_gpib.c      | 32 +++++++--------
drivers/staging/gpib/fmh_gpib/fmh_gpib.h      |  6 +--
drivers/staging/gpib/gpio/gpib_bitbang.c      | 20 +++++-----
drivers/staging/gpib/hp_82335/hp82335.c       | 20 +++++-----
drivers/staging/gpib/hp_82341/hp_82341.c      | 26 ++++++------
drivers/staging/gpib/include/gpib_proto.h     | 14 +++----
drivers/staging/gpib/include/gpib_types.h     | 18 ++++-----
drivers/staging/gpib/include/nec7210.h        | 33 ++++++++-------
drivers/staging/gpib/include/tms9914.h        | 24 +++++------
drivers/staging/gpib/ines/ines.h              | 20 +++++-----
drivers/staging/gpib/ines/ines_gpib.c         | 22 +++++-----
.../gpib/lpvo_usb_gpib/lpvo_usb_gpib.c        |  8 ++--
drivers/staging/gpib/nec7210/nec7210.c        | 40 +++++++++----------
drivers/staging/gpib/ni_usb/ni_usb_gpib.c     | 16 ++++----
drivers/staging/gpib/pc2/pc2_gpib.c           | 16 ++++----
drivers/staging/gpib/tms9914/tms9914.c        | 34 ++++++++--------
drivers/staging/gpib/tnt4882/tnt4882_gpib.c   | 28 ++++++-------
drivers/staging/gpib/uapi/gpib_ioctl.h        | 16 ++++----
drivers/staging/gpib/uapi/gpib_user.h         | 26 ++++++------
29 files changed, 290 insertions(+), 291 deletions(-)
[PATCH v1 00/23] staging: gpib: u8 over uint8_t and u64 over uint64_t
Posted by Michael Rubin 1 month ago
Preferring u8 over uint8_t and preferring u64 over uint64_t to adhere to
Linux code style.

* Patch 1 - Patch 22:
	Reported by checkpatch.pl.
	CHECK: Prefer kernel type 'u8' over 'uint8_t'

* Patch 23:
	Reported by checkpatch.pl.
	CHECK: Prefer kernel type 'u64' over 'uint64_t'

Michael Rubin (23):
  staging: gpib: cb7210: u8 over uint8_t
  staging: gpib: ibsys: u8 over uint8_t
  staging: gpib: fluke_gpib: u8 over uint8_t
  staging: gpib: fmh_gpib: u8 over uint8_t
  staging: gpib: gpib_proto: u8 over uint8_t
  staging: gpib: gpib_types: u8 over uint8_t
  staging: gpib: nec7210: u8 over uint8_t
  staging: gpib: tms9914: u8 over uint8_t
  staging: gpib: ines: u8 over uint8_t
  staging: gpib: gpib_ioctl: u8 over uint8_t
  staging: gpib: gpib_user: u8 over uint8_t
  staging: gpib: agilent_82350b: u8 over uint8_t
  staging: gpib: agilent_82357a: u8 over uint8_t
  staging: gpib: cec: u8 over uint8_t
  staging: gpib: gpib_os: u8 over uint8_t
  staging: gpib: gpib_bitbang: u8 over uint8_t
  staging: gpib: hp_82335: u8 over uint8_t
  staging: gpib: hp_82341: u8 over uint8_t
  staging: gpib: lpvo_usb_gpib: u8 over uint8_t
  staging: gpib: ni_usb: u8 over uint8_t
  staging: gpib: pc2: u8 over uint8_t
  staging: gpib: tnt4882: u8 over uint8_t
  staging: gpib: gpib_ioctl: u64 over uint64_t

 .../gpib/agilent_82350b/agilent_82350b.c      | 22 +++++-----
 .../gpib/agilent_82357a/agilent_82357a.c      | 20 +++++-----
 drivers/staging/gpib/cb7210/cb7210.c          | 26 ++++++------
 drivers/staging/gpib/cb7210/cb7210.h          | 12 +++---
 drivers/staging/gpib/cec/cec_gpib.c           | 16 ++++----
 drivers/staging/gpib/common/gpib_os.c         |  8 ++--
 drivers/staging/gpib/common/iblib.c           | 12 +++---
 drivers/staging/gpib/common/ibsys.h           |  6 +--
 drivers/staging/gpib/eastwood/fluke_gpib.c    | 28 ++++++-------
 drivers/staging/gpib/eastwood/fluke_gpib.h    | 12 +++---
 drivers/staging/gpib/fmh_gpib/fmh_gpib.c      | 32 +++++++--------
 drivers/staging/gpib/fmh_gpib/fmh_gpib.h      |  6 +--
 drivers/staging/gpib/gpio/gpib_bitbang.c      | 20 +++++-----
 drivers/staging/gpib/hp_82335/hp82335.c       | 20 +++++-----
 drivers/staging/gpib/hp_82341/hp_82341.c      | 26 ++++++------
 drivers/staging/gpib/include/gpib_proto.h     | 14 +++----
 drivers/staging/gpib/include/gpib_types.h     | 18 ++++-----
 drivers/staging/gpib/include/nec7210.h        | 33 ++++++++-------
 drivers/staging/gpib/include/tms9914.h        | 24 +++++------
 drivers/staging/gpib/ines/ines.h              | 20 +++++-----
 drivers/staging/gpib/ines/ines_gpib.c         | 22 +++++-----
 .../gpib/lpvo_usb_gpib/lpvo_usb_gpib.c        |  8 ++--
 drivers/staging/gpib/nec7210/nec7210.c        | 40 +++++++++----------
 drivers/staging/gpib/ni_usb/ni_usb_gpib.c     | 16 ++++----
 drivers/staging/gpib/pc2/pc2_gpib.c           | 16 ++++----
 drivers/staging/gpib/tms9914/tms9914.c        | 34 ++++++++--------
 drivers/staging/gpib/tnt4882/tnt4882_gpib.c   | 28 ++++++-------
 drivers/staging/gpib/uapi/gpib_ioctl.h        | 16 ++++----
 drivers/staging/gpib/uapi/gpib_user.h         | 26 ++++++------
 29 files changed, 290 insertions(+), 291 deletions(-)

-- 
2.43.0
Re: [PATCH v1 00/23] staging: gpib: u8 over uint8_t and u64 over uint64_t
Posted by Greg KH 3 weeks, 4 days ago
On Wed, Apr 09, 2025 at 06:09:30PM +0000, Michael Rubin wrote:
> Preferring u8 over uint8_t and preferring u64 over uint64_t to adhere to
> Linux code style.
> 
> * Patch 1 - Patch 22:
> 	Reported by checkpatch.pl.
> 	CHECK: Prefer kernel type 'u8' over 'uint8_t'
> 
> * Patch 23:
> 	Reported by checkpatch.pl.
> 	CHECK: Prefer kernel type 'u64' over 'uint64_t'
> 

Only a portion of this series could be applied, as I hit some patch
errors that I couldn't resolve easily.  Odds are we are out of sync
somehow, can you rebase and resend the remaining ones?

thanks,

greg k-h
Re: [PATCH v1 00/23] staging: gpib: u8 over uint8_t and u64 over uint64_t
Posted by Michael Rubin 3 weeks, 3 days ago
On Tue, Apr 15, 2025 at 04:45:58PM +0200, Greg KH wrote:
> On Wed, Apr 09, 2025 at 06:09:30PM +0000, Michael Rubin wrote:
> > Preferring u8 over uint8_t and preferring u64 over uint64_t to adhere to
> > Linux code style.
> > 
> > * Patch 1 - Patch 22:
> > 	Reported by checkpatch.pl.
> > 	CHECK: Prefer kernel type 'u8' over 'uint8_t'
> > 
> > * Patch 23:
> > 	Reported by checkpatch.pl.
> > 	CHECK: Prefer kernel type 'u64' over 'uint64_t'
> > 
> 
> Only a portion of this series could be applied, as I hit some patch
> errors that I couldn't resolve easily.  Odds are we are out of sync
> somehow, can you rebase and resend the remaining ones?

Done. Hope these are the right ones.

Thanks for your patience.

Michael Rubin