Commit eac38aed authored by Jeffrey Kardatzke's avatar Jeffrey Kardatzke Committed by Commit Bot

exo: Validate cmd line params for yuv wayland client

The --use-drm parameter is required to allocate buffers that have a
valid bo inside of them which is used by the yuv client.

Bug: None
Test: Ran wayland_yuv_client w/ and w/out --use-drm param
Change-Id: I29027ee4608da05ed82aeb03f6e8b63713df32ff
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1914759Reviewed-by: default avatarDaniele Castagna <dcastagna@chromium.org>
Commit-Queue: Daniele Castagna <dcastagna@chromium.org>
Cr-Commit-Position: refs/heads/master@{#715061}
parent 4a6524ad
...@@ -132,6 +132,12 @@ int main(int argc, char* argv[]) { ...@@ -132,6 +132,12 @@ int main(int argc, char* argv[]) {
if (!params.FromCommandLine(*command_line)) if (!params.FromCommandLine(*command_line))
return 1; return 1;
if (!params.use_drm) {
LOG(ERROR) << "Missing --use-drm parameter which is required for buffer "
"allocation";
return 1;
}
// TODO(dcastagna): Support other YUV formats. // TODO(dcastagna): Support other YUV formats.
params.drm_format = DRM_FORMAT_NV12; params.drm_format = DRM_FORMAT_NV12;
params.bo_usage = params.bo_usage =
...@@ -140,5 +146,5 @@ int main(int argc, char* argv[]) { ...@@ -140,5 +146,5 @@ int main(int argc, char* argv[]) {
base::SingleThreadTaskExecutor main_task_executor(base::MessagePumpType::UI); base::SingleThreadTaskExecutor main_task_executor(base::MessagePumpType::UI);
exo::wayland::clients::YuvClient client; exo::wayland::clients::YuvClient client;
client.Run(params); client.Run(params);
return 1; return 0;
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment