Re: [PATCH v3] drm: Drop explicit initialization of struct i2c_device_id::driver_data to 0

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

 



Hi,

On Wed, Dec 4, 2024 at 2:58 AM Uwe Kleine-König
<u.kleine-koenig@xxxxxxxxxxxx> wrote:
>
> These drivers don't use the driver_data member of struct i2c_device_id,
> so don't explicitly initialize this member.
>
> This prepares putting driver_data in an anonymous union which requires
> either no initialization or named designators. But it's also a nice
> cleanup on its own.
>
> While add it, also remove commas after the sentinel entries.
>
> Reviewed-by: Thomas Zimmermann <tzimmermann@xxxxxxx>
> Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>
> Reviewed-by: Liu Ying <victor.liu@xxxxxxx> # ite-it6263.c
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxx>
> ---
> Hello,
>
> On Mon, Dec 02, 2024 at 04:50:01PM -0800, Doug Anderson wrote in reply
> to v2:
> > This conflicts with commit dbf7986f8a56 ("drm/bridge: Constify struct
> > i2c_device_id"). Can you rebase, please?
> >
> > Other than that:
> >
> > Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>
>
> It took me a moment to find a tree + branch that contained the
> above mentioned commit. It would be a real time saver if drm-misc-next
> was included in next. At least saying "drm-misc-next" in such a context
> would be helpful, bonus points for mentioning
>
>         https://gitlab.freedesktop.org/drm/misc/kernel.git drm-misc-next

Sorry about that! drm-misc-next often feeds linux-next, but not
always. I'll try to remember to be more explicit in the future.

> The changes in this patch compared to v2,
> https://lore.kernel.org/dri-devel/20241202114731.1005549-2-u.kleine-koenig@xxxxxxxxxxxx/:
>
>  - Rebased to drm-misc-next, resolving conflicts in
>    drivers/gpu/drm/bridge/lontium-lt9611.c
>    drivers/gpu/drm/bridge/lontium-lt9611uxc.c
>    drivers/gpu/drm/bridge/ti-sn65dsi86.c
>
>  - Applied the R-b tag received for v2.
>
>  drivers/gpu/drm/bridge/analogix/analogix-anx6345.c       | 2 +-
>  drivers/gpu/drm/bridge/analogix/anx7625.c                | 2 +-
>  drivers/gpu/drm/bridge/chrontel-ch7033.c                 | 2 +-
>  drivers/gpu/drm/bridge/ite-it6263.c                      | 2 +-
>  drivers/gpu/drm/bridge/ite-it6505.c                      | 2 +-
>  drivers/gpu/drm/bridge/lontium-lt8912b.c                 | 4 ++--
>  drivers/gpu/drm/bridge/lontium-lt9611.c                  | 2 +-
>  drivers/gpu/drm/bridge/lontium-lt9611uxc.c               | 2 +-
>  drivers/gpu/drm/bridge/megachips-stdpxxxx-ge-b850v3-fw.c | 8 ++++----
>  drivers/gpu/drm/bridge/nxp-ptn3460.c                     | 4 ++--
>  drivers/gpu/drm/bridge/sii902x.c                         | 4 ++--
>  drivers/gpu/drm/bridge/sii9234.c                         | 4 ++--
>  drivers/gpu/drm/bridge/sil-sii8620.c                     | 4 ++--
>  drivers/gpu/drm/bridge/tc358767.c                        | 2 +-
>  drivers/gpu/drm/bridge/tc358768.c                        | 4 ++--
>  drivers/gpu/drm/bridge/ti-dlpc3433.c                     | 2 +-
>  drivers/gpu/drm/bridge/ti-sn65dsi86.c                    | 4 ++--
>  drivers/gpu/drm/bridge/ti-tfp410.c                       | 2 +-
>  drivers/gpu/drm/i2c/ch7006_drv.c                         | 2 +-
>  drivers/gpu/drm/i2c/sil164_drv.c                         | 2 +-
>  drivers/gpu/drm/i2c/tda9950.c                            | 4 ++--
>  drivers/gpu/drm/i2c/tda998x_drv.c                        | 2 +-
>  22 files changed, 33 insertions(+), 33 deletions(-)

Pushed to drm-misc-next:

[1/1] drm: Drop explicit initialization of struct
i2c_device_id::driver_data to 0
      commit: b690f5d955e033b9bd80de0486a04adf150f89a8




[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