Merge "ipa: update wlan upstream iface name"

Merge "msm: mdss: Change function from disbale_irq_nosync to disbale_irq"

Merge "ASoC: wcd934x-dsp-cntl: initialize local char array val"

Merge "diag: Protect md_info structure while reallocation"

Merge "msm: ipa3: using rate limit api for ioctl calls"

Merge "msm: ipa3: fix hdr offset issue"

ipa: update wlan upstream iface name

Update WLAN upstream name to support STA SAP scenario
with wlan1 iface.

Change-Id: I0223c5b4aff8dfe24562a2c6d4ac581a7843224e
Acked-by: Pooja Kumari <kumarip@qti.qualcomm.com>
Signed-off-by: Mohammed Javid <mjavid@codeaurora.org>

Merge "msm: kgsl: unload/reload zap shader"

Merge "cnss2: Clear flag CNSS_DRIVER_UNLOADING after unregister driver finished"

Merge "cnss2: Skip PCIe link suspend if link has already been suspended"

ASoC: wcd934x-dsp-cntl: initialize local char array val

Due to the local char array that stores the codec dsp control command
is not initialized, an invalid command could cause the stack content
to be printed out in kernel dmesg. Initialize the array with memset.

Change-Id: I9573958fbe308c170c203a6a50a94d0540f3e7ce
Signed-off-by: Xiaoyu Ye <benyxy@codeaurora.org>

msm: ipa3: using rate limit api for ioctl calls

ipav3 ioctl calls input arguments validation failed cases,
replacing IPAERR()/IPAHAL_ERR() with IPAERR_RL()/IPAHAL_ERR_RL()
marco for controlling logs printing on the console.

Change-Id: I0b48b4a2f8b37c71aebe82c7b747b1d8f6c1e143
Acked-by: Pooja Kumari <kumarip@qti.qualcomm.com>
Acked-by: Ashok Vuyyuru <avuyyuru@qti.qualcomm.com>
Signed-off-by: Mohammed Javid <mjavid@codeaurora.org>

msm: ipa3: fix hdr offset issue

Hdr offset calculation is wrong when
driver received clean-up ioctl from
user-space. The fix is to find right
hdr offset to commit ned headers.

Change-Id: I70878a19b64c3defa6101bd68d435f0b74bcfb9b
Acked-by: Pooja Kumari <kumarip@qti.qualcomm.com>
Signed-off-by: Mohammed Javid <mjavid@codeaurora.org>
Signed-off-by: Skylar Chang <chiaweic@codeaurora.org>

cnss2: Clear flag CNSS_DRIVER_UNLOADING after unregister driver finished

Flag CNSS_DRIVER_UNLOADING is set when unregister wlan started, but it's
cleared in cnss_shutdown, which is also used in other call flow. Clear this
flag after unregister finished.

Change-Id: I0b5d63e5570c2604ff66d7852d7c7ca1c008a475
Signed-off-by: Kai Liu <kaliu@codeaurora.org>

Merge "soc: qcom: hab: fix some NULL pointer dereference issues"

Merge "usb: dwc3-msm: Queue resume_work unconditionally on pm_resume"

Merge "msm: vidc: Increase minimum input buffer count for HEVC decode"

Merge "soc: qcom: hab: add hab statistics support"

Merge "diag: Read the HSIC data in a work queue"

soc: qcom: hab: fix some NULL pointer dereference issues

With this patch, some potential NULL pointer dereference
issues are resolved. For example, if an invalid MMID
(MultiMedia ID defined by HAB driver) is gotten but we
fail to directly return with an error, the vchan variable
as NULL pointer will be dereferenced.

Change-Id: I92f9c5d08f99ef9db3e6a8b2ecdc0f074c8e8bff
Signed-off-by: Yong Ding <yongding@codeaurora.org>

msm: vidc: Increase minimum input buffer count for HEVC decode

Increase minimum input buffer count for HEVC decode to 5 to
avoid hang when the DPB is not full and FW keeps
waiting for FTB although sufficient ETB are being queued.

CRs-Fixed: 2252419

Change-Id: Id344cd896aeac46765e250e715d14a6c835b4221
Signed-off-by: Paras Nagda <pnagda@codeaurora.org>

Merge changes  into msm-4.4

Merge "msm: ipa: support ipacm cleanup"

Merge "cnss2: Add support for PCIe WLAN IPA uc SMMU feature"

Merge "diag: Modify md_info and diag client map synchronization"

Merge "msm: camera: Fix out-of-bounds read in string class name."

Merge "msm: camera: sensor: Add support for front aux sensor"

msm: ipa: support ipacm cleanup

Support header/filter, routing rules
cleanup when user-space module
crashed like ipacm and also cached
the wlan client connect messages
for ipacm to query.

Change-Id: Ib09cbe0e9114aa5a5673898ff796de7e7944af35
Acked-by: Pooja Kumari <kumarip@qti.qualcomm.com>
Signed-off-by: Mohammed Javid <mjavid@codeaurora.org>
Signed-off-by: Skylar Chang <chiaweic@codeaurora.org>

diag: Modify md_info and diag client map synchronization

Extend the scope of protection to md_info till queueing
the write to sdcard. Patch adds protection to diag client map
synchronizing access by various clients.

CRs-Fixed: 2282558
Change-Id: If076af2d09180a282a9077b4ebcda0184e9f67b5
Signed-off-by: Manoj Prabhu B <bmanoj@codeaurora.org>

Merge "power: wakeup_reason: send uevent to user space"

Merge "net: ipc_router: Initialize the sockaddr in recvmsg() handler"

cnss2: Add support for PCIe WLAN IPA uc SMMU feature

To add support for PCIe WLAN IPA uc SMMU feature, prvoide related
platform api for wlan driver to get the smmu map handle and do the
mapping.

Change-Id: I672b1a48879ada65b3ddb3f16c4bd787dc1b70a6
Signed-off-by: Frank Liu <qiliu@codeaurora.org>

diag: Protect md_info structure while reallocation

The possibility of md_info structure being accessed simultaneously
by two threads is prevented by synchronizing while buffer
reallocation for hdlc encoding. Extend the scope of protection to
md_info till queueing the write to sdcard.

CRs-Fixed: 2279473
Change-Id: I75ddb102adfc6c79f35ed69914c9140cb82894c9
Signed-off-by: Manoj Prabhu B <bmanoj@codeaurora.org>

Merge "msm: kgsl: Fix the missing IBs from the snapshot"

Merge "usb: phy-msm-ssusb-qmp: Make vls_clamp_reg as optional"

Merge "icnss: Remove sending uevent after FW ready"

msm: kgsl: Fix the missing IBs from the snapshot

We missed updating index for internal IBs like setstate
or power-on fixup.

The index value was never updated after that, so all the
IBs after that index were never parsed, and are missing
from the snapshot.

Change-Id: I61db75f33630db322fe5ee1adce9f120987d32ee
Signed-off-by: Sunil Khatri <sunilkh@codeaurora.org>

usb: phy-msm-ssusb-qmp: Make vls_clamp_reg as optional

Currently the SSUSB QMP PHY driver mandates that vls_clamp_reg be
passed from the DTSI. But this register cannot be accessed on a
platform on which Linux is a guest OS. This can lead to errors
when trying to enable autonomous mode. Work around this situation
by making the vls_clamp_reg property as optional.

Change-Id: Idb103d9b7bda717a5f12689951a7452c46aa76dd
Signed-off-by: Ajay Agarwal <ajaya@codeaurora.org>

usb: dwc3-msm: Queue resume_work unconditionally on pm_resume

Currently on pm_resume, the driver queues resume_work only if
vbus_active is true or id_state is false. So when the system is
suspended with host mode cable connected and the cable is
disconnected, the port still remains in host mode. Fix this by
unconditionally queueing resume_work from pm_resume and adding
required pm_relax in B_IDLE branch of sm_work.
Also create a freezable workqueue for sm_work so that the work
items do not race with pm_resume of child devices.

Change-Id: I9b79ddcc44c87111fc85086038361d9140be2af5
Signed-off-by: Ajay Agarwal <ajaya@codeaurora.org>

Merge "msm: ipa: Add support to IPA platform shutdown cleanup"

Merge "arch: arm64: Reenable HW breakpoint"

soc: qcom: hab: add hab statistics support

This allows user to read back hab runtime information.

Change-Id: Id266dd17b9c9d38f0e93aa600510ae1c6b12cca5
Signed-off-by: Yong Ding <yongding@codeaurora.org>

arch: arm64: Reenable HW breakpoint

HW breakpoint was disabled in
'commit f2a3f3ef8e73 ("arch: arm64: disable HW breakpoint")'.
However, it is required for setting breakpoints using ptrace
syscall, which is required for certain userspace applications
to work. So, reenable it.

Change-Id: Ib2a49488b92b0f77d13871d151778b3f0feab273
Signed-off-by: Neeraj Upadhyay <neeraju@codeaurora.org>

msm: ipa: Add support to IPA platform shutdown cleanup

Add support for platform reboot from IPA APPS
driver perspective.

Change-Id: Id65336da06a59a28944bfd51d9482d3d82a9cc3e
Signed-off-by: Mohammed Javid <mjavid@codeaurora.org>

Merge "msm: ipa3: Update holb config on USB DPL ep"

Merge "defconfig: msm: Enable QUOTA encryption support"

Merge "mm-camera2:isp2: Add NULL check for stream_info pointer"

msm: ipa3: Update holb config on USB DPL ep

Once USB DPL consumer ep is configured,
there is a chance of IPA hardware getting stalled,
if DPL client is not pulling the data over other end.
so, set holb discard over USB DPL cons ep
to avoid this stall.

Change-Id: I520b8c1ec84e03b783677c43670cbb9f904a4b4f
Signed-off-by: Mohammed Javid <mjavid@codeaurora.org>

Merge "mmc: core: Use mmc_get_cd_gpio to get cd gpio status in resume path"

defconfig: msm: Enable QUOTA encryption support

Add sdcardfs support and QUOTA encryption support for
CTS issue fixes.

Change-Id: Ie7120b21bf5b6b00432cc79302a3cf740a2cc710
Signed-off-by: Venkata Rao Kakani <vkakani@codeaurora.org>

mm-camera2:isp2: Add NULL check for stream_info pointer

The function msm_isp_get_stream_common_data could return a NULL,
add check to avoid NULL pointer dereference.

Change-Id: I157a74189b581d8ba90ef31f71c3be9b8369a764
Signed-off-by: Srikanth Uyyala <suyyala@codeaurora.org>

msm: mdss: Change function from disbale_irq_nosync to disbale_irq

Potential deadlock can happen while we disable irq and at the same
time an irq is triggered to request for the same resource. Thus,
it will pre-empt the spin-lock. Hence, change the function call to
disbale_irq to prevent local interrupts.

Change-Id: I7b80810b13660df8801533d418c78f2db75dd5b6
Signed-off-by: Rashi Bindra <rbindra@codeaurora.org>

Merge "ARM: dts: msm: Enable Apps shutdown feature in IPA driver for 8996HQX"

Merge "defconfig: msm: Define common hypervisor flag"

Merge "defconfig: msm8998: Enable VTI config for v4 and v6"

Merge "defconfig: sdm660: Enable VTI config for v4 and v6"

msm: kgsl: unload/reload zap shader

Unload zap shader during device hibernation and reload it
during resume otherwise scm calls during post hibernation
GPU initialization will fail as there is possibility that
TZ driver is not aware of the hibernation.

Change-Id: I1f62fb97cbc8e6c3e3536d4d5260a543ca15b685
Signed-off-by: Suprith Malligere Shankaregowda <supgow@codeaurora.org>
Signed-off-by: Thomas Yun <wyun@codeaurora.org>

defconfig: msm8998: Enable VTI config for v4 and v6

Enable VTI to give userspace (netd) the ability to create and
update virtual interfaces.

CRs-Fixed: 2214527
Change-Id: I5076d59230538c0b88fbd8d3c9188141a600526b
Signed-off-by: Devi Sandeep Endluri V V <dendluri@codeaurora.org>

defconfig: sdm660: Enable VTI config for v4 and v6

Enable VTI to give userspace (netd) the ability to create and
update virtual interfaces.

Change-Id: Ia32c78a8675757f4b600c6e4211f42afdb8249d4
CRs-Fixed: 2214527
Signed-off-by: Devi Sandeep Endluri V V <dendluri@codeaurora.org>

defconfig: msm: Define common hypervisor flag

Define common flag for hypervisor platforms and make use of it to:
1. Enable drm/msm-hyp
2. Disable drm/msm

Change-Id: I6eb6a7cd2a1128ed8d2bf6efd96c4829556eb876
Signed-off-by: Camus Wong <camusw@codeaurora.org>

Merge "USB: pd: Send PS_RDY within tNewSrc after PS_RDY received"

Merge "dwc3: Check USB LPM status before accessing any register"

USB: pd: Send PS_RDY within tNewSrc after PS_RDY received

USBPD spec mentions max time within which device that transitions
from sink power role to source needs to send PS_RDY message after
PS_RDY received from other device as part of PR_SWAP. Currently
USB PD driver is taking time around 380msec i.e greater than tNewSrc
(275msec) and resulting in USB PD complaince test "2.2.3.1.2
Procedure/Checks for Tester (Source) Originated Swap" failure. Fix it
by waiting only till VBUS voltage reaches VSafe5Vmin.

Change-Id: I3138e6d31ba964507230fe5f914aaaf2e261647d
Signed-off-by: Vijayavardhan Vennapusa <vvreddy@codeaurora.org>

dwc3: Check USB LPM status before accessing any register

USB debugfs entries can be accessed when USB session is not active or
USB is in low power mode. This results into unclocked register access.
Fix this issue by checking USB LPM status.

debugfs_create_regset32() fops related APIs are used to get USB dwc3
register dump. These fops don't have support to query device status
before accessing registers. Hence by default disabling creation of
regdump debugfs entry.

Change-Id: I67b00532eca159e000c848cb056d9759c47aefe9
Signed-off-by: Mayank Rana <mrana@codeaurora.org>

ARM: dts: msm: Disable runtime PM in host mode for automotive

Add "qcom,disable-host-mode-pm" flag to the USB nodes of
automotive targets to disable runtime PM in host mode.

Change-Id: I45949b92e7199b3ee9a95a7e0a3a584f813d0deb
Signed-off-by: Ajay Agarwal <ajaya@codeaurora.org>

Merge "msm: sps: Suppress bind/unbind attributes"

Merge "msm: mdss: use uaccess routines to access user space buffers"

diag: Read the HSIC data in a work queue

Add a new work queue to process the HSIC diag
data instead of processing it in the interrupt
context.

Change-Id: I8c546cd608c662d1c3133194f70af4953d734b08
Signed-off-by: Gustavo Solaira <gustavos@codeaurora.org>

msm: sps: Suppress bind/unbind attributes

SPS driver does not support manual bind/unbind operations
through sysfs. Suppress the bind/unbind nodes. Do not free
SPS struct in sps_device_de_init since it is being done in
sps_exit, and also to avoid use-after-free.

Change-Id: If6da6c5fb9d1a44d0420c6151f7f9d0a33cb2d04
Signed-off-by: Siva Kumar Akkireddi <sivaa@codeaurora.org>

mmc: core: Use mmc_get_cd_gpio to get cd gpio status in resume path

The change a4d7728mmc(core: Update SD card removal logic based on cd-
gpio state), has introduced logic to read the card presence status in
resume path of mmc_pm_notify(), for forcing card detection if card is
removed. But the API being used for getting gpio status invoking sleep
calls from resume context which shouldn't have any sleep calls.

So updated the resume path with different API for getting cd-gpio
state.

This chagne is not needed 4.9 kernel onwards since the there the
sdhci_get_cd() API has different implementation.

Change-Id: I5e7e760c6345a6fcdc873849d11f0bb32af09295
Signed-off-by: Veerabhadrarao Badiganti <vbadigan@codeaurora.org>

Merge "msm: mdss: Constant fetch across dfps"

net: ipc_router: Initialize the sockaddr in recvmsg() handler

sockaddr structure is filled with required information only which
results in few memory locations of structure with uninitialized data.

Memset complete structure before using it to remove uninitialized data.

CRs-Fixed: 2274853
Change-Id: I181710bde100fb1553b925d9fdf227af35ff38b5
Signed-off-by: Arun Kumar Neelakantam <aneela@codeaurora.org>

Merge "msm: mdss: cleanup used pipes during overlay off"

Merge "drm/sde: report MDP underrun error only if not stabilized"

Merge "drm/sde: bridge chip error and smmu fault handling for recovery"

Merge "drm: msm: Hot plug cpu1 once early domain is finished"

ARM: dts: msm: Enable Apps shutdown feature in IPA driver for 8996HQX

