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 X-Spam-Level: X-Spam-Status: No, score=-6.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9BD57C4361B for ; Fri, 11 Dec 2020 07:10:42 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 4F17123F2B for ; Fri, 11 Dec 2020 07:10:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2436662AbgLKHKM (ORCPT ); Fri, 11 Dec 2020 02:10:12 -0500 Received: from mail.kernel.org ([198.145.29.99]:57910 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2388329AbgLKHKA (ORCPT ); Fri, 11 Dec 2020 02:10:00 -0500 Content-Type: text/plain; charset="utf-8" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1607670559; bh=BXdz3hjV9jTQAtUU5cp0nkac+pcOQK0P5JoqsDaeC7I=; h=In-Reply-To:References:Subject:From:Cc:To:Date:From; b=pPwddj/XzCH/y5kSeZ1YEPHZIHLnsz48i3uN6f7SD5OPrPsI8yalnC9+FAQFOKq4p rquC9D6cGyeWgFOL0bZgRDaG+Z7uhtPUAiOA23M8k6zE2EhUF2l5OnXdu5cOoAekXC pgpOebCGNXoiUSbhDrjlfAxMEQj9PftJxHUxVJp1uQ3MFHQxbmONBqFIpICK/beVeP 724d79IZHW4KM8yejj4xwMNiyn+zMjMXin1BXNBs2ieWgOnoA6ozHAAFlwfQhA/1Wl s2AQgepvhfd1PqJH5/6FydT3dZoGO6DE5kj3vtNbfUlZJdMxDJTJoF4dGqV2HuMS2c kUqD7x6G2mjxQ== MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable In-Reply-To: <20201211050257.GR8403@vkoul-mobl> References: <20201208064702.3654324-1-vkoul@kernel.org> <20201208064702.3654324-5-vkoul@kernel.org> <160763259636.1580929.12912274485007017282@swboyd.mtv.corp.google.com> <20201211050257.GR8403@vkoul-mobl> Subject: Re: [PATCH v2 4/5] clk: qcom: clk-alpha-pll: Add support for Lucid 5LPE PLL From: Stephen Boyd Cc: linux-arm-msm@vger.kernel.org, Bjorn Andersson , Vivek Aknurwar , Andy Gross , Michael Turquette , Rob Herring , Taniya Das , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Jeevan Shriram To: Vinod Koul Date: Thu, 10 Dec 2020 23:09:18 -0800 Message-ID: <160767055805.1580929.10874243981268932091@swboyd.mtv.corp.google.com> User-Agent: alot/0.9.1 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Vinod Koul (2020-12-10 21:02:57) > On 10-12-20, 12:36, Stephen Boyd wrote: > > > + > > > + return regmap_update_bits(pll->clkr.regmap, PLL_USER_CTL(pll), > > > + (BIT(pll->width) - 1) << pll->post_di= v_shift, > >=20 > > Use GENMASK? >=20 > Looks like this can be: > GENMASK(pll->width + pll->post_div_shift - 1, pll->post_d= iv_shift) >=20 > Not sure which one you like :) Preferably a local u32 mask =3D GENMASK(...)