From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bjorn Andersson Subject: Re: [PATCH v2 4/5] soc: qcom: smem: Support dynamic item limit Date: Fri, 15 Sep 2017 11:40:11 -0700 Message-ID: <20170915184011.GR2016@tuxbook> References: <1505424302-29147-1-git-send-email-clew@codeaurora.org> <1505424302-29147-5-git-send-email-clew@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pg0-f48.google.com ([74.125.83.48]:52673 "EHLO mail-pg0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751443AbdIOSkO (ORCPT ); Fri, 15 Sep 2017 14:40:14 -0400 Received: by mail-pg0-f48.google.com with SMTP id i195so1982600pgd.9 for ; Fri, 15 Sep 2017 11:40:14 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1505424302-29147-5-git-send-email-clew@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Chris Lew Cc: andy.gross@linaro.org, david.brown@linaro.org, aneela@codeaurora.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, linux-kernel@vger.kernel.org On Thu 14 Sep 14:25 PDT 2017, Chris Lew wrote: > In V12 SMEM, SBL writes SMEM parameter information > after the TOC. Use the SBL provided item count > as the max item number. Please wrap your commit messages at 72 chars (50 for the subject). [..] > +static u32 qcom_smem_get_dynamic_item(struct qcom_smem *smem) This naming is too similar to the functions for looking up items, please rename it to "qcom_smem_get_item_count()". Regards, Bjorn