Merge "sdm: Add pipe rect number in mdp structure."
diff --git a/sdm/include/core/layer_stack.h b/sdm/include/core/layer_stack.h
index 6ee2e75..f12e5c1 100644
--- a/sdm/include/core/layer_stack.h
+++ b/sdm/include/core/layer_stack.h
@@ -1,5 +1,5 @@
/*
-* Copyright (c) 2014 - 2016, The Linux Foundation. All rights reserved.
+* Copyright (c) 2014 - 2017, The Linux Foundation. All rights reserved.
*
* Redistribution and use in source and binary forms, with or without modification, are permitted
* provided that the following conditions are met:
@@ -189,6 +189,7 @@
struct {
uint32_t tone_map : 1; //!< This flag will be set by SDM when the layer needs tone map
uint32_t secure: 1; //!< This flag will be set by SDM when the layer must be secure
+ uint32_t flip_buffer: 1; //!< This flag will be set by SDM when the layer needs FBT flip
};
uint32_t request_flags = 0; //!< For initialization purpose only.
//!< Shall not be refered directly.
diff --git a/sdm/include/private/extension_interface.h b/sdm/include/private/extension_interface.h
index a10a2c8..2e5bd49 100644
--- a/sdm/include/private/extension_interface.h
+++ b/sdm/include/private/extension_interface.h
@@ -69,9 +69,9 @@
virtual DisplayError DestroyStrategyExtn(StrategyInterface *interface) = 0;
virtual DisplayError CreateResourceExtn(const HWResourceInfo &hw_resource_info,
- ResourceInterface **interface,
BufferAllocator *buffer_allocator,
- BufferSyncHandler *buffer_sync_handler) = 0;
+ BufferSyncHandler *buffer_sync_handler,
+ ResourceInterface **interface) = 0;
virtual DisplayError DestroyResourceExtn(ResourceInterface *interface) = 0;
virtual DisplayError CreateDppsControlExtn(DppsControlInterface **dpps_control_interface,
SocketHandler *socket_handler) = 0;
diff --git a/sdm/include/private/hw_info_types.h b/sdm/include/private/hw_info_types.h
index 9621c07..68e953e 100644
--- a/sdm/include/private/hw_info_types.h
+++ b/sdm/include/private/hw_info_types.h
@@ -36,8 +36,10 @@
namespace sdm {
using std::string;
-const int kMaxSDELayers = 16; // Maximum number of layers that can be handled by hardware in a
- // given layer stack.
+const int kMaxSDELayers = 16; // Maximum number of layers that can be handled by MDP5 hardware
+ // in a given layer stack.
+const int kMaxBlitLayers = 32; // Maximum number of layers that can be handled by MDP3 hardware
+ // in a given layer stack.
#define MAX_PLANES 4
#define MAX_DETAIL_ENHANCE_CURVE 3
@@ -489,6 +491,7 @@
bool use_hw_cursor = false; // Indicates that HWCursor pipe needs to be used for cursor layer
DestScaleInfoMap dest_scale_info_map = {};
HWHDRLayerInfo hdr_layer_info = {};
+ Handle pvt_data = NULL; // Private data used by sdm extension only.
};
struct HWLayers {
diff --git a/sdm/include/private/resource_interface.h b/sdm/include/private/resource_interface.h
index e32af41..e7df7a5 100644
--- a/sdm/include/private/resource_interface.h
+++ b/sdm/include/private/resource_interface.h
@@ -67,8 +67,6 @@
virtual DisplayError SetDetailEnhancerData(Handle display_ctx,
const DisplayDetailEnhancerData &de_data) = 0;
virtual DisplayError Perform(int cmd, ...) = 0;
-
- protected:
virtual ~ResourceInterface() { }
};
diff --git a/sdm/include/private/strategy_interface.h b/sdm/include/private/strategy_interface.h
index 6aec9cf..f903d5f 100644
--- a/sdm/include/private/strategy_interface.h
+++ b/sdm/include/private/strategy_interface.h
@@ -56,7 +56,6 @@
virtual DisplayError Purge() = 0;
virtual DisplayError SetIdleTimeoutMs(uint32_t active_ms) = 0;
- protected:
virtual ~StrategyInterface() { }
};
diff --git a/sdm/include/utils/factory.h b/sdm/include/utils/factory.h
new file mode 100644
index 0000000..f77a299
--- /dev/null
+++ b/sdm/include/utils/factory.h
@@ -0,0 +1,63 @@
+/*
+* Copyright (c) 2017, The Linux Foundation. All rights reserved.
+*
+* Redistribution and use in source and binary forms, with or without
+* modification, are permitted provided that the following conditions are
+* met:
+* * Redistributions of source code must retain the above copyright
+* notice, this list of conditions and the following disclaimer.
+* * Redistributions in binary form must reproduce the above
+* copyright notice, this list of conditions and the following
+* disclaimer in the documentation and/or other materials provided
+* with the distribution.
+* * Neither the name of The Linux Foundation nor the names of its
+* contributors may be used to endorse or promote products derived
+* from this software without specific prior written permission.
+*
+* THIS SOFTWARE IS PROVIDED "AS IS" AND ANY EXPRESS OR IMPLIED
+* WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
+* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT
+* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS
+* BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR
+* BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
+* WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE
+* OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN
+* IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+*/
+
+#ifndef __FACTORY_H__
+#define __FACTORY_H__
+
+#include <utility>
+#include <map>
+#include <string>
+
+namespace sdm {
+
+template <class Creator>
+class Factory {
+ public:
+ int Add(const std::string &name, const Creator &creator) {
+ map_.insert(std::pair<std::string, Creator>(name, creator));
+
+ return 0;
+ }
+
+ Creator Get(const std::string &name) {
+ typename std::map<std::string, Creator>::iterator it = map_.find(name);
+ if (it != map_.end()) {
+ return it->second;
+ }
+
+ return nullptr;
+ }
+
+ private:
+ std::map<std::string, Creator> map_;
+};
+
+} // namespace sdm
+
+#endif // __FACTORY_H__
diff --git a/sdm/libs/core/comp_manager.cpp b/sdm/libs/core/comp_manager.cpp
index 578d04a..2b9fa92 100644
--- a/sdm/libs/core/comp_manager.cpp
+++ b/sdm/libs/core/comp_manager.cpp
@@ -49,8 +49,8 @@
DisplayError error = kErrorNone;
if (extension_intf) {
- error = extension_intf->CreateResourceExtn(hw_res_info, &resource_intf_, buffer_allocator,
- buffer_sync_handler);
+ error = extension_intf->CreateResourceExtn(hw_res_info, buffer_allocator, buffer_sync_handler,
+ &resource_intf_);
extension_intf->CreateDppsControlExtn(&dpps_ctrl_intf_, socket_handler);
} else {
error = ResourceDefault::CreateResourceDefault(hw_res_info, &resource_intf_);
diff --git a/sdm/libs/core/display_base.cpp b/sdm/libs/core/display_base.cpp
index c268183..3953ce2 100644
--- a/sdm/libs/core/display_base.cpp
+++ b/sdm/libs/core/display_base.cpp
@@ -77,10 +77,9 @@
error = comp_manager_->GetScaleLutConfig(&lut_info);
if (error == kErrorNone) {
error = hw_intf_->SetScaleLutConfig(&lut_info);
- }
-
- if (error != kErrorNone) {
- goto CleanupOnError;
+ if (error != kErrorNone) {
+ goto CleanupOnError;
+ }
}
error = comp_manager_->RegisterDisplay(display_type_, display_attributes_, hw_panel_info_,
diff --git a/sdm/libs/core/fb/hw_device.h b/sdm/libs/core/fb/hw_device.h
index b94a5f3..374df56 100644
--- a/sdm/libs/core/fb/hw_device.h
+++ b/sdm/libs/core/fb/hw_device.h
@@ -1,5 +1,5 @@
/*
-* Copyright (c) 2014 - 2016, The Linux Foundation. All rights reserved.
+* Copyright (c) 2014 - 2017, The Linux Foundation. All rights reserved.
*
* Redistribution and use in source and binary forms, with or without modification, are permitted
* provided that the following conditions are met:
@@ -143,7 +143,7 @@
int fb_node_index_;
const char *fb_path_;
BufferSyncHandler *buffer_sync_handler_;
- int device_fd_;
+ int device_fd_ = -1;
int stored_retire_fence = -1;
HWDeviceType device_type_;
mdp_layer_commit mdp_disp_commit_;
diff --git a/sdm/libs/hwc2/hwc_display.cpp b/sdm/libs/hwc2/hwc_display.cpp
index a899c0e..7756020 100644
--- a/sdm/libs/hwc2/hwc_display.cpp
+++ b/sdm/libs/hwc2/hwc_display.cpp
@@ -999,6 +999,12 @@
i++;
}
}
+
+ auto client_target_layer = client_target_->GetSDMLayer();
+ if (client_target_layer->request.flags.flip_buffer) {
+ *out_display_requests = INT32(HWC2::DisplayRequest::FlipClientTarget);
+ }
+
return HWC2::Error::None;
}
diff --git a/sdm/libs/utils/Android.mk b/sdm/libs/utils/Android.mk
index 36d1148..aaadfa2 100644
--- a/sdm/libs/utils/Android.mk
+++ b/sdm/libs/utils/Android.mk
@@ -30,5 +30,7 @@
$(SDM_HEADER_PATH)/utils/locker.h \
$(SDM_HEADER_PATH)/utils/rect.h \
$(SDM_HEADER_PATH)/utils/sys.h \
- $(SDM_HEADER_PATH)/utils/utils.h
+ $(SDM_HEADER_PATH)/utils/utils.h \
+ $(SDM_HEADER_PATH)/utils/factory.h
+
include $(BUILD_COPY_HEADERS)