Return-Path: Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 10.2 \(3259\)) Subject: Re: [PATCH 1/3] soc: qcom: smd: Transition client drivers from smd to rpmsg From: Marcel Holtmann In-Reply-To: <20170320233544.1656-1-bjorn.andersson@linaro.org> Date: Tue, 21 Mar 2017 09:20:01 +0100 Cc: "Gustavo F. Padovan" , Johan Hedberg , Eugene Krasnikov , Kalle Valo , Andy Gross , David Brown , "David S. Miller" , Arnd Bergmann , "open list:BLUETOOTH DRIVERS" , LKML , wcn36xx@lists.infradead.org, linux-wireless , Network Development , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org Message-Id: <3DCEF38C-F8FC-4941-936F-921247E13B78@holtmann.org> References: <20170320233544.1656-1-bjorn.andersson@linaro.org> To: Bjorn Andersson Sender: linux-kernel-owner@vger.kernel.org List-ID: Hi Bjorn, > By moving these client drivers to use RPMSG instead of the direct SMD > API we can reuse them ontop of the newly added GLINK wire-protocol > support found in the 820 and 835 Qualcomm platforms. > > As the new (RPMSG-based) and old SMD implementations are mutually > exclusive we have to change all client drivers in one commit, to make > sure we have a working system before and after this transition. > > Signed-off-by: Bjorn Andersson > --- > > Based on v4.11-rc3 with Arnd's Kconfig dependency fixes for BT_QCOMSMD > (https://lkml.org/lkml/2017/3/20/1038). > > drivers/bluetooth/Kconfig | 2 +- > drivers/bluetooth/btqcomsmd.c | 32 +++++++++---------- > drivers/net/wireless/ath/wcn36xx/Kconfig | 2 +- > drivers/net/wireless/ath/wcn36xx/main.c | 6 ++-- > drivers/net/wireless/ath/wcn36xx/smd.c | 10 +++--- > drivers/net/wireless/ath/wcn36xx/smd.h | 6 ++-- > drivers/net/wireless/ath/wcn36xx/wcn36xx.h | 2 +- > drivers/soc/qcom/Kconfig | 4 +-- > drivers/soc/qcom/smd-rpm.c | 43 +++++++++++++------------ > drivers/soc/qcom/wcnss_ctrl.c | 50 +++++++++++++++++------------- > include/linux/soc/qcom/wcnss_ctrl.h | 11 ++++--- > net/qrtr/Kconfig | 2 +- > net/qrtr/smd.c | 42 ++++++++++++------------- > 13 files changed, 108 insertions(+), 104 deletions(-) I think that it is best if Dave takes these directly then and I pull them back down later into our trees. Including the Kconfig fix that I will ACK as well. Acked-by: Marcel Holtmann Regards Marcel