lkml.org 
[lkml]   [2020]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.9 346/757] pinctrl: aspeed: Use the right pinconf mask
    Date
    From: Andrew Jeffery <andrew@aj.id.au>

    [ Upstream commit 1d6db5ae6b090d1a8edfcb36b9bf47c5f4fe27f6 ]

    The Aspeed pinconf data structures are split into 'conf' and 'map'
    types, where the 'conf' struct defines which register and bitfield to
    manipulate, while the 'map' struct defines what value to write to
    the register and bitfield.

    Both structs have a mask member, and the wrong mask was being used to
    tell the regmap which bits to update.

    A todo is to look at whether we can remove the mask from the 'map'
    struct.

    Fixes: 5f52c853847f ("pinctrl: aspeed: Use masks to describe pinconf bitfields")
    Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
    Reviewed-by: Joel Stanley <joel@jms.id.au>
    Cc: Johnny Huang <johnny_huang@aspeedtech.com>
    Link: https://lore.kernel.org/r/20200910025631.2996342-3-andrew@aj.id.au
    Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/pinctrl/aspeed/pinctrl-aspeed.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/drivers/pinctrl/aspeed/pinctrl-aspeed.c b/drivers/pinctrl/aspeed/pinctrl-aspeed.c
    index 53f3f8aec6956..3e6567355d97d 100644
    --- a/drivers/pinctrl/aspeed/pinctrl-aspeed.c
    +++ b/drivers/pinctrl/aspeed/pinctrl-aspeed.c
    @@ -534,7 +534,7 @@ int aspeed_pin_config_set(struct pinctrl_dev *pctldev, unsigned int offset,
    val = pmap->val << __ffs(pconf->mask);

    rc = regmap_update_bits(pdata->scu, pconf->reg,
    - pmap->mask, val);
    + pconf->mask, val);

    if (rc < 0)
    return rc;
    --
    2.25.1


    \
     
     \ /
      Last update: 2020-10-27 17:37    [W:4.058 / U:0.076 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site