Re: [PATCH] drm/bridge: ti-sn65dsi83: Fix sparse warnings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hey Marek,

Thanks for submitting this.

On Tue, 8 Jun 2021 at 16:22, Marek Vasut <marex@xxxxxxx> wrote:
>
> Fix the following sparse warnings generated by "make C=1":
>
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:429:13: warning: incorrect type in assignment (different base types)
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:429:13:    expected unsigned short [assigned] [usertype] val
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:429:13:    got restricted __le16 [usertype]
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:432:13: warning: incorrect type in assignment (different base types)
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:432:13:    expected unsigned short [addressable] [assigned] [usertype] val
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:432:13:    got restricted __le16 [usertype]
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:436:13: warning: incorrect type in assignment (different base types)
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:436:13:    expected unsigned short [addressable] [assigned] [usertype] val
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:436:13:    got restricted __le16 [usertype]
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:438:13: warning: incorrect type in assignment (different base types)
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:438:13:    expected unsigned short [addressable] [assigned] [usertype] val
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:438:13:    got restricted __le16 [usertype]
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:441:13: warning: incorrect type in assignment (different base types)
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:441:13:    expected unsigned short [addressable] [assigned] [usertype] val
> drivers/gpu/drm/bridge/ti-sn65dsi83.c:441:13:    got restricted __le16 [usertype]
>
> Signed-off-by: Marek Vasut <marex@xxxxxxx>
> Cc: Adam Ford <aford173@xxxxxxxxx>
> Cc: Douglas Anderson <dianders@xxxxxxxxxxxx>
> Cc: Frieder Schrempf <frieder.schrempf@xxxxxxxxxx>
> Cc: Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx>
> Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>
> Cc: Loic Poulain <loic.poulain@xxxxxxxxxx>
> Cc: Marek Vasut <marex@xxxxxxx>
> Cc: Philippe Schenker <philippe.schenker@xxxxxxxxxxx>
> Cc: Robert Foss <robert.foss@xxxxxxxxxx>
> Cc: Sam Ravnborg <sam@xxxxxxxxxxxx>
> Cc: Stephen Boyd <swboyd@xxxxxxxxxxxx>
> Cc: Valentin Raevsky <valentin@xxxxxxxxxxxxxx>
> To: dri-devel@xxxxxxxxxxxxxxxxxxxxx
> ---
>  drivers/gpu/drm/bridge/ti-sn65dsi83.c | 21 +++++++++++----------
>  1 file changed, 11 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi83.c b/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> index 750f2172ef08..8f3158f5281a 100644
> --- a/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> +++ b/drivers/gpu/drm/bridge/ti-sn65dsi83.c
> @@ -368,6 +368,7 @@ static void sn65dsi83_enable(struct drm_bridge *bridge)
>  {
>         struct sn65dsi83 *ctx = bridge_to_sn65dsi83(bridge);
>         unsigned int pval;
> +       __le16 le16val;
>         u16 val;
>         int ret;
>
> @@ -426,21 +427,21 @@ static void sn65dsi83_enable(struct drm_bridge *bridge)
>                      REG_LVDS_LANE_CHB_LVDS_TERM);
>         regmap_write(ctx->regmap, REG_LVDS_CM, 0x00);
>
> -       val = cpu_to_le16(ctx->mode.hdisplay);
> +       le16val = cpu_to_le16(ctx->mode.hdisplay);
>         regmap_bulk_write(ctx->regmap, REG_VID_CHA_ACTIVE_LINE_LENGTH_LOW,
> -                         &val, 2);
> -       val = cpu_to_le16(ctx->mode.vdisplay);
> +                         &le16val, 2);
> +       le16val = cpu_to_le16(ctx->mode.vdisplay);
>         regmap_bulk_write(ctx->regmap, REG_VID_CHA_VERTICAL_DISPLAY_SIZE_LOW,
> -                         &val, 2);
> +                         &le16val, 2);
>         /* 32 + 1 pixel clock to ensure proper operation */
> -       val = cpu_to_le16(32 + 1);
> -       regmap_bulk_write(ctx->regmap, REG_VID_CHA_SYNC_DELAY_LOW, &val, 2);
> -       val = cpu_to_le16(ctx->mode.hsync_end - ctx->mode.hsync_start);
> +       le16val = cpu_to_le16(32 + 1);
> +       regmap_bulk_write(ctx->regmap, REG_VID_CHA_SYNC_DELAY_LOW, &le16val, 2);
> +       le16val = cpu_to_le16(ctx->mode.hsync_end - ctx->mode.hsync_start);
>         regmap_bulk_write(ctx->regmap, REG_VID_CHA_HSYNC_PULSE_WIDTH_LOW,
> -                         &val, 2);
> -       val = cpu_to_le16(ctx->mode.vsync_end - ctx->mode.vsync_start);
> +                         &le16val, 2);
> +       le16val = cpu_to_le16(ctx->mode.vsync_end - ctx->mode.vsync_start);
>         regmap_bulk_write(ctx->regmap, REG_VID_CHA_VSYNC_PULSE_WIDTH_LOW,
> -                         &val, 2);
> +                         &le16val, 2);
>         regmap_write(ctx->regmap, REG_VID_CHA_HORIZONTAL_BACK_PORCH,
>                      ctx->mode.htotal - ctx->mode.hsync_end);
>         regmap_write(ctx->regmap, REG_VID_CHA_VERTICAL_BACK_PORCH,

Reviewed and pulled into drm-misc-next.

https://cgit.freedesktop.org/drm/drm-misc/commit/?id=241a9e233e7cf1001d09426c9fc520768f8961bf



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux