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%)
>
next prev parent reply other threads:[~2019-08-27 10:43 UTC|newest]
Thread overview: 41+ 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 14:30 ` Rui Miguel Silva
2019-08-27 15:43 ` Greg KH
2019-08-27 15:43 ` Greg KH
2019-08-27 15:57 ` Rui Miguel Silva
2019-08-27 15:57 ` Rui Miguel Silva
2019-08-27 16:59 ` Greg KH
2019-08-27 16:59 ` Greg KH
2019-08-27 14:37 ` Dan Carpenter
2019-09-01 17:23 ` Pavel Machek
2019-09-01 17:23 ` Pavel Machek
2019-09-02 6:31 ` Greg Kroah-Hartman
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 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.