Add support for multiple V4L2 video devices of the same type.

media/gpu/v4l2_video_encode_accelerator.cc

Index: media/gpu/v4l2_video_encode_accelerator.cc
diff --git a/media/gpu/v4l2_video_encode_accelerator.cc b/media/gpu/v4l2_video_encode_accelerator.cc
index e546c0c75bf93ed7d8546f1d1c534da4267d38c3..f1f9a954029e2bc096cbfa854e69e9746897a6fc 100644
--- a/media/gpu/v4l2_video_encode_accelerator.cc
+++ b/media/gpu/v4l2_video_encode_accelerator.cc
@@ -153,6 +153,21 @@ bool V4L2VideoEncodeAccelerator::Initialize(VideoPixelFormat input_format,
   DCHECK(child_task_runner_->BelongsToCurrentThread());
   DCHECK_EQ(encoder_state_, kUninitialized);
 
+  output_format_fourcc_ =
+      V4L2Device::VideoCodecProfileToV4L2PixFmt(output_profile, false);
+  if (!output_format_fourcc_) {
+    LOG(ERROR) << "Initialize(): invalid output_profile="
+               << GetProfileName(output_profile);
+    return false;
+  }
+
+  if (!device_->Open(output_format_fourcc_)) {
+    DVLOG(1) << "Failed to open device for profile="
+             << GetProfileName(output_profile) << ", fourcc=" << std::hex
+             << output_format_fourcc_;
+    return false;
+  }
+
   struct v4l2_capability caps;
   memset(&caps, 0, sizeof(caps));
   const __u32 kCapsRequired = V4L2_CAP_VIDEO_M2M_MPLANE | V4L2_CAP_STREAMING;
@@ -173,7 +188,7 @@ bool V4L2VideoEncodeAccelerator::Initialize(VideoPixelFormat input_format,
              << VideoPixelFormatToString(device_input_format_);
 
     scoped_refptr<V4L2Device> device =
-        V4L2Device::Create(V4L2Device::kImageProcessor);
+        V4L2Device::Create(V4L2Device::Type::kImageProcessor);
     image_processor_.reset(new V4L2ImageProcessor(device));
 
     // Convert from input_format to device_input_format_, keeping the size
@@ -349,41 +364,12 @@ void V4L2VideoEncodeAccelerator::Destroy() {
 
 VideoEncodeAccelerator::SupportedProfiles
 V4L2VideoEncodeAccelerator::GetSupportedProfiles() {
-  SupportedProfiles profiles;
-  SupportedProfile profile;
-  profile.max_framerate_numerator = 30;
-  profile.max_framerate_denominator = 1;
-
-  gfx::Size min_resolution;
-  v4l2_fmtdesc fmtdesc;
-  memset(&fmtdesc, 0, sizeof(fmtdesc));
-  fmtdesc.type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
-  for (; device_->Ioctl(VIDIOC_ENUM_FMT, &fmtdesc) == 0; ++fmtdesc.index) {
-    device_->GetSupportedResolution(fmtdesc.pixelformat, &min_resolution,
-                                    &profile.max_resolution);
-    switch (fmtdesc.pixelformat) {
-      case V4L2_PIX_FMT_H264:
-        profile.profile = H264PROFILE_MAIN;
-        profiles.push_back(profile);
-        break;
-      case V4L2_PIX_FMT_VP8:
-        profile.profile = VP8PROFILE_ANY;
-        profiles.push_back(profile);
-        break;
-      case V4L2_PIX_FMT_VP9:
-        profile.profile = VP9PROFILE_PROFILE0;
-        profiles.push_back(profile);
-        profile.profile = VP9PROFILE_PROFILE1;
-        profiles.push_back(profile);
-        profile.profile = VP9PROFILE_PROFILE2;
-        profiles.push_back(profile);
-        profile.profile = VP9PROFILE_PROFILE3;
-        profiles.push_back(profile);
-        break;
-    }
-  }
+  scoped_refptr<V4L2Device> device =
+      V4L2Device::Create(V4L2Device::Type::kEncoder);
+  if (!device)
+    return SupportedProfiles();
 
-  return profiles;
+  return device->GetSupportedEncodeProfiles();
 }
 
 void V4L2VideoEncodeAccelerator::FrameProcessed(bool force_keyframe,
@@ -991,13 +977,6 @@ bool V4L2VideoEncodeAccelerator::SetOutputFormat(
   DCHECK(!input_streamon_);
   DCHECK(!output_streamon_);
 
-  output_format_fourcc_ =
-      V4L2Device::VideoCodecProfileToV4L2PixFmt(output_profile, false);
-  if (!output_format_fourcc_) {
-    LOG(ERROR) << "Initialize(): invalid output_profile=" << output_profile;
-    return false;
-  }
-
   output_buffer_byte_size_ = kOutputBufferSize;
 
   struct v4l2_format format;
@@ -1315,6 +1294,11 @@ void V4L2VideoEncodeAccelerator::DestroyInputBuffers() {
   DCHECK(child_task_runner_->BelongsToCurrentThread());
   DCHECK(!input_streamon_);
 
+  free_input_buffers_.clear();
+
+  if (input_buffer_map_.empty())
+    return;
+
   struct v4l2_requestbuffers reqbufs;
   memset(&reqbufs, 0, sizeof(reqbufs));
   reqbufs.count = 0;
@@ -1323,7 +1307,6 @@ void V4L2VideoEncodeAccelerator::DestroyInputBuffers() {
   IOCTL_OR_LOG_ERROR(VIDIOC_REQBUFS, &reqbufs);
 
   input_buffer_map_.clear();
-  free_input_buffers_.clear();
 }
 
 void V4L2VideoEncodeAccelerator::DestroyOutputBuffers() {
@@ -1331,6 +1314,11 @@ void V4L2VideoEncodeAccelerator::DestroyOutputBuffers() {
   DCHECK(child_task_runner_->BelongsToCurrentThread());
   DCHECK(!output_streamon_);
 
+  free_output_buffers_.clear();
+
+  if (output_buffer_map_.empty())
+    return;
+
   for (size_t i = 0; i < output_buffer_map_.size(); ++i) {
     if (output_buffer_map_[i].address != NULL)
       device_->Munmap(output_buffer_map_[i].address,
@@ -1345,7 +1333,6 @@ void V4L2VideoEncodeAccelerator::DestroyOutputBuffers() {
   IOCTL_OR_LOG_ERROR(VIDIOC_REQBUFS, &reqbufs);
 
   output_buffer_map_.clear();
-  free_output_buffers_.clear();
 }
 
 }  // namespace media