In order to support GVM restart functionality where apps VM can alone
shutdown, IPA apps driver need to reset IPA HW. So make a change to
enable the DT node to support apps shutdown feature.

Change-Id: I6076d4914a62004b06f5b555e64e22ed49e2551f
Signed-off-by: Mohammed Javid <mjavid@codeaurora.org>

Merge "ANDROID: Bluetooth: hidp: buffer overflow in hidp_process_report"

Merge "ARM: dts: msm: Add proper irq type in device tree for msm device"

ANDROID: Bluetooth: hidp: buffer overflow in hidp_process_report

The buffer length is unsigned at all layers, but gets cast to int and
checked in hidp_process_report and can lead to a buffer overflow.
Switch len parameter to unsigned int to resolve issue.

Signed-off-by: Mark Salyzyn <salyzyn@google.com>
Fixes: 678af93e46ac10318b54f2f0c9abbdfe75c4e078 ("HID: Bluetooth: hidp: make sure input buffers are big enough")
Bug: 65853588
Change-Id: I779ce783ae7c3bce8c5a66c0954ef31347e42cfc
Git-repo: https://android.googlesource.com/kernel/msm
Git-commit: 34c56d552bffcaefa84975bea0e55fb4481964fd
Signed-off-by: Srinivasa Rao Kuppala <srkupp@codeaurora.org>

msm: camera: Fix out-of-bounds read in string class name.

jpeg driver is calling class_create with stack variable, which
can be overwritten by other stack variables.

Change-Id: I3c22a5b3375b970ff6b1c6de983dd5833f4e11d0
Signed-off-by: Trishansh Bhardwaj <tbhardwa@codeaurora.org>
Signed-off-by: VijayaKumar T M <vtmuni@codeaurora.org>

cnss2: Skip PCIe link suspend if link has already been suspended

If wlan is loaded but without any activity for a period, wlan will
enter into shutdown status, which will suspend PCIe link.

Then if wlan unloading is issued, unloading process should skip
link suspend.

Change-Id: Id366401c656102cac5c5432a5b9a9a77e429cbbc
Signed-off-by: Kai Liu <kaliu@codeaurora.org>

Merge "drm: msm: fix variable initialization"

Merge "Merge android-4.4.141 (b1bad9e) into msm-4.4"

drm: msm: fix variable initialization

This change initializes variable i and j value to 0 in
dsi_display_prepare() as it was used uninitialized earlier.

Change-Id: I84ed753cfa64a719ec2f1b44a8ef24d5331b0fbc
Signed-off-by: Suprith Malligere Shankaregowda <supgow@codeaurora.org>

Merge "soc: qcom: hab: change the unregistration to be dummy"

Merge android-4.4.141 (b1bad9e) into msm-4.4

* refs/heads/tmp-b1bad9e
  Linux 4.4.141
  loop: remember whether sysfs_create_group() was done
  RDMA/ucm: Mark UCM interface as BROKEN
  PM / hibernate: Fix oops at snapshot_write()
  loop: add recursion validation to LOOP_CHANGE_FD
  netfilter: x_tables: initialise match/target check parameter struct
  netfilter: nf_queue: augment nfqa_cfg_policy
  uprobes/x86: Remove incorrect WARN_ON() in uprobe_init_insn()
  x86/cpufeature: Add helper macro for mask check macros
  x86/cpufeature: Make sure DISABLED/REQUIRED macros are updated
  x86/cpufeature: Update cpufeaure macros
  x86/cpufeature, x86/mm/pkeys: Fix broken compile-time disabling of pkeys
  x86/cpu: Add detection of AMD RAS Capabilities
  x86/mm/pkeys: Fix mismerge of protection keys CPUID bits
  x86/cpufeature, x86/mm/pkeys: Add protection keys related CPUID definitions
  x86/cpufeature: Speed up cpu_feature_enabled()
  x86/boot: Simplify kernel load address alignment check
  x86/vdso: Use static_cpu_has()
  x86/alternatives: Discard dynamic check after init
  x86/alternatives: Add an auxilary section
  x86/cpufeature: Get rid of the non-asm goto variant
  x86/cpufeature: Replace the old static_cpu_has() with safe variant
  x86/cpufeature: Carve out X86_FEATURE_*
  x86/headers: Don't include asm/processor.h in asm/atomic.h
  x86/fpu: Get rid of xstate_fault()
  x86/fpu: Add an XSTATE_OP() macro
  x86/cpu: Provide a config option to disable static_cpu_has
  x86/cpufeature: Cleanup get_cpu_cap()
  x86/cpufeature: Move some of the scattered feature bits to x86_capability
  iw_cxgb4: correctly enforce the max reg_mr depth
  tools build: fix # escaping in .cmd files for future Make
  Fix up non-directory creation in SGID directories
  HID: usbhid: add quirk for innomedia INNEX GENESIS/ATARI adapter
  xhci: xhci-mem: off by one in xhci_stream_id_to_ring()
  usb: quirks: add delay quirks for Corsair Strafe
  USB: serial: mos7840: fix status-register error handling
  USB: yurex: fix out-of-bounds uaccess in read handler
  USB: serial: keyspan_pda: fix modem-status error handling
  USB: serial: cp210x: add another USB ID for Qivicon ZigBee stick
  USB: serial: ch341: fix type promotion bug in ch341_control_in()
  ahci: Disable LPM on Lenovo 50 series laptops with a too old BIOS
  vmw_balloon: fix inflation with batching
  ibmasm: don't write out of bounds in read handler
  MIPS: Fix ioremap() RAM check
  cpufreq: Kconfig: Remove CPU_FREQ_DEFAULT_GOV_SCHED

