gst-plugins-imsdk: Enable qmmfsrc plugin#1558
gst-plugins-imsdk: Enable qmmfsrc plugin#1558ricardosalveti merged 2 commits intoqualcomm-linux:masterfrom
Conversation
|
This does too much in a single commit, at a minimum the bbappend addition needs to be its own commit. |
koenkooi
left a comment
There was a problem hiding this comment.
This does too much in a single commit, at a minimum the bbappend addition needs to be its own commit.
e8772bc to
33ccd39
Compare
Agreed, created separate commit for bbappend |
lumag
left a comment
There was a problem hiding this comment.
Write commit messages as text rather than bullet lists
ac25a57 to
1101ee5
Compare
lumag
left a comment
There was a problem hiding this comment.
Added qmmfsrc to PACKAGECONFIG.
Excluded qmmfsrc from the opensource package group.
Don't describe what was done. Describe why it is done.
|
In your first commit you are not really enabling the plugin, you are adding the support to allow qmmfsrc to be built, so say that instead. Also, please explain what qmmfsrc actually does, in a short summary, to justify having it here. In your second commit you are not really enabling in meta-openembedded, you can just say that you are enabling such PACKAGECONFIG by default, saying why and that it is done under dynamic-layers due the protobuf dependency. |
|
It's not ready until we merge camera service |
|
updated the commit message please help to review. |
|
|
||
| PACKAGECONFIG[messaging] = "-DENABLE_GST_MESSAGING_PLUGINS=1, -DENABLE_GST_MESSAGING_PLUGINS=0, librdkafka mosquitto" | ||
| PACKAGECONFIG[ml] = "-DENABLE_GST_ML_PLUGINS=1, -DENABLE_GST_ML_PLUGINS=0, cairo json-glib opencv qairt-sdk, qairt-sdk" | ||
| PACKAGECONFIG[qmmfsrc] = "-DENABLE_GST_PLUGIN_QMMFSRC=1 -DVHDR_MODES_ENABLE=ON -DEIS_MODES_ENABLE=ON, -DENABLE_GST_PLUGIN_QMMFSRC=0 -DVHDR_MODES_ENABLE=OFF -DEIS_MODES_ENABLE=OFF, camera-service" |
There was a problem hiding this comment.
I don't see any EIS_MODES_ENABLE in gst-plugins-imsdk code. Where is it used?
There was a problem hiding this comment.
My bad, recent code changes we changed it to get the info from static metadata. I will remove this flag
Test run workflowTest jobs for commit 5485dbb
All jobs summary
|
Test Results 46 files + 22 193 suites +169 4h 2m 57s ⏱️ + 2h 45m 9s For more details on these failures, see this check. Results for commit beff847. ± Comparison against base commit b062add. ♻️ This comment has been updated with latest results. |
|
Second commit says:
Please fix. And while there, please also include the reason why qtiqmmfsrc is in IMSDK_OSS_METAPKG_EXCLUDE |
Nice catch, not obvious from the github interface. |
ricardosalveti
left a comment
There was a problem hiding this comment.
Please fix your author line.
Add the support so that qmmfsrc GStreamer source plugin can be built as part of gst-plugins-imsdk. qmmfsrc helps to provide a high-performance camera capture source exposing advanced Qualcomm camera features. Excluded qmmfsrc from the opensource package group, since it depends on camera-service which is not present in opensource Signed-off-by: Shyam Komakula <skomakul@qti.qualcomm.com>
Enable the qmmfsrc PACKAGECONFIG by default so that platforms requiring the camera functionality gets the plugin built into the build. This is done under dynamic-layers due to the protobuf dependency. Signed-off-by: Shyam Komakula <skomakul@qti.qualcomm.com>
updated the author and added reason for excluding the qmmfsrc from opensource packaging. Please help to review. |
Test run workflowTest jobs for commit beff847
All jobs summary
|
3b2c2c8
into
qualcomm-linux:master
Added qmmfsrc to PACKAGECONFIG
This allows the plugin to be enabled when building gst-plugins-imsdk.
Excluded qmmfsrc from the opensource package group
The plugin relies on components that are not part of the public opensource deliverables.
Added qmmfsrc under dynamic-layers
The plugin depends on the client component, which internally depends on protobuf coming from the meta-openembedded layer.
Adding it to dynamic layers ensures correct layer ordering and dependency availability.