Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752324AbbH0R4a (ORCPT ); Thu, 27 Aug 2015 13:56:30 -0400 Received: from mail-yk0-f172.google.com ([209.85.160.172]:34450 "EHLO mail-yk0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751329AbbH0R41 (ORCPT ); Thu, 27 Aug 2015 13:56:27 -0400 MIME-Version: 1.0 In-Reply-To: <1440335381-5122-7-git-send-email-ygardi@codeaurora.org> References: <1440335381-5122-1-git-send-email-ygardi@codeaurora.org> <1440335381-5122-7-git-send-email-ygardi@codeaurora.org> From: Rob Herring Date: Thu, 27 Aug 2015 12:56:06 -0500 Message-ID: Subject: Re: [PATCH v3 6/8] scsi: ufs: make the UFS variant a platform device To: Yaniv Gardi Cc: Jej B , Paul Bolle , hch@infradead.org, "linux-kernel@vger.kernel.org" , linux-scsi@vger.kernel.org, linux-arm-msm , Santosh Y , linux-scsi-owner@vger.kernel.org, Subhash Jadavani , Gilad Broner , Dolev Raviv , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Vinayak Holikatti , "James E.J. Bottomley" , Christoph Hellwig , Sujit Reddy Thumma , Raviv Shvili , Sahitya Tummala , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2779 Lines: 56 On Sun, Aug 23, 2015 at 8:09 AM, Yaniv Gardi wrote: > This change turns the UFS variant (SCSI_UFS_QCOM) into a UFS > a platform device. > In order to do so a few additional changes are required: > 1. The ufshcd-pltfrm is no longer serves as a platform device. > Now it only serves as a group of platform APIs such as PM APIs > (runtime suspend/resume, system suspend/resume etc), parsers of > clocks, regulators and pm_levels from DT. > 2. What used to be the old platform "probe" is now "only" > a pltfrm_init() routine, that does exactly the same, but only > being called by the new probe function of the UFS variant. > > Signed-off-by: Yaniv Gardi > > --- > .../devicetree/bindings/ufs/ufshcd-pltfrm.txt | 2 +- > drivers/scsi/ufs/ufs-qcom.c | 78 +++++++++++++++++- > drivers/scsi/ufs/ufshcd-pltfrm.c | 92 ++++++---------------- > drivers/scsi/ufs/ufshcd-pltfrm.h | 41 ++++++++++ > drivers/scsi/ufs/ufshcd.c | 10 +++ > drivers/scsi/ufs/ufshcd.h | 1 + > 6 files changed, 152 insertions(+), 72 deletions(-) > create mode 100644 drivers/scsi/ufs/ufshcd-pltfrm.h > > diff --git a/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt b/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt > index 5357919..b39e765 100644 > --- a/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt > +++ b/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt > @@ -4,7 +4,7 @@ UFSHC nodes are defined to describe on-chip UFS host controllers. > Each UFS controller instance should have its own node. > > Required properties: > -- compatible : compatible list, contains "jedec,ufs-1.1" > +- compatible : compatible list, contains "jedec,ufs-1.1" or "qcom,ufshc" Replying again as I inadvertently dropped everyone. This should also have a more specific compatible string with the SOC name/number in it. It may be "the same in all SOCs", but there is always the possibility for bugs/limitations to be found that are specific to an SOC even if all RTL versions are identical (e.g. different max clock speeds). It is about making the dtb future proof, not about what exactly you need today. You can keep qcom,ufshc for driver matching if you want. > - interrupts : > - reg : What about phy properties? No Unipro PHY block that requires setup? Rob -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/