On Wed, Aug 30, 2023 at 06:06:54PM +0530, Ayush Singh wrote:
On 8/30/23 17:58, Greg KH wrote:
On Wed, Aug 30, 2023 at 05:54:27PM +0530, Ayush Singh wrote:
This is Serdev driver that communicates with Greybus SVC connected over UART. This driver has only been tested on BeaglePlay (with CC1352 in BeaglePlay serving as SVC).
This driver is responsible for the following:
- Start SVC (CC1352) on driver load.
- Send/Receive Greybus messages to/from CC1352 using HDLC over UART.
- Print Logs from CC1352.
- Stop SVC (CC1352) on driver load.
This patch does the following:
- Add Driver
- Add Kconfig option to enable the driver
- Update Makefile to build the driver
- Update MAINTAINERS
Signed-off-by: Ayush Singh ayushdevel1325@gmail.com
MAINTAINERS | 5 + drivers/staging/greybus/Kconfig | 9 + drivers/staging/greybus/Makefile | 3 + drivers/staging/greybus/gb-beagleplay.c | 493 ++++++++++++++++++++++++
Why is this in drivers/staging/ and not in drivers/greybus/ ?
Well, there are a few reasons I can give:
- It has not been thoroughly tested. I have only tested it in my personal
setup.
- Since mikrobus code has not been merged, running a normal beagleconnect
freedom as the greybus node is not possible.
What is "mikrobus"?
- I thought new and experimental drivers should go into staging?
staging is only for code that is not ready to be in the real portion of the kernel and it comes with a list of what needs to be done to get it out of staging.
For this, new driver, it should just go into the real portion of the kernel as obviously you've tested it and it works :)
thanks,
greg k-h