Re: [PATCH v4 12/12] staging: most: Documentation: update driver documentation
Hi Chris, We really appreciate your feedback. Thanks for that. For us going the vanilla way roughly means backporting to v4.14: - v5.1-rc5 MOST state (easy/straightforward) - MOST patches from either https://github.com/CogentEmbedded/meta-rcar/ or https://github.com/microchip-ais/meta-medialb-rcar - A number of patches from mld-1.8.0 >From the above list, the last step looks a bit more painful, but we will try to deal with that. If there are any mainline-relevant fixes developed on the way, we hope to find some time to submit them to you and get feedback. Best regards, Eugeniu. On Wed, Apr 17, 2019 at 01:45:41PM +, christian.gr...@microchip.com wrote: > On Mo, 2019-04-15 at 23:21 +0200, Eugeniu Rosca wrote: > > External E-Mail > > > > > > Hello Christian, hello Andrey, > > cc: Vladimir Barinov > > cc: linux-renesas-soc > > > > Our team currently has the requirement of adding MOST support to > > the v4.14-based R-Car3 kernel [1], matching the level of features > > and interfaces of mld-1.8.0 [2] release. > > > > Based on a quick check [3], the mld-1.8.0 release contains 221 non- > > merge > > non-mainline commits. It seems like at least some (most?) of them > > have > > been reworked during upstreaming and are now part of vanilla, thanks > > to > > your efforts. > > > > Since you've actively participated in pushing the out-of-tree drivers > > to mainline, could you please share your gut feeling whether the > > current mainline state of the MOST subsystem matches the mld-1.8.0 > > release in terms of features and interfaces? > > > > No, it doesn't. The version upstream does not have all the bells > and whistles the mld-1.8.0 version has, yet. Focus of the latest > mainline commits was on the driver model and the switch to configfs. > > > At first glance, such mld-1.8.0 functionalities like "flexible PCM > > rate support" [4], as well as a number of dim2 sysfs entries [5-8] > > appear to be missing in v5.1-rc5. Could you please feedback if these > > have been deliberately dropped or didn't make their way for another > > reason? What would be your recommendation between going the mld-1.8.0 > > and going the v5.1-rc5 way for MOST? > > > > The functionalities you're referring to have _not_ intentionally been > dropped. They will find their way into mainline. If there are urgent > feature requests, we could prioritize them. > > Bottom line is, the upstream version is the one you should be using, > as it is going to replace the mld-1.x branch. This is exactly what we > will soon be doing for AGL by the way. > > thanks, > Chris ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH v4 12/12] staging: most: Documentation: update driver documentation
On Mo, 2019-04-15 at 23:21 +0200, Eugeniu Rosca wrote: > External E-Mail > > > Hello Christian, hello Andrey, > cc: Vladimir Barinov > cc: linux-renesas-soc > > Our team currently has the requirement of adding MOST support to > the v4.14-based R-Car3 kernel [1], matching the level of features > and interfaces of mld-1.8.0 [2] release. > > Based on a quick check [3], the mld-1.8.0 release contains 221 non- > merge > non-mainline commits. It seems like at least some (most?) of them > have > been reworked during upstreaming and are now part of vanilla, thanks > to > your efforts. > > Since you've actively participated in pushing the out-of-tree drivers > to mainline, could you please share your gut feeling whether the > current mainline state of the MOST subsystem matches the mld-1.8.0 > release in terms of features and interfaces? > No, it doesn't. The version upstream does not have all the bells and whistles the mld-1.8.0 version has, yet. Focus of the latest mainline commits was on the driver model and the switch to configfs. > At first glance, such mld-1.8.0 functionalities like "flexible PCM > rate support" [4], as well as a number of dim2 sysfs entries [5-8] > appear to be missing in v5.1-rc5. Could you please feedback if these > have been deliberately dropped or didn't make their way for another > reason? What would be your recommendation between going the mld-1.8.0 > and going the v5.1-rc5 way for MOST? > The functionalities you're referring to have _not_ intentionally been dropped. They will find their way into mainline. If there are urgent feature requests, we could prioritize them. Bottom line is, the upstream version is the one you should be using, as it is going to replace the mld-1.x branch. This is exactly what we will soon be doing for AGL by the way. thanks, Chris ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH v4 12/12] staging: most: Documentation: update driver documentation
Hello Christian, hello Andrey, cc: Vladimir Barinov cc: linux-renesas-soc Our team currently has the requirement of adding MOST support to the v4.14-based R-Car3 kernel [1], matching the level of features and interfaces of mld-1.8.0 [2] release. Based on a quick check [3], the mld-1.8.0 release contains 221 non-merge non-mainline commits. It seems like at least some (most?) of them have been reworked during upstreaming and are now part of vanilla, thanks to your efforts. Since you've actively participated in pushing the out-of-tree drivers to mainline, could you please share your gut feeling whether the current mainline state of the MOST subsystem matches the mld-1.8.0 release in terms of features and interfaces? At first glance, such mld-1.8.0 functionalities like "flexible PCM rate support" [4], as well as a number of dim2 sysfs entries [5-8] appear to be missing in v5.1-rc5. Could you please feedback if these have been deliberately dropped or didn't make their way for another reason? What would be your recommendation between going the mld-1.8.0 and going the v5.1-rc5 way for MOST? Best regards, Eugeniu. [1] https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas-bsp.git/ [2] https://github.com/microchip-ais/linux/releases/tag/mld-1.8.0 [3] git rev-list --no-merges --count \ $(git merge-base linux/master mld-1.8.0)..mld-1.8.0 221 [4] https://github.com/microchip-ais/linux/commit/8821bad23ff2 ("staging: most: aim-sound: add flexible PCM rate support") [5] https://github.com/microchip-ais/linux/commit/5030f77717a1d7 ("staging: most: dim2: add sysfs property dci/node_position") [6] https://github.com/microchip-ais/linux/commit/3a840d7b504552 ("staging: most: dim2: add sysfs property dci/node_address") [7] https://github.com/microchip-ais/linux/commit/0494ccb4bc5f39 ("staging: most: dim2: add sysfs property dci/ni_state") [8] https://github.com/microchip-ais/linux/commit/3198a5c5f0663 ("staging: most: dim2: add sysfs property dci/mep_eui48") ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH v4 12/12] staging: most: Documentation: update driver documentation
This patch updates the driver documentation files to reflect the latest changes regarding configfs. Signed-off-by: Christian Gromm v2: - changed kernel version to 5.2 v3: v4: Signed-off-by: Christian Gromm --- .../most/Documentation/ABI/configfs-most.txt | 204 + .../staging/most/Documentation/driver_usage.txt| 131 +++-- 2 files changed, 284 insertions(+), 51 deletions(-) create mode 100644 drivers/staging/most/Documentation/ABI/configfs-most.txt diff --git a/drivers/staging/most/Documentation/ABI/configfs-most.txt b/drivers/staging/most/Documentation/ABI/configfs-most.txt new file mode 100644 index 000..25b3e18 --- /dev/null +++ b/drivers/staging/most/Documentation/ABI/configfs-most.txt @@ -0,0 +1,204 @@ +What: /sys/kernel/config/most_ +Date: March 8, 2019 +KernelVersion: 5.2 +Description: Interface is used to configure and connect device channels + to component drivers. + + Attributes are visible only when configfs is mounted. To mount + configfs in /sys/kernel/config directory use: + # mount -t configfs none /sys/kernel/config/ + + +What: /sys/kernel/config/most_cdev/ +Date: March 8, 2019 +KernelVersion: 5.2 +Description: + The attributes: + + buffer_size configure the buffer size for this channel + + subbuffer_size configure the sub-buffer size for this channel + (needed for synchronous and isochrnous data) + + + num_buffers configure number of buffers used for this + channel + + datatypeconfigure type of data that will travel over + this channel + + direction configure whether this link will be an input + or output + + dbr_sizeconfigure DBR data buffer size (this is used + for MediaLB communiction only) + + packets_per_xact + configure the number of packets that will be + collected from the network before being + transmitted via USB (this is used for USB + communiction only) + + device name of the device the link is to be attached to + + channel name of the channel the link is to be attached to + + comp_params pass parameters needed by some components + + create_link write '1' to this attribute to trigger the + creation of the link. In case of speculative + configuration, the creation is post-poned until + a physical device is being attached to the bus. + + destroy_linkwrite '1' to this attribute to destroy an + active link + +What: /sys/kernel/config/most_video/ +Date: March 8, 2019 +KernelVersion: 5.2 +Description: + The attributes: + + buffer_size configure the buffer size for this channel + + subbuffer_size configure the sub-buffer size for this channel + (needed for synchronous and isochrnous data) + + + num_buffers configure number of buffers used for this + channel + + datatypeconfigure type of data that will travel over + this channel + + direction configure whether this link will be an input + or output + + dbr_sizeconfigure DBR data buffer size (this is used + for MediaLB communiction only) + + packets_per_xact + configure the number of packets that will be + collected from the network before being + transmitted via USB (this is used for USB + communiction only) + + device name of the device the link is to be attached to + + channel name of the channel the link is to be attached to + + comp_params pass parameters needed by some components + + create_link write '1' to this attribute to trigger the + creation of the link. In case of speculative + configuration, the creation is post-poned until + a physical device is being attached to the bus. + + destroy_linkwrite '1' to this attribute to destroy an + active link + +What: /sys/kernel/config/most_net/ +Date: March