linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Alex Elder <elder@linaro.org>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	devel@driverdev.osuosl.org, greybus-dev@lists.linaro.org,
	elder@kernel.org, johan@kernel.org
Cc: linux-kernel@vger.kernel.org
Subject: Re: [greybus-dev] [PATCH 0/9] staging: move greybus core out of staging
Date: Tue, 27 Aug 2019 05:43:19 -0500	[thread overview]
Message-ID: <cfefa25a-9ee5-6011-a41c-ce09dd9608e6@linaro.org> (raw)
In-Reply-To: <20190825055429.18547-1-gregkh@linuxfoundation.org>

On 8/25/19 12:54 AM, Greg Kroah-Hartman wrote:
> The Greybus code has long been "stable" but was living in the
> drivers/staging/ directory to see if there really was going to be
> devices using this protocol over the long-term.  With the success of
> millions of phones with this hardware and code in it, and the recent
> Google Summer of Code project, and a number of of new devices in the
> works from various companies, it is time to finally move this code out
> of staging into the "real" portion of the kernel so that people know
> they can rely on it.
> 
> This series first does a little bit of checkpatch cleanups for some
> basic remaining issues in the greybus files, and then moves the include
> directory, the greybus core code, and the es2 greybus host controller
> driver into drivers/greybus.
> 
> To come after this is the movement of the Documentation entries and a
> number of the module drivers that are stable.

They all look good to me.  (I don't always agree with checkpatch,
but standardization is good.)  Thanks Greg.

Acked-by: Alex Elder <elder@kernel.org>