Change-Id: I0909a2917621f2384cdfe27078577cc2c06b9612
Signed-off-by: Blagovest Kolenichev <bkolenichev@codeaurora.org>
Signed-off-by: Srinivasarao P <spathi@codeaurora.org>

msm: mdss: Constant fetch across dfps

Timing engine HW restricts changing programmable fetch
start when off. Program constant fetch start in SW drivers.
Fix fetch start is calculated based on panel DT timings.

Change-Id: I2124c92b2f83d9c963e05c14421f61a08878d63b
Signed-off-by: Animesh Kishore <animeshk@codeaurora.org>

msm: mdss: use uaccess routines to access user space buffers

Use routines defined in uaccess.h (like put_user, get_user)
while accessing user space buffers. Otherwise this leads to
userspace access violation when PAN is enabled.

Change-Id: Iea16e347ee71f0845711a21b6b9b01e16a6ab619
Signed-off-by: Nirmal Abraham <nabrah@codeaurora.org>

Merge "msm: camera: isp : Fix race condition in close sequence"

msm: camera: isp : Fix race condition in close sequence

Serializing reset_hw and reset_irq, to avoid race condition

Change-Id: I8f21cb816748129bde7f0f1455b203b42603d244
Signed-off-by: Srikanth Uyyala <suyyala@codeaurora.org>

Merge changes  into msm-4.4

Merge "msm: camera: skip recovery during stream off in dual vfe"

drm/sde: report MDP underrun error only if not stabilized

MDP underrun error is being reported to recovery manager as and when they
are detected by the interrupt handler. Since most of the times errors are
recovered successfully and stabilizes and hence we do not need to report in
those scenarios.

This implementation uses a delayed work to figure out if the error is
stabilized. In case if error is still persists then it would be reported
to the recovery manager.

Change-Id: I3509484867d86dc946dd4be856d85de444c0aa60
Signed-off-by: Rahul Sharma <rahsha@codeaurora.org>

Merge "defconfig: sdm660: align with android config requirement"

Merge "msm: vidc: Addition of enums for HLG and HDR10"

Merge "Merge android-4.4.140 (789274d) into msm-4.4"

Merge "diag: Featurize IPC logging"

soc: qcom: hab: change the unregistration to be dummy

HAB service should be kept available as long as possible
since it is very fundamental. It can possibly be used in
the GVM(Guest Virtual Machine) shutdown procedure to
talk with the hypervisor. And the unregistration function
will only be left to give a log message.

Change-Id: I0cbc153a68a0fb496d14fc49db6cfee211a01722
Signed-off-by: Yong Ding <yongding@codeaurora.org>

msm: camera: skip recovery during stream off in dual vfe

in dual vfe usecase, skip pingpong mismatch recovery
if one of the vfe's active stream count is zero.

Change-Id: I1b4dce66ad6665e41c4185d3ac510204d40131da
Signed-off-by: Srikanth Uyyala <suyyala@codeaurora.org>