From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gilad Avidov Subject: [PATCH V5 0/2] add support for pmic_arb v2 and correct framework Date: Wed, 25 Mar 2015 11:37:30 -0600 Message-ID: <1427305052-1284-1-git-send-email-gavidov@codeaurora.org> Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:55937 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754184AbbCYRhl (ORCPT ); Wed, 25 Mar 2015 13:37:41 -0400 Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: gavidov@codeaurora.org, sdharia@codeaurora.org, mlocke@codeaurora.org, linux-arm-msm@vger.kernel.org, gregkh@linuxfoundation.org, svarbanov@mm-sol.com Cc: linux-kernel@vger.kernel.org, iivanov@mm-sol.com, galak@codeaurora.org, agross@codeaurora.org pmic_arb v2 has no support for spmi non-data commands and thus returns -EOPNOTSUPP on .cmd callback. This causes a failure in spmi_drv_probe() which sends a wakeup command to the slave before probing its driver. This patchset removes the wakeup from spmi_drv_probe() since the spmi spec stipulates that a slaves default state is active and doesn't need a wakeup. Changes from v3 to v4: * Remove the claim that this is a bug fix off the commit text * Unmap the core register space as soon as we know that it will not be used * Assign the core reg space to a local until we know if it appropriate to use it to configure the controller fields (on v1) or unmap it (on v2). Changes from v4 to v5: * remove the unmap added in v4 since it is used again and the mapping is managed. * correct printf formatting for 64bit compilers. * correct base address for reading the channel table. Gilad Avidov (2): spmi: remove wakeup command before slave probe spmi: pmic_arb: add support for hw version 2 .../bindings/spmi/qcom,spmi-pmic-arb.txt | 6 +- drivers/spmi/spmi-pmic-arb.c | 319 +++++++++++++++++---- drivers/spmi/spmi.c | 9 +- 3 files changed, 267 insertions(+), 67 deletions(-) -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation