From mboxrd@z Thu Jan 1 00:00:00 1970 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 EFC05C32771 for ; Wed, 28 Sep 2022 17:36:53 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234467AbiI1Rgw (ORCPT ); Wed, 28 Sep 2022 13:36:52 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37790 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234255AbiI1Rgt (ORCPT ); Wed, 28 Sep 2022 13:36:49 -0400 Received: from mail-lf1-x12f.google.com (mail-lf1-x12f.google.com [IPv6:2a00:1450:4864:20::12f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 116D7F1D7D for ; Wed, 28 Sep 2022 10:36:48 -0700 (PDT) Received: by mail-lf1-x12f.google.com with SMTP id g1so3830935lfu.12 for ; Wed, 28 Sep 2022 10:36:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:mime-version:message-id:references :in-reply-to:user-agent:subject:cc:to:from:date:from:to:cc:subject :date; bh=w+MFvmRFQw8K94SsImKDP38Aj2nWJgY5XIWG9ozikEw=; b=Dwkprnz8wFxJLVMDPF//mbzEHe74Ic/GADCjZdO1ADy1CIGntvxDOdncWqpCJvKAV+ qNyvAf0NMQ16vsKR7f3E0TA3SfB7BQBv6847ltA6IGRjwc0R7RedTK09rAytQ/IlOp0g 5+x7/vuYY1aHCLqwW3QqIGpMp2dCZZVWjekkNyGvdTSzaAzjTGsbhUeJ7yZSEloTGcby Arz1mb51fW1VaERgu/NssIT5/txVM2chZudhawS+2mMgfzu5ougHqyoxRLQi1OC2Ib84 /VEIHAqnVicdswHJemmV+0MuVjzc6fBCh99/XB7bNuvaR8HMzy6we3VZxv50B6jQhrLe yeYg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:message-id:references :in-reply-to:user-agent:subject:cc:to:from:date:x-gm-message-state :from:to:cc:subject:date; bh=w+MFvmRFQw8K94SsImKDP38Aj2nWJgY5XIWG9ozikEw=; b=cYyo0sH9nXdHHuMWMC1eRm9vtpGQG1yD0QDPdxwYcd4ZXLsxsD0MUianP4ZwO0iGh3 dQ571OTmL+0bZqL06ISrAGnwe0SXXqmu3hNc/KSu2dNiO0tKHTD6PcRJIbWO0FHK4fK2 LLqo6CAweEN3YviZTwaSPq8zxax9yYPdsMkvOAd2TCoVKBjcryiEI8u8fFenI4gUZhaa TusUdVvtQjO3oYIROxrlzCK4iOv9+yQn/15iM7N7gWVxIQh9Kg12Zs87n77blI7PJbyT +/+FuGrttc8qwQc0So0bWeYOXHPuT22ScRIxNNjfWOHCDJ+qhB+Qvwy9qPExgKGKlvgK 4qmw== X-Gm-Message-State: ACrzQf1wF2+bcFl4cpKXQLDfvNQM/b+YFDp0mdkQmIyc+q7wpo4qKW5J P4UVKNHBaVHvxFtw5si88mHwxw== X-Google-Smtp-Source: AMsMyM69qcnLaoGodQcX/ivuUVEVAxn4HKJwBR3GkIzuZ2dJCAOzzbaj4OPtuBRd9ESjriBGCjhFHg== X-Received: by 2002:a05:6512:10d6:b0:49a:1fc0:cc62 with SMTP id k22-20020a05651210d600b0049a1fc0cc62mr14733885lfg.138.1664386606380; Wed, 28 Sep 2022 10:36:46 -0700 (PDT) Received: from [127.0.0.1] ([94.25.229.4]) by smtp.gmail.com with ESMTPSA id o27-20020ac25e3b000000b0049464d89e40sm529972lfg.72.2022.09.28.10.36.45 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Wed, 28 Sep 2022 10:36:46 -0700 (PDT) Date: Wed, 28 Sep 2022 20:34:37 +0300 From: Dmitry Baryshkov To: Johan Hovold , Vinod Koul CC: Andy Gross , Bjorn Andersson , Konrad Dybcio , Kishon Vijay Abraham I , linux-arm-msm@vger.kernel.org, linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 04/13] phy: qcom-qmp-pcie: unify ipq registers User-Agent: K-9 Mail for Android In-Reply-To: <20220928152822.30687-5-johan+linaro@kernel.org> References: <20220928152822.30687-1-johan+linaro@kernel.org> <20220928152822.30687-5-johan+linaro@kernel.org> Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 28 September 2022 18:28:13 GMT+03:00, Johan Hovold wrote: >The IPQ register array is identical to sm8250_pcie_regs_layout so drop >the former=2E I'd not do such merge=2E They belong to different generations=2E I'd sugge= st changing these arrays to use symbolic names defined in corresponding qmp= headers=2E > >Signed-off-by: Johan Hovold >--- > drivers/phy/qualcomm/phy-qcom-qmp-pcie=2Ec | 11 ++--------- > 1 file changed, 2 insertions(+), 9 deletions(-) > >diff --git a/drivers/phy/qualcomm/phy-qcom-qmp-pcie=2Ec b/drivers/phy/qua= lcomm/phy-qcom-qmp-pcie=2Ec >index ec453f908f1d=2E=2E7b3f7e42edd5 100644 >--- a/drivers/phy/qualcomm/phy-qcom-qmp-pcie=2Ec >+++ b/drivers/phy/qualcomm/phy-qcom-qmp-pcie=2Ec >@@ -86,13 +86,6 @@ enum qphy_reg_layout { > QPHY_LAYOUT_SIZE > }; >=20 >-static const unsigned int ipq_pciephy_gen3_regs_layout[QPHY_LAYOUT_SIZE]= =3D { >- [QPHY_SW_RESET] =3D 0x00, >- [QPHY_START_CTRL] =3D 0x44, >- [QPHY_PCS_STATUS] =3D 0x14, >- [QPHY_PCS_POWER_DOWN_CONTROL] =3D 0x40, >-}; >- > static const unsigned int pciephy_regs_layout[QPHY_LAYOUT_SIZE] =3D { > [QPHY_SW_RESET] =3D 0x00, > [QPHY_START_CTRL] =3D 0x08, >@@ -1492,7 +1485,7 @@ static const struct qmp_phy_cfg ipq8074_pciephy_gen= 3_cfg =3D { > =2Enum_resets =3D ARRAY_SIZE(ipq8074_pciephy_reset_l), > =2Evreg_list =3D NULL, > =2Enum_vregs =3D 0, >- =2Eregs =3D ipq_pciephy_gen3_regs_layout, >+ =2Eregs =3D sm8250_pcie_regs_layout, >=20 > =2Estart_ctrl =3D SERDES_START | PCS_START, > =2Epwrdn_ctrl =3D SW_PWRDN | REFCLK_DRV_DSBL, >@@ -1523,7 +1516,7 @@ static const struct qmp_phy_cfg ipq6018_pciephy_cfg= =3D { > =2Enum_resets =3D ARRAY_SIZE(ipq8074_pciephy_reset_l), > =2Evreg_list =3D NULL, > =2Enum_vregs =3D 0, >- =2Eregs =3D ipq_pciephy_gen3_regs_layout, >+ =2Eregs =3D sm8250_pcie_regs_layout, >=20 > =2Estart_ctrl =3D SERDES_START | PCS_START, > =2Epwrdn_ctrl =3D SW_PWRDN | REFCLK_DRV_DSBL, --=20 With best wishes Dmitry