Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753986AbbGBRhi (ORCPT ); Thu, 2 Jul 2015 13:37:38 -0400 Received: from seldrel01.sonyericsson.com ([37.139.156.2]:2158 "EHLO seldrel01.sonyericsson.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753455AbbGBRha (ORCPT ); Thu, 2 Jul 2015 13:37:30 -0400 Date: Thu, 2 Jul 2015 10:37:25 -0700 From: Bjorn Andersson To: Stephen Boyd CC: Kumar Gala , Andy Gross , "linux-kernel@vger.kernel.org" , "linux-arm-msm@vger.kernel.org" , "linux-soc@vger.kernel.org" Subject: Re: [PATCH] firmware: qcom: scm: Peripheral Authentication Service Message-ID: <20150702173725.GE9633@sonymobile.com> References: <1435693579-16109-1-git-send-email-bjorn.andersson@sonymobile.com> <559564AC.6030100@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <559564AC.6030100@codeaurora.org> User-Agent: Mutt/1.5.22 (2013-10-16) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1085 Lines: 41 On Thu 02 Jul 09:19 PDT 2015, Stephen Boyd wrote: > On 06/30/2015 12:46 PM, Bjorn Andersson wrote: > > + > > +int __qcom_scm_pas_init_image(u32 peripheral, const void *metadata, size_t size) > > +{ > > + dma_addr_t mdata_phys; > > + void *mdata_buf; > > + u32 scm_ret; > > + int ret; > > + struct pas_init_image_req { > > + u32 proc; > > + u32 image_addr; > > Please mark these as __le32 and do the appropriate conversions to little > endian. Just out of curiosity, are there any big endian systems out there that will run this code? Either way I will fix this, to follow the style of the other parts of this file. > Also consider removing the struct name and just make it > anonymous, i.e. > > struct { > __le32 proc; > __le32 image_addr; > } cmd; > Right, there's no reason to name these. Thanks, Bjorn -- 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/