From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bjorn Andersson Subject: Re: [PATCH v2 1/3] remoteproc: qcom: Encapsulate pvt data structure for q6v56 hexagon. Date: Fri, 18 Nov 2016 10:40:30 -0800 Message-ID: <20161118184030.GJ28340@tuxbot> References: <1478268057-11847-1-git-send-email-akdwived@codeaurora.org> <1478268057-11847-2-git-send-email-akdwived@codeaurora.org> <20161110203013.fphvzd4df3zvjdfi@rob-hp-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Avaneesh Kumar Dwivedi Cc: Rob Herring , Ohad Ben-Cohen , Mark Rutland , "open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM" , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , open list List-Id: devicetree@vger.kernel.org On Wed 16 Nov 06:02 PST 2016, Avaneesh Kumar Dwivedi wrote: > > > On 11/11/2016 2:00 AM, Rob Herring wrote: > >On Fri, Nov 04, 2016 at 07:30:54PM +0530, Avaneesh Kumar Dwivedi wrote: > >>Encapsulate resources specific to each version of hexagon chip to > >>device node to avoid conditional check for manipulation of those > >>resources in driver code. > >> > >>Signed-off-by: Avaneesh Kumar Dwivedi > >>--- > >> .../devicetree/bindings/remoteproc/qcom,q6v5.txt | 1 + > >> drivers/remoteproc/qcom_q6v5_pil.c | 137 ++++++++++++++++++--- > >> 2 files changed, 120 insertions(+), 18 deletions(-) > >> > >>diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,q6v5.txt b/Documentation/devicetree/bindings/remoteproc/qcom,q6v5.txt > >>index 57cb49e..cbc165c 100644 > >>--- a/Documentation/devicetree/bindings/remoteproc/qcom,q6v5.txt > >>+++ b/Documentation/devicetree/bindings/remoteproc/qcom,q6v5.txt > >>@@ -8,6 +8,7 @@ on the Qualcomm Hexagon core. > >> Value type: > >> Definition: must be one of: > >> "qcom,q6v5-pil" > >>+ "qcom,q6v56-pil" > >Perhaps some explanation in the commit message about what these magic > >numbers mean? > > "v56" represent class of hexagon chip, which again is differentiated > based on version number. Two > different MSM SOC may use same class of hexagon chip. example is as > below. > > msm8974 q6v5 version 5.0.0 > msm8916 q6v5 version 5.1.1 But looking at the Qualcomm tree I think I got 8916 wrong, it seems that it should be "q6v56" and your patches indicates that the 8996 has a q6v55 - which doesn't make sense to me and in some places there's comments indicating it's version 6. I asked you about this but I can't find an answer in any of your replies. Regards, Bjorn -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html