> Greg Kroah-Hartman (9):
>   staging: greybus: fix up SPDX comment in .h files
>   staging: greybus: remove license "boilerplate"
>   staging: greybus: hd: Fix up some alignment checkpatch issues
>   staging: greybus: manifest: Fix up some alignment checkpatch issues
>   staging: greybus: log: Fix up some alignment checkpatch issues
>   staging: greybus: loopback: Fix up some alignment checkpatch issues
>   staging: greybus: move core include files to include/linux/greybus/
>   staging: greybus: move the greybus core to drivers/greybus
>   staging: greybus: move es2 to drivers/greybus/
> 
>  MAINTAINERS                                   |   3 +
>  drivers/Kconfig                               |   2 +
>  drivers/Makefile                              |   1 +
>  drivers/greybus/Kconfig                       |  32 +++++
>  drivers/greybus/Makefile                      |  26 +++++
>  drivers/greybus/arpc.h                        |  63 ++++++++++
>  drivers/{staging => }/greybus/bundle.c        |   2 +-
>  drivers/{staging => }/greybus/connection.c    |   2 +-
>  drivers/{staging => }/greybus/control.c       |   2 +-
>  drivers/{staging => }/greybus/core.c          |   2 +-
>  drivers/{staging => }/greybus/debugfs.c       |   3 +-
>  drivers/{staging => }/greybus/es2.c           |   3 +-
>  drivers/{staging => }/greybus/greybus_trace.h |   2 +-
>  drivers/{staging => }/greybus/hd.c            |  12 +-
>  drivers/{staging => }/greybus/interface.c     |   2 +-
>  drivers/{staging => }/greybus/manifest.c      |  41 ++++---
>  drivers/{staging => }/greybus/module.c        |   2 +-
>  drivers/{staging => }/greybus/operation.c     |   2 +-
>  drivers/{staging => }/greybus/svc.c           |   3 +-
>  drivers/{staging => }/greybus/svc_watchdog.c  |   2 +-
>  .../Documentation/firmware/authenticate.c     |  46 --------
>  .../greybus/Documentation/firmware/firmware.c |  46 --------
>  drivers/staging/greybus/Kconfig               |  27 -----
>  drivers/staging/greybus/Makefile              |  22 ----
>  drivers/staging/greybus/arche-platform.c      |   2 +-
>  drivers/staging/greybus/arpc.h                | 109 ------------------
>  drivers/staging/greybus/audio_apbridgea.c     |   3 +-
>  drivers/staging/greybus/audio_apbridgea.h     |  26 +----
>  drivers/staging/greybus/audio_codec.h         |   4 +-
>  drivers/staging/greybus/audio_gb.c            |   4 +-
>  drivers/staging/greybus/authentication.c      |   3 +-
>  drivers/staging/greybus/bootrom.c             |   2 +-
>  drivers/staging/greybus/camera.c              |   2 +-
>  drivers/staging/greybus/firmware.h            |   4 +-
>  drivers/staging/greybus/fw-core.c             |   2 +-
>  drivers/staging/greybus/fw-download.c         |   2 +-
>  drivers/staging/greybus/fw-management.c       |   2 +-
>  drivers/staging/greybus/gb-camera.h           |   2 +-
>  drivers/staging/greybus/gbphy.c               |   2 +-
>  drivers/staging/greybus/gbphy.h               |   2 +-
>  drivers/staging/greybus/gpio.c                |   2 +-
>  .../staging/greybus/greybus_authentication.h  |  48 +-------
>  drivers/staging/greybus/greybus_firmware.h    |  48 +-------
>  drivers/staging/greybus/hid.c                 |   3 +-
>  drivers/staging/greybus/i2c.c                 |   2 +-
>  drivers/staging/greybus/light.c               |   4 +-
>  drivers/staging/greybus/log.c                 |   9 +-
>  drivers/staging/greybus/loopback.c            |   9 +-
>  drivers/staging/greybus/power_supply.c        |   3 +-
>  drivers/staging/greybus/pwm.c                 |   2 +-
>  drivers/staging/greybus/raw.c                 |   3 +-
>  drivers/staging/greybus/sdio.c                |   2 +-
>  drivers/staging/greybus/spi.c                 |   2 +-
>  drivers/staging/greybus/spilib.c              |   2 +-
>  drivers/staging/greybus/spilib.h              |   2 +-
>  drivers/staging/greybus/tools/loopback_test.c |   2 -
>  drivers/staging/greybus/uart.c                |   2 +-
>  drivers/staging/greybus/usb.c                 |   2 +-
>  drivers/staging/greybus/vibrator.c            |   3 +-
>  .../greybus => include/linux}/greybus.h       |  26 ++---
>  .../linux}/greybus/bundle.h                   |   0
>  .../linux}/greybus/connection.h               |   0
>  .../linux}/greybus/control.h                  |   0
>  .../linux}/greybus/greybus_id.h               |   0
>  .../linux}/greybus/greybus_manifest.h         |   2 +-
>  .../linux}/greybus/greybus_protocols.h        |  48 +-------
>  .../staging => include/linux}/greybus/hd.h    |   2 +-
>  .../linux}/greybus/interface.h                |   2 +-
>  .../linux}/greybus/manifest.h                 |   2 +-
>  .../linux}/greybus/module.h                   |   2 +-
>  .../linux}/greybus/operation.h                |   2 +-
>  .../staging => include/linux}/greybus/svc.h   |   2 +-
>  72 files changed, 225 insertions(+), 532 deletions(-)
>  create mode 100644 drivers/greybus/Kconfig
>  create mode 100644 drivers/greybus/Makefile
>  create mode 100644 drivers/greybus/arpc.h
>  rename drivers/{staging => }/greybus/bundle.c (99%)
>  rename drivers/{staging => }/greybus/connection.c (99%)
>  rename drivers/{staging => }/greybus/control.c (99%)
>  rename drivers/{staging => }/greybus/core.c (99%)
>  rename drivers/{staging => }/greybus/debugfs.c (94%)
>  rename drivers/{staging => }/greybus/es2.c (99%)
>  rename drivers/{staging => }/greybus/greybus_trace.h (99%)
>  rename drivers/{staging => }/greybus/hd.c (96%)
>  rename drivers/{staging => }/greybus/interface.c (99%)
>  rename drivers/{staging => }/greybus/manifest.c (95%)
>  rename drivers/{staging => }/greybus/module.c (99%)
>  rename drivers/{staging => }/greybus/operation.c (99%)
>  rename drivers/{staging => }/greybus/svc.c (99%)
>  rename drivers/{staging => }/greybus/svc_watchdog.c (99%)
>  delete mode 100644 drivers/staging/greybus/arpc.h
>  rename {drivers/staging/greybus => include/linux}/greybus.h (88%)
>  rename {drivers/staging => include/linux}/greybus/bundle.h (100%)
>  rename {drivers/staging => include/linux}/greybus/connection.h (100%)
>  rename {drivers/staging => include/linux}/greybus/control.h (100%)
>  rename {drivers/staging => include/linux}/greybus/greybus_id.h (100%)
>  rename {drivers/staging => include/linux}/greybus/greybus_manifest.h (99%)
>  rename {drivers/staging => include/linux}/greybus/greybus_protocols.h (96%)
>  rename {drivers/staging => include/linux}/greybus/hd.h (98%)
>  rename {drivers/staging => include/linux}/greybus/interface.h (98%)
>  rename {drivers/staging => include/linux}/greybus/manifest.h (87%)
>  rename {drivers/staging => include/linux}/greybus/module.h (94%)
>  rename {drivers/staging => include/linux}/greybus/operation.h (99%)
>  rename {drivers/staging => include/linux}/greybus/svc.h (98%)
> 


  parent reply	other threads:[~2019-08-27 10:43 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-25  5:54 [PATCH 0/9] staging: move greybus core out of staging Greg Kroah-Hartman
2019-08-25  5:54 ` [PATCH 1/9] staging: greybus: fix up SPDX comment in .h files Greg Kroah-Hartman
2019-08-26  5:51   ` Viresh Kumar
2019-08-27  7:58     ` Greg Kroah-Hartman
2019-08-27  9:20       ` Gao Xiang
2019-08-27 15:43         ` Greg Kroah-Hartman
2019-08-27 16:01           ` Gao Xiang
2019-08-25  5:54 ` [PATCH 2/9] staging: greybus: remove license "boilerplate" Greg Kroah-Hartman
2019-08-26  5:52   ` Viresh Kumar
2019-08-27 15:37   ` Mark Greer
2019-08-25  5:54 ` [PATCH 3/9] staging: greybus: hd: Fix up some alignment checkpatch issues Greg Kroah-Hartman
2019-08-26  5:57   ` [greybus-dev] " Viresh Kumar
2019-08-25  5:54 ` [PATCH 4/9] staging: greybus: manifest: " Greg Kroah-Hartman
2019-08-26  6:11   ` [greybus-dev] " Viresh Kumar
2019-08-25  5:54 ` [PATCH 5/9] staging: greybus: log: " Greg Kroah-Hartman
2019-08-26  6:12   ` [greybus-dev] " Viresh Kumar
2019-08-25  5:54 ` [PATCH 6/9] staging: greybus: loopback: " Greg Kroah-Hartman
2019-08-26  6:12   ` [greybus-dev] " Viresh Kumar
2019-08-25  5:54 ` [PATCH 7/9] staging: greybus: move core include files to include/linux/greybus/ Greg Kroah-Hartman
2019-08-26  5:55   ` Viresh Kumar
2019-08-27 15:38   ` Mark Greer
2019-08-25  5:54 ` [PATCH 8/9] staging: greybus: move the greybus core to drivers/greybus Greg Kroah-Hartman
2019-08-26  6:13   ` [greybus-dev] " Viresh Kumar
2019-08-25  5:54 ` [PATCH 9/9] staging: greybus: move es2 to drivers/greybus/ Greg Kroah-Hartman
2019-08-26  6:14   ` [greybus-dev] " Viresh Kumar
2019-08-27 10:43 ` Alex Elder [this message]
2019-08-27 13:36 ` [PATCH 0/9] staging: move greybus core out of staging Dan Carpenter
2019-08-27 13:45   ` Greg Kroah-Hartman
2019-08-27 14:30     ` Rui Miguel Silva
2019-08-27 15:43       ` Greg KH
2019-08-27 15:57         ` Rui Miguel Silva
2019-08-27 16:59           ` Greg KH
2019-08-27 14:37     ` Dan Carpenter
2019-09-01 17:23 ` Pavel Machek
2019-09-02  6:31   ` Greg Kroah-Hartman

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=cfefa25a-9ee5-6011-a41c-ce09dd9608e6@linaro.org \
    --to=elder@linaro.org \
    --cc=devel@driverdev.osuosl.org \
    --cc=elder@kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=greybus-dev@lists.linaro.org \
    --cc=johan@kernel.org \
    --cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).