[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200831091936.GM2639@vkoul-mobl>
Date: Mon, 31 Aug 2020 14:49:36 +0530
From: Vinod Koul <vkoul@...nel.org>
To: Wan Ahmad Zainie <wan.ahmad.zainie.wan.mohamad@...el.com>
Cc: kishon@...com, robh+dt@...nel.org, andriy.shevchenko@...el.com,
eswara.kota@...ux.intel.com,
vadivel.muruganx.ramuthevar@...ux.intel.com,
lakshmi.bai.raja.subramanian@...el.com,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org
Subject: Re: [PATCH v7 3/3] phy: intel: Add Keem Bay eMMC PHY support
On 21-08-20, 19:37, Wan Ahmad Zainie wrote:
> +/* From ACS_eMMC51_16nFFC_RO1100_Userguide_v1p0.pdf p17 */
> +#define FREQSEL_200M_170M 0x0
> +#define FREQSEL_170M_140M 0x1
> +#define FREQSEL_140M_110M 0x2
> +#define FREQSEL_110M_80M 0x3
> +#define FREQSEL_80M_50M 0x4
> +
> +#define maskval(mask, val) (((val) << (ffs(mask) - 1)) & mask)
Kernel has a macro do this for you, please use FIELD_PREP instead of
your own macro
--
~Vinod
Powered by blists - more mailing lists