Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6152a309 authored by Biswarup Pal's avatar Biswarup Pal
Browse files

Add frame rate and sensor orientation for virtual camera 

Test: atest CtsVirtualDevicesCameraTestCases
Test: atest virtual_camera_tests
Bug: 310857519
Change-Id: I0dc882f86881f5aaaa6501fdc3fd2f5d64fe2bc0
parent d4cc0179

services/camera/virtualcamera/VirtualCameraDevice.cc

+53 −59
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2023 The Android Open Source Project

 * Copyright 2023 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.
@@ -28,6 +28,7 @@ 


#include "VirtualCameraSession.h"

#include "aidl/android/companion/virtualcamera/SupportedStreamConfiguration.h"

#include "aidl/android/companion/virtualcamera/VirtualCameraConfiguration.h"

#include "aidl/android/hardware/camera/common/Status.h"

#include "aidl/android/hardware/camera/device/CameraMetadata.h"

#include "aidl/android/hardware/camera/device/StreamConfiguration.h"
@@ -44,7 +45,9 @@ namespace virtualcamera { 


using ::aidl::android::companion::virtualcamera::Format;

using ::aidl::android::companion::virtualcamera::IVirtualCameraCallback;

using ::aidl::android::companion::virtualcamera::SensorOrientation;

using ::aidl::android::companion::virtualcamera::SupportedStreamConfiguration;

using ::aidl::android::companion::virtualcamera::VirtualCameraConfiguration;

using ::aidl::android::hardware::camera::common::CameraResourceCost;

using ::aidl::android::hardware::camera::common::Status;

using ::aidl::android::hardware::camera::device::CameraMetadata;
@@ -64,11 +67,15 @@ using namespace std::chrono_literals; 
// Prefix of camera name - "device@1.1/virtual/{numerical_id}"

const char* kDevicePathPrefix = "device@1.1/virtual/";



constexpr std::chrono::nanoseconds kMinFrameDuration30Fps = 1s / 30;

constexpr int32_t kMaxJpegSize = 3 * 1024 * 1024 /*3MiB*/;



constexpr MetadataBuilder::ControlRegion kDefaultEmptyControlRegion{};



const std::array<int32_t, 3> kOutputFormats{

    ANDROID_SCALER_AVAILABLE_FORMATS_IMPLEMENTATION_DEFINED,

    ANDROID_SCALER_AVAILABLE_FORMATS_YCbCr_420_888,

    ANDROID_SCALER_AVAILABLE_FORMATS_BLOB};



struct Resolution {

  Resolution(const int w, const int h) : width(w), height(h) {

  }
@@ -103,24 +110,34 @@ std::optional<Resolution> getMaxResolution( 
  return Resolution(itMax->width, itMax->height);

}



std::set<Resolution> getUniqueResolutions(

// Returns a map of unique resolution to maximum maxFps for all streams with

// that resolution.

std::map<Resolution, int> getResolutionToMaxFpsMap(

    const std::vector<SupportedStreamConfiguration>& configs) {

  std::set<Resolution> uniqueResolutions;

  std::transform(configs.begin(), configs.end(),

                 std::inserter(uniqueResolutions, uniqueResolutions.begin()),

                 [](const SupportedStreamConfiguration& config) {

                   return Resolution(config.width, config.height);

                 });

  return uniqueResolutions;

  std::map<Resolution, int> resolutionToMaxFpsMap;



  for (const SupportedStreamConfiguration& config : configs) {

    Resolution resolution(config.width, config.height);

    if (resolutionToMaxFpsMap.find(resolution) == resolutionToMaxFpsMap.end()) {

      resolutionToMaxFpsMap[resolution] = config.maxFps;

    } else {

      int currentMaxFps = resolutionToMaxFpsMap[resolution];

      resolutionToMaxFpsMap[resolution] = std::max(currentMaxFps, config.maxFps);

    }

  }



  return resolutionToMaxFpsMap;

}



// TODO(b/301023410) - Populate camera characteristics according to camera configuration.

std::optional<CameraMetadata> initCameraCharacteristics(

    const std::vector<SupportedStreamConfiguration>& supportedInputConfig) {

    const std::vector<SupportedStreamConfiguration>& supportedInputConfig,

    const SensorOrientation sensorOrientation) {

  if (!std::all_of(supportedInputConfig.begin(), supportedInputConfig.end(),

                   [](const SupportedStreamConfiguration& config) {

                     return isFormatSupportedForInput(

                         config.width, config.height, config.pixelFormat);

                         config.width, config.height, config.pixelFormat,

                         config.maxFps);

                   })) {

    ALOGE("%s: input configuration contains unsupported format", __func__);

    return std::nullopt;
@@ -132,7 +149,7 @@ std::optional<CameraMetadata> initCameraCharacteristics( 
              ANDROID_INFO_SUPPORTED_HARDWARE_LEVEL_EXTERNAL)

          .setFlashAvailable(false)

          .setLensFacing(ANDROID_LENS_FACING_EXTERNAL)

          .setSensorOrientation(0)

          .setSensorOrientation(static_cast<int32_t>(sensorOrientation))

          .setAvailableFaceDetectModes({ANDROID_STATISTICS_FACE_DETECT_MODE_OFF})

          .setAvailableMaxDigitalZoom(1.0)

          .setControlAvailableModes({ANDROID_CONTROL_MODE_AUTO})
@@ -166,45 +183,24 @@ std::optional<CameraMetadata> initCameraCharacteristics( 
  // TODO(b/301023410) Add also all "standard" resolutions we can rescale the

  // streams to (all standard resolutions with same aspect ratio).



  // Add IMPLEMENTATION_DEFINED format for all supported input resolutions.

  std::set<Resolution> uniqueResolutions =

      getUniqueResolutions(supportedInputConfig);

  std::transform(

      uniqueResolutions.begin(), uniqueResolutions.end(),

      std::back_inserter(outputConfigurations),

      [](const Resolution& resolution) {

        return MetadataBuilder::StreamConfiguration{

            .width = resolution.width,

            .height = resolution.height,

            .format = ANDROID_SCALER_AVAILABLE_FORMATS_IMPLEMENTATION_DEFINED,

            .minFrameDuration = kMinFrameDuration30Fps,

            .minStallDuration = 0s};

      });

  std::map<Resolution, int> resolutionToMaxFpsMap =

      getResolutionToMaxFpsMap(supportedInputConfig);



  // Add all supported configuration with explicit pixel format.

  std::transform(supportedInputConfig.begin(), supportedInputConfig.end(),

                 std::back_inserter(outputConfigurations),

                 [](const SupportedStreamConfiguration& config) {

                   return MetadataBuilder::StreamConfiguration{

                       .width = config.width,

                       .height = config.height,

                       .format = static_cast<int>(config.pixelFormat),

                       .minFrameDuration = kMinFrameDuration30Fps,

                       .minStallDuration = 0s};

                 });



  // TODO(b/301023410) We currently don't support rescaling for still capture,

  // so only announce BLOB support for formats exactly matching the input.

  std::transform(uniqueResolutions.begin(), uniqueResolutions.end(),

                 std::back_inserter(outputConfigurations),

                 [](const Resolution& resolution) {

  // Add configurations for all unique input resolutions and output formats.

  for (int32_t format : kOutputFormats) {

    std::transform(

        resolutionToMaxFpsMap.begin(), resolutionToMaxFpsMap.end(),

        std::back_inserter(outputConfigurations), [format](const auto& entry) {

          Resolution resolution = entry.first;

          int maxFps = entry.second;

          return MetadataBuilder::StreamConfiguration{

              .width = resolution.width,

              .height = resolution.height,

                       .format = ANDROID_SCALER_AVAILABLE_FORMATS_BLOB,

                       .minFrameDuration = kMinFrameDuration30Fps,

              .format = format,

              .minFrameDuration = std::chrono::nanoseconds(1s) / maxFps,

              .minStallDuration = 0s};

        });

  }



  ALOGV("Adding %zu output configurations", outputConfigurations.size());

  builder.setAvailableOutputStreamConfigurations(outputConfigurations);
@@ -221,14 +217,12 @@ std::optional<CameraMetadata> initCameraCharacteristics( 
}  // namespace



VirtualCameraDevice::VirtualCameraDevice(

    const uint32_t cameraId,

    const std::vector<SupportedStreamConfiguration>& supportedInputConfig,

    std::shared_ptr<IVirtualCameraCallback> virtualCameraClientCallback)

    const uint32_t cameraId, const VirtualCameraConfiguration& configuration)

    : mCameraId(cameraId),

      mVirtualCameraClientCallback(virtualCameraClientCallback),

      mSupportedInputConfigurations(supportedInputConfig) {

  std::optional<CameraMetadata> metadata =

      initCameraCharacteristics(mSupportedInputConfigurations);

      mVirtualCameraClientCallback(configuration.virtualCameraCallback),

      mSupportedInputConfigurations(configuration.supportedStreamConfigs) {

  std::optional<CameraMetadata> metadata = initCameraCharacteristics(

      mSupportedInputConfigurations, configuration.sensorOrientation);

  if (metadata.has_value()) {

    mCameraCharacteristics = *metadata;

  } else {

services/camera/virtualcamera/VirtualCameraDevice.h

+4 −7
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2023 The Android Open Source Project

 * Copyright 2023 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.
@@ -22,6 +22,7 @@ 


#include "aidl/android/companion/virtualcamera/IVirtualCameraCallback.h"

#include "aidl/android/companion/virtualcamera/SupportedStreamConfiguration.h"

#include "aidl/android/companion/virtualcamera/VirtualCameraConfiguration.h"

#include "aidl/android/hardware/camera/device/BnCameraDevice.h"



namespace android {
@@ -35,12 +36,8 @@ class VirtualCameraDevice 
 public:

  explicit VirtualCameraDevice(

      uint32_t cameraId,

      const std::vector<

          aidl::android::companion::virtualcamera::SupportedStreamConfiguration>&

          supportedInputConfig,

      std::shared_ptr<

          ::aidl::android::companion::virtualcamera::IVirtualCameraCallback>

          virtualCameraClientCallback = nullptr);

      const aidl::android::companion::virtualcamera::VirtualCameraConfiguration&

          configuration);



  virtual ~VirtualCameraDevice() override = default;

services/camera/virtualcamera/VirtualCameraProvider.cc

+5 −7
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2023 The Android Open Source Project

 * Copyright 2023 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.
@@ -33,8 +33,7 @@ namespace android { 
namespace companion {

namespace virtualcamera {



using ::aidl::android::companion::virtualcamera::IVirtualCameraCallback;

using ::aidl::android::companion::virtualcamera::SupportedStreamConfiguration;

using ::aidl::android::companion::virtualcamera::VirtualCameraConfiguration;

using ::aidl::android::hardware::camera::common::CameraDeviceStatus;

using ::aidl::android::hardware::camera::common::Status;

using ::aidl::android::hardware::camera::common::VendorTagSection;
@@ -155,10 +154,9 @@ ndk::ScopedAStatus VirtualCameraProvider::isConcurrentStreamCombinationSupported 
}



std::shared_ptr<VirtualCameraDevice> VirtualCameraProvider::createCamera(

    const std::vector<SupportedStreamConfiguration>& supportedInputConfig,

    std::shared_ptr<IVirtualCameraCallback> virtualCameraClientCallback) {

  auto camera = ndk::SharedRefBase::make<VirtualCameraDevice>(

      sNextId++, supportedInputConfig, virtualCameraClientCallback);

    const VirtualCameraConfiguration& configuration) {

  auto camera =

      ndk::SharedRefBase::make<VirtualCameraDevice>(sNextId++, configuration);

  std::shared_ptr<ICameraProviderCallback> callback;

  {

    const std::lock_guard<std::mutex> lock(mLock);

services/camera/virtualcamera/VirtualCameraProvider.h

+4 −9
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2023 The Android Open Source Project

 * Copyright 2023 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.
@@ -74,14 +74,9 @@ class VirtualCameraProvider 


  // Create new virtual camera devices

  // Returns nullptr if creation was not successful.

  //

  // TODO(b/301023410) - Add camera configuration.

  std::shared_ptr<VirtualCameraDevice> createCamera(

      const std::vector<

          aidl::android::companion::virtualcamera::SupportedStreamConfiguration>&

          supportedInputConfig,

      std::shared_ptr<aidl::android::companion::virtualcamera::IVirtualCameraCallback>

          virtualCameraClientCallback = nullptr);

      const aidl::android::companion::virtualcamera::VirtualCameraConfiguration&

          configuration);



  std::shared_ptr<VirtualCameraDevice> getCamera(const std::string& name);
@@ -105,4 +100,4 @@ class VirtualCameraProvider 
}  // namespace companion

}  // namespace android



#endif  // ANDROID_SERVICES_VIRTUAL_CAMERA_VIRTUALCAMERAPROVIDER_H
 
#endif  // ANDROID_COMPANION_VIRTUALCAMERA_VIRTUALCAMERAPROVIDER_H

services/camera/virtualcamera/VirtualCameraService.cc

+18 −7
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2023 The Android Open Source Project

 * Copyright 2023 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.
@@ -41,6 +41,7 @@ namespace companion { 
namespace virtualcamera {



using ::aidl::android::companion::virtualcamera::Format;

using ::aidl::android::companion::virtualcamera::SensorOrientation;

using ::aidl::android::companion::virtualcamera::SupportedStreamConfiguration;

using ::aidl::android::companion::virtualcamera::VirtualCameraConfiguration;
@@ -48,6 +49,7 @@ namespace { 


constexpr int kVgaWidth = 640;

constexpr int kVgaHeight = 480;

constexpr int kMaxFps = 60;

constexpr char kEnableTestCameraCmd[] = "enable_test_camera";

constexpr char kDisableTestCameraCmd[] = "disable_test_camera";

constexpr char kShellCmdHelp[] = R"(
@@ -69,13 +71,22 @@ ndk::ScopedAStatus validateConfiguration( 
  for (const SupportedStreamConfiguration& config :

       configuration.supportedStreamConfigs) {

    if (!isFormatSupportedForInput(config.width, config.height,

                                   config.pixelFormat)) {

                                   config.pixelFormat, config.maxFps)) {

      ALOGE("%s: Requested unsupported input format: %d x %d (%d)", __func__,

            config.width, config.height, static_cast<int>(config.pixelFormat));

      return ndk::ScopedAStatus::fromServiceSpecificError(

          Status::EX_ILLEGAL_ARGUMENT);

    }

  }



  if (configuration.sensorOrientation != SensorOrientation::ORIENTATION_0 &&

      configuration.sensorOrientation != SensorOrientation::ORIENTATION_90 &&

      configuration.sensorOrientation != SensorOrientation::ORIENTATION_180 &&

      configuration.sensorOrientation != SensorOrientation::ORIENTATION_270) {

    return ndk::ScopedAStatus::fromServiceSpecificError(

        Status::EX_ILLEGAL_ARGUMENT);

  }



  return ndk::ScopedAStatus::ok();

}
@@ -121,10 +132,8 @@ ndk::ScopedAStatus VirtualCameraService::registerCamera( 
    return ndk::ScopedAStatus::ok();

  }



  // TODO(b/301023410) Validate configuration and pass it to the camera.

  std::shared_ptr<VirtualCameraDevice> camera =

      mVirtualCameraProvider->createCamera(configuration.supportedStreamConfigs,

                                           configuration.virtualCameraCallback);

      mVirtualCameraProvider->createCamera(configuration);

  if (camera == nullptr) {

    ALOGE("Failed to create camera for binder token 0x%" PRIxPTR,

          reinterpret_cast<uintptr_t>(token.get()));
@@ -241,8 +250,10 @@ void VirtualCameraService::enableTestCameraCmd(const int out, const int err) { 


  bool ret;

  VirtualCameraConfiguration configuration;

  configuration.supportedStreamConfigs.push_back(

      {.width = kVgaWidth, .height = kVgaHeight, Format::YUV_420_888});

  configuration.supportedStreamConfigs.push_back({.width = kVgaWidth,

                                                  .height = kVgaHeight,

                                                  Format::YUV_420_888,

                                                  .maxFps = kMaxFps});

  registerCamera(mTestCameraToken, configuration, &ret);

  if (ret) {

    dprintf(out, "Successfully registered test camera %s",