Re: [PATCH v3 0/2] greybus: Add BeaglePlay Greybus Driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 19:34-20230831, Ayush Singh wrote:
> This driver can be used in any Greybus setup where SVC and Linux Host
> are connected over UART.
> 
> This driver has been tested on BeaglePlay by BeagleBoard.org. It
> communicates with BeaglePlay CC1352 co-processor which serves as Greybus
> SVC. This replaces the old setup with bcfserial, wpanusb and GBridge. This
> driver also contains async HDLC code since communication with SVC take
> place over UART using HDLC.
> 
> This patchset also introduces a compatible property for the UART that is
> connected to CC1352 in BeaglePlay. I am not quite sure if it needs its
> own DT Schema file under `Documentation/devicetree/bindings/serial`.
> 
> This driver has been created as a part of my Google Summer of Code 2023.
> For more information, take a look at my blog.
> 
> This patchset has been tested over `next-20230825`.
> 
> My GSoC23 Blog: https://programmershideaway.xyz/tags/gsoc23/
> Zephyr App: https://git.beagleboard.org/gsoc/greybus/cc1352-firmware
> GitHub Branch: https://github.com/Ayush1325/linux/tree/gb-beagleplay
> Video Demo: https://youtu.be/GVuIB7i5pjk
> 
> This the v3 of this patch
> v2 -> v3:
> - Move gb-beagleplay out of staging

You missed the comments I provided you offline:
a) The sequence of patches should be:
Device tree binding yaml binding
driver changes
dts update
b) subject line: follow the convention of the files being touched - do git log --oneline file_name to see convention

I am not cced on the patches, so I will repeat what I had commented
previously on gsoc channel:
c) commit message should have more detailed commit message to explain to the reviewer why the change is needed etc..

You also should CC relevant mailing lists including lkml devicetree etc.

> 
> v1 -> v2:
> - Combine the driver into a single file
> - Remove redundant code
> - Fix Checkpatch complaints
> - Other suggested changes
> 
> Ayush Singh (2):
>   dts: Add beaglecc1352 to devicetree
>   greybus: Add BeaglePlay Linux Driver
> 
>  MAINTAINERS                                   |   5 +
>  .../arm64/boot/dts/ti/k3-am625-beagleplay.dts |   4 +
>  drivers/greybus/Kconfig                       |  10 +
>  drivers/greybus/Makefile                      |   3 +-
>  drivers/greybus/gb-beagleplay.c               | 494 ++++++++++++++++++
>  5 files changed, 515 insertions(+), 1 deletion(-)
>  create mode 100644 drivers/greybus/gb-beagleplay.c
> 
> -- 
> 2.41.0
> 

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
_______________________________________________
greybus-dev mailing list -- greybus-dev@xxxxxxxxxxxxxxxx
To unsubscribe send an email to greybus-dev-leave@xxxxxxxxxxxxxxxx



[Index of Archives]     [Asterisk App Development]     [PJ SIP]     [Gnu Gatekeeper]     [IETF Sipping]     [Info Cyrus]     [ALSA User]     [Fedora Linux Users]     [Linux SCTP]     [DCCP]     [Gimp]     [Yosemite News]     [Deep Creek Hot Springs]     [Yosemite Campsites]     [ISDN Cause Codes]     [Asterisk Books]

  Powered by Linux