Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 8 Nov 2022 14:25:47 +0100
From:   Thierry Reding <treding@nvidia.com>
To:     Ninad Malwade <nmalwade@nvidia.com>
Cc:     jonathanh@nvidia.com, linux@roeck-us.net, jdelvare@suse.com,
        nicolinc@nvidia.com, rkasirajan@nvidia.com,
        linux-hwmon@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-pm@vger.kernel.org
Subject: Re: [PATCH] ina3221: Fix shunt sum critical calculation
Message-ID: <Y2pY2zuJOII7ko5l@orome>
References: <20221108044508.23463-1-nmalwade@nvidia.com>
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="Dz+E06QEn44x9X3U"
Content-Disposition: inline
In-Reply-To: <20221108044508.23463-1-nmalwade@nvidia.com>
User-Agent: Mutt/2.2.7 (2022-08-07)
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?N9QLY7UL6T0v8osRDxBMNV2N9KXqxMGOY9Kr4tdVuQUnUad6cbnQa3crWE7q?=
 =?us-ascii?Q?K//XiKzO9swPLivkKfL+ek/MYO1Lg29MVNEL5gEhbnwJG9j8xv0jFUoVUPs9?=
 =?us-ascii?Q?/86vWrZsmRL8wQTR7fwR3QK+45KvAOLy2HxP1hObHMzHt+12ZsuPEKG8c9GJ?=
 =?us-ascii?Q?4YsvtInhmmMc3ip4aDzqkJeA79uSfp5SMWCXh36aV0o2R9Cd7bv8oFYmdxRX?=
 =?us-ascii?Q?ruQWVRzB+OPEJj0YPV+aEMNAw/rV1d4S1NcWJzzmPV1XOXlLOnGsgua8g4Yp?=
 =?us-ascii?Q?jr/qvaZ7azAX9peuBHj+W1THWXoT1P0xkrXR5pxUPtmesOvpGksnV2JVlXre?=
 =?us-ascii?Q?+Ot2TrAaXTD5eVMDue1RFDGaeq4yAlqyT/Na/W8ezFu8Uze2sM3JKf0pVo1C?=
 =?us-ascii?Q?jfPZyUsEtB3FiabJDTmH5AXUDyeM8FUHuJbjLseDS5BM3oEFdqUa9aDRnVFY?=
 =?us-ascii?Q?w7KKLzrfYw+Lq6yVs6tkME49uufr17+4if4JJ4q6S5V26L+mmYdLP/Fb6rcc?=
 =?us-ascii?Q?XND2dYDGVDW/nl/JxB8yOpuF5v5iHYbSUJpIBeCRSxl2rj3ZP/tbeuy+2Zsq?=
 =?us-ascii?Q?Wq0DrcGArkXgoa8LPbN7PhiZkUbA+Smw5LGcUFsniixFtNWPbKTIlT582fyQ?=
 =?us-ascii?Q?lDM296R6lIHP+Lg8GLfdGTvGDSjGHWQqzYFHZ4ZuL2FFO9Sc9kq9RJOcuVmT?=
 =?us-ascii?Q?uuGI8zpi3CcSnbyBap6BrIQvl4ltLl7y7tFthTRDZzQQ7qJ4lNL731kjm/OK?=
 =?us-ascii?Q?czZoH5CArhlX7znd70s7G6Z9lBuY5Ok6wLK8AtR+aL8fctkU9IbP+NLxHEyO?=
 =?us-ascii?Q?QgwqEolIbmI2CQEtAKPH5Y6TgB4MT/qyN0LvihrJRUs+TMkhINrDH/ElnlZ2?=
 =?us-ascii?Q?bflxDqCLZU3CzetShsDszPYTAufFL7ftOt8qpBq+hqrhSKgF6I+dAWO4aoso?=
 =?us-ascii?Q?zLz8fCh7N1e+AyjNlhXrq1+j3EaT88pwDaf3TDWxwYJCiZ/EMmDasDXKGxgF?=
 =?us-ascii?Q?NtnY5Y/lux0wOtVXB5O6NH4CmYLKluS3xAVL/+qG3DpP6PQyTeLEIngHeUAM?=
 =?us-ascii?Q?X/VukGF2wnUuQ1vsPGHvT+sTXGgGNGOrqIdPhdlyD8WkCB8iDOy7cmzT+N0J?=
 =?us-ascii?Q?ANI2+oxl1T/QIi/8abDfSjlzopeg99BRRas2uvjOD3Z2vf0qw2ATx+UEJP9w?=
 =?us-ascii?Q?XP2GhJYeeO8wpU3qwCWyjts5/sYoWiYnFvRFAS/8INQ+1DoeP/oNXZo3VNRW?=
 =?us-ascii?Q?8Afq1aDVA/bkwppsu54EROooVUEsX9d+9XOd/Etk19v7st1pBvmtaeYhB6dH?=
 =?us-ascii?Q?tccR8cfLcGL4lFc0Ez2KPCqMd6N3oUAdoRL1FCjuhMSpCgKxxATCcRqFhzfq?=
 =?us-ascii?Q?gtBFAIVTgVeN7TiPZ4M4UprrWxkb9g67fx7un3uclWBXTsj8GGYDjiv9pBdi?=
 =?us-ascii?Q?tIZPsvQm/b6C5nzYuxtrapxg6WiOPkeLfD3AgkoPT5EgSLOmE2sjZJqxwJn3?=
 =?us-ascii?Q?bk9mtfbvtKiZSVg9ikuzDHISbDazlKJptThgas4gnCHeu/4X6oguToaYaLRd?=
 =?us-ascii?Q?uc/PIFoh6s3zjIqLGm2bUviAluzV4HAZnlLzueD9bn6xl0RxtTSU0R7E6/jI?=
 =?us-ascii?Q?T0zGO53SEzilJXgBsz6WwcWqv+JL4/hrhRZ373fV+BGu?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5327df66-b1d4-4d6c-4dc8-08dac18cc363
X-MS-Exchange-CrossTenant-AuthSource: DM8PR12MB5413.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Nov 2022 13:25:54.5712
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 9Cnx0zB4JPJsThGWJL+WGu7mtJ2Qdbulbq4Q2InSCFZ9vyxdf0JFFPOrLhKesMImrilAJ6M8hvdCMuQp64Ullw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB6022
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE
        autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

--Dz+E06QEn44x9X3U
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Tue, Nov 08, 2022 at 12:45:08PM +0800, Ninad Malwade wrote:
> The shunt sum critical limit register value should be left shifted
> by one bit as its LSB-0 is a reserved bit.
>=20
> Signed-off-by: Ninad Malwade <nmalwade@nvidia.com>
> ---
>  drivers/hwmon/ina3221.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Looks correct:

Reviewed-by: Thierry Reding <treding@nvidia.com>

--Dz+E06QEn44x9X3U
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmNqWNsACgkQ3SOs138+
s6FHzBAAnMEN9nKG7OlDS8UebUlTaljXcj2Nux1w4gFpte6f6RCkCeK1yhFuoJZg
UiRRnX6rc2ZQV7v7pxQ4Obc6c0UlNLw1jEBcBimpbT9UpzqcCpT5etgOhSHlxGpa
02NVf1/cSul3WqYVOyoHhj2o5ZTpb/YbwGJnt/0NJBfkjxaxkU8/8Zoa6Db1VTHT
de81ZWZ1goshN1SSGbrVuMCRzuyPo1sWUaW7xfxx+J8+jHQYtfrixupEQdP/v4o9
dxJamUXqHNfkeWvH4AxURbrS2wvEAy+5HumxdKuylnvn/jtnR2rK51MFmUZiHq+R
vYacepcSah0aQJCruh62gNMfPVHYK2PkR7IWtW8q0ZsdQ7reAn/M9B3T1BstoKkc
EpmYygAAjngRhpH2cGtvFMOPW2c9IFng27iKUjQqMkHR4t03bXQSu+LlrqHjZj2h
V1d4g0G9pb8+mQyQSgLuW0+v5uTT/9RFc1uoD9RCuQKt735tFTvtGgj6wU1Qn1KK
AUjurxDgC8DzqwJCPhQckTp/fKpJmiB3K6j+dM77StEUhw6MuDrsPFl1ietcERn2
tB0WOpKWU2e12Z3UuCANgIHKK5EN0XPV+NmogfblgcC77MdNXIrR/+QyBg8lgvda
EDvlLvimsjeuFKWf1G72AjoVb9wzmsgZNoBEW9NoNirI/gpND6I=
=fca1
-----END PGP SIGNATURE-----

--Dz+E06QEn44x9X3U--