From 90aeb6e15cef4087947dab96f91a478a2f52242d Mon Sep 17 00:00:00 2001 From: joenchen Date: Wed, 7 Sep 2022 12:55:19 +0000 Subject: [PATCH 1/2] RRS: Apply the default config from persist prop vendor_config plays as another role to control the display config during the boot time. To change the default configuration of the user selected mode, we use persist config to store the value. Bug: 244492960 Test: Boot w/ and w/o user selected configs and check the resolution Change-Id: Ic3eb4e1c8a2c5eed83d10799a1965dd7a6be58e1 --- display/gs101/hal_graphics_composer_default.te | 4 ++-- whitechapel/vendor/google/property_contexts | 1 + whitechapel/vendor/google/vendor_init.te | 3 +++ 3 files changed, 6 insertions(+), 2 deletions(-) diff --git a/display/gs101/hal_graphics_composer_default.te b/display/gs101/hal_graphics_composer_default.te index c1eac9ce..dccddf0e 100644 --- a/display/gs101/hal_graphics_composer_default.te +++ b/display/gs101/hal_graphics_composer_default.te @@ -25,8 +25,8 @@ allow hal_graphics_composer_default sysfs_leds:file rw_file_perms; # allow HWC to get vendor_persist_sys_default_prop get_prop(hal_graphics_composer_default, vendor_persist_sys_default_prop) -# allow HWC to get vendor_display_prop -get_prop(hal_graphics_composer_default, vendor_display_prop) +# allow HWC to get/set vendor_display_prop +set_prop(hal_graphics_composer_default, vendor_display_prop) # allow HWC to get device_config_surface_flinger_native_boot_prop for adpf flags get_prop(hal_graphics_composer_default, device_config_surface_flinger_native_boot_prop) diff --git a/whitechapel/vendor/google/property_contexts b/whitechapel/vendor/google/property_contexts index 5eba1f8d..9f4e8dc9 100644 --- a/whitechapel/vendor/google/property_contexts +++ b/whitechapel/vendor/google/property_contexts @@ -65,6 +65,7 @@ vendor.audiodump.encode.disable u:object_r:vendor_audio_prop:s0 # for display ro.vendor.hwc.drm.device u:object_r:vendor_display_prop:s0 +persist.vendor.display. u:object_r:vendor_display_prop:s0 # for camera persist.vendor.camera. u:object_r:vendor_camera_prop:s0 diff --git a/whitechapel/vendor/google/vendor_init.te b/whitechapel/vendor/google/vendor_init.te index dfd8e996..9686bccb 100644 --- a/whitechapel/vendor/google/vendor_init.te +++ b/whitechapel/vendor/google/vendor_init.te @@ -35,3 +35,6 @@ set_prop(vendor_init, vendor_battery_defender_prop) # Fingerprint property set_prop(vendor_init, vendor_fingerprint_prop) + +# Display +set_prop(vendor_init, vendor_display_prop) From c76556752479a59b7ba006d45792e9ff152e7292 Mon Sep 17 00:00:00 2001 From: Siarhei Vishniakou Date: Thu, 16 Jun 2022 15:59:46 -0700 Subject: [PATCH 2/2] Allow InputProcessor HAL to read display resolution Currently, there's no API to read the resolution from the system domain, so the HAL has to read this from the sysprop provided by the display code. Allow the HAL to do so in this CL. Bug: 244492960 Test: adb shell dmesg | grep input_processor Change-Id: Ibdc3589234bbee8641e3c1f7a300b622803ca1a9 --- whitechapel/vendor/google/hal_input_processor_default.te | 2 ++ 1 file changed, 2 insertions(+) create mode 100644 whitechapel/vendor/google/hal_input_processor_default.te diff --git a/whitechapel/vendor/google/hal_input_processor_default.te b/whitechapel/vendor/google/hal_input_processor_default.te new file mode 100644 index 00000000..00d4c695 --- /dev/null +++ b/whitechapel/vendor/google/hal_input_processor_default.te @@ -0,0 +1,2 @@ +# allow InputProcessor HAL to read the display resolution system property +get_prop(hal_input_processor_default, vendor_display_prop)