Re: [RFC bluetooth-next 18/21] mrf24j40: add cca ed level support

[Alexander Aring <alex.aring@gmail.com>]

On Thu, Aug 13, 2015 at 02:22:45PM +0200, Alexander Aring wrote:
> This patch supports handling to set the cca energy detection level for
> the mrf24j40 transceiver.
> 
> Signed-off-by: Alexander Aring <alex.aring@gmail.com>
> ---
>  drivers/net/ieee802154/mrf24j40.c | 51 ++++++++++++++++++++++++++++++++++++++-
>  1 file changed, 50 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ieee802154/mrf24j40.c b/drivers/net/ieee802154/mrf24j40.c
> index 0fc7628..00fd97e 100644
> --- a/drivers/net/ieee802154/mrf24j40.c
> +++ b/drivers/net/ieee802154/mrf24j40.c
> @@ -820,6 +820,49 @@ static int mrf24j40_set_cca_mode(struct ieee802154_hw *hw,
>  				  val << 6);
>  }
>  
> +/* array for representing ed levels */
> +static const s32 mrf24j40_ed_levels[] = {
> +	-9000, -8900, -8800, -8700, -8600, -8500, -8400, -8300, -8200, -8100,
> +	-8000, -7900, -7800, -7700, -7600, -7500, -7400, -7300, -7200, -7100,
> +	-7000, -6900, -6800, -6700, -6600, -6500, -6400, -6300, -6200, -6100,
> +	-6000, -5900, -5800, -5700, -5600, -5500, -5400, -5300, -5200, -5100,
> +	-5000, -4900, -4800, -4700, -4600, -4500, -4400, -4300, -4200, -4100,
> +	-4000, -3900, -3800, -3700, -3600, -3500
> +};
> +
> +/* map ed levels to register value */
> +static const s32 mrf24j40_ed_levels_map[][2] = {
> +	{ -9000, 0 }, { -8900, 1 }, { -8800, 2 }, { -8700, 5 }, { -8600, 9 },
> +	{ -8500, 13 }, { -8400, 18 }, { -8300, 23 }, { -8200, 27 },
> +	{ -8100, 32 }, { -8000, 37 }, { -7900, 43 }, { -7800, 48 },
> +	{ -7700, 53 }, { -7600, 58 }, { -7500, 63 }, { -7400, 68 },
> +	{ -7300, 73 }, { -7200, 78 }, { -7100, 83 }, { -7000, 89 },
> +	{ -6900, 95 }, { -6800, 100 }, { -6700, 107 }, { -6600, 111 },
> +	{ -6500, 117 }, { -6400, 121 }, { -6300, 125 }, { -6200, 129 },
> +	{ -6100, 133 },	{ -6000, 138 }, { -5900, 143 }, { -5800, 148 },
> +	{ -5700, 153 }, { -5600, 159 },	{ -5500, 165 }, { -5400, 170 },
> +	{ -5300, 176 }, { -5200, 183 }, { -5100, 188 }, { -5000, 193 },
> +	{ -4900, 198 }, { -4800, 203 }, { -4700, 207 }, { -4600, 212 },
> +	{ -4500, 216 }, { -4400, 221 }, { -4300, 225 }, { -4200, 228 },
> +	{ -4100, 233 }, { -4000, 239 }, { -3900, 245 }, { -3800, 250 },
> +	{ -3700, 253 }, { -3600, 254 }, { -3500, 255 },
> +};
> +
> +static int mrf24j40_set_cca_ed_level(struct ieee802154_hw *hw, s32 mbm)
> +{
> +	struct mrf24j40 *devrec = hw->priv;
> +	int i;
> +
> +	for (i = 0; i < ARRAY_SIZE(mrf24j40_ed_levels_map); i++) {
> +		if (mrf24j40_ed_levels_map[i][0] == mbm)
> +			return regmap_update_bits(devrec->regmap_short,
> +						  REG_CCAEDTH, 0xff,
> +						  mrf24j40_ed_levels_map[i][1]);

I changed this function from regmap_update_bits to regmap_write, the
mask is "0xff" here and then regmap_write is enough.

- Alex