Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E7FA8C54EAA for ; Fri, 27 Jan 2023 05:43:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229452AbjA0FnK (ORCPT ); Fri, 27 Jan 2023 00:43:10 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52502 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229536AbjA0FnI (ORCPT ); Fri, 27 Jan 2023 00:43:08 -0500 Received: from mail-yw1-x1129.google.com (mail-yw1-x1129.google.com [IPv6:2607:f8b0:4864:20::1129]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 76D5E18B31 for ; Thu, 26 Jan 2023 21:43:07 -0800 (PST) Received: by mail-yw1-x1129.google.com with SMTP id 00721157ae682-50660e2d2ffso53885727b3.1 for ; Thu, 26 Jan 2023 21:43:07 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=fqJbS1bFJa5utvCnTv+wYBXRWVr+UnUdQNczPlCi+3U=; b=TLdkMwBmkOpoMtKqXKSYzWj+S6KAXh59dODKmFzdRWBsvbUGbq5nfaABjBysst3wgK ObuQsZl4vSPe+p/0xNzLiHEuvZNWp3Le22Q9XWQAobiyTLi4GIu8D+4Xb1/hb8Szd+AV KlVROzaQ4/YeKAVaJLVqzlvIoqTLPzOgZZvKmvYCSkW9TJfMz9tTdSYvy/Npb9X932iQ R2xE6eeZHxB1uTC786OSNmmrm7gwbbHo3FZbHVy+MAVAjRXMHkapH/aX0umXMlewggcr gR/UobIuX7wiilvG5/WR6P3ztVaWWmVGku7Ev5ICsMMgw6aiocrJGVPRJcVjctT8V6WI WwyQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=fqJbS1bFJa5utvCnTv+wYBXRWVr+UnUdQNczPlCi+3U=; b=ZsPYcKQaj1/3hy88akWkb7RnkE8MPQfIl7SNhZbR/sASOjtp+sTfazrk/EZoDGQF0M Cl5PdKpPy1n13j+qIu33i1AFMujWt33mn2GmHZXOix1oxBOwZr5q3x1E3L8/zXGSHTBg jw1FRhzTrOQqntYwOgSLlV2fCjjocJ2h4hzLe/5PCuFj7+GNbY2PHK+kcmhO/zvQIE4G Cq8IBaAiO0aGh8R5lXatJ6rFW4aC8KtwRjVTv3+SYkq3ZuP8K6eOmi63W4k+RSldQqwd N8ptMwuCsSa/Ayrl3VckTCc1phgkg3mzeMuOHRaLZHyyUbMYbBqftZUCQDPKpWXbNl7q 51vg== X-Gm-Message-State: AFqh2kpD3vYTrE1vEe29ZOczQA4u+UUR3YiRbSNM6cP7ZsjsCLPyzpmd mBHkYAlPWJ6qGrESnA2ACiI/uT7LhwitSsxxhzLATRRduhhFITcCYAg= X-Google-Smtp-Source: AMrXdXv8jXCaIMBzTzmvSJhUZCAILeJ1JQhpL6j3RKiZ3tge8MD4DQJuzQfpd1Jnud+SHRAU0UymE07hNfn68KwB2iM= X-Received: by 2002:a81:3e0d:0:b0:4fe:1d13:5980 with SMTP id l13-20020a813e0d000000b004fe1d135980mr4580021ywa.378.1674798186717; Thu, 26 Jan 2023 21:43:06 -0800 (PST) MIME-Version: 1.0 References: <20230127041200.29094-1-quic_namajain@quicinc.com> In-Reply-To: <20230127041200.29094-1-quic_namajain@quicinc.com> From: Dmitry Baryshkov Date: Fri, 27 Jan 2023 07:42:55 +0200 Message-ID: Subject: Re: [PATCH v2] soc: qcom: socinfo: Add support for new field in revision 17 To: Naman Jain Cc: Bjorn Andersson , Andy Gross , Konrad Dybcio , Trilok Soni , linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, quic_pkondeti@quicinc.com Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 27 Jan 2023 at 06:12, Naman Jain wrote: > > Add support for new field coming with socinfo structure under v17 to get > hardware platform's oem variant id. This is to enable OEMs to have minor > changes in the board, but to use the same platform subtype as the one > supported by Qualcomm. The new field is to be used in platform overlay > file. Default value is 0, reserved for Qualcomm platforms. Also, add > debugfs support to read this field for a device. > > Signed-off-by: Naman Jain Reviewed-by: Dmitry Baryshkov > --- > v1 -> v2: Removed change-id, included SOB line > > drivers/soc/qcom/socinfo.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/drivers/soc/qcom/socinfo.c b/drivers/soc/qcom/socinfo.c > index 10efdbcfdf05..10cdd32ce74d 100644 > --- a/drivers/soc/qcom/socinfo.c > +++ b/drivers/soc/qcom/socinfo.c > @@ -174,6 +174,8 @@ struct socinfo { > __le32 pcode; > __le32 npartnamemap_offset; > __le32 nnum_partname_mapping; > + /* Version 17 */ > + __le32 oem_variant; > }; > > #ifdef CONFIG_DEBUG_FS > @@ -196,6 +198,7 @@ struct socinfo_params { > u32 nmodem_supported; > u32 feature_code; > u32 pcode; > + u32 oem_variant; > }; > > struct smem_image_version { > @@ -519,6 +522,11 @@ static void socinfo_debugfs_init(struct qcom_socinfo *qcom_socinfo, > &qcom_socinfo->info.fmt); > > switch (qcom_socinfo->info.fmt) { > + case SOCINFO_VERSION(0, 17): > + qcom_socinfo->info.oem_variant = __le32_to_cpu(info->oem_variant); > + debugfs_create_u32("oem_variant", 0444, qcom_socinfo->dbg_root, > + &qcom_socinfo->info.oem_variant); > + fallthrough; > case SOCINFO_VERSION(0, 16): > qcom_socinfo->info.feature_code = __le32_to_cpu(info->feature_code); > qcom_socinfo->info.pcode = __le32_to_cpu(info->pcode); > -- > 2.17.1 > -- With best wishes Dmitry