[PATCH] drm/imx: imx-ldb: Remove the leftover LVDS_BIT_MAP_SPWG/JEIDA enums

[Liu Ying]

Hi Macro,

On Thu, 2020-05-28 at 11:34 +0200, Marco Felsch wrote:
> Hi Liu,
> 
> pls check [1], I already send patches for it. Those stuck because we
> need to verify Philipp's proposal.

Yeah, '[PATCH 09/17] drm/imx: imx-ldb: remove useless enum' in your
patch series does the same thing as this patch does.  I'm fine to drop
this patch then.

Regarding Philipp's proposal on the use-after-free issue, we've got the
basically the same implemetation in downstream ldb driver to address
the issue.

Regards,
Liu Ying 

> 
> [1] 
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.spinics.net%2Flists%2Farm-kernel%2Fmsg789309.html&data=02%7C01%7Cvictor.liu%40nxp.com%7C8da3e383727f4fe1e0dd08d802ea5458%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637262552748306435&sdata=DWN%2F3jRT07qmPWxlgSTuD9eRiRVoOHWGRmfrgj0ZA3A%3D&reserved=0
> 
> Regards,
>   Marco
> 
> On 20-05-28 10:55, Liu Ying wrote:
> > Gentle ping...
> > 
> > On Tue, 2020-05-12 at 15:21 +0800, Liu Ying wrote:
> > > No one is using the LVDS_BIT_MAP_SPWG/JEIDA enums since a
> > > previous
> > > commit which changes the imx_ldb_bit_mappings[] array definition,
> > > so let's remove them.
> > > 
> > > Fixes: 5e501ed7253b ("drm/imx: imx-ldb: allow to determine bus
> > > format
> > > from the connected panel")
> > > Cc: Philipp Zabel <p.zabel at pengutronix.de>
> > > Cc: Sascha Hauer <s.hauer at pengutronix.de>
> > > Cc: Pengutronix Kernel Team <kernel at pengutronix.de>
> > > Cc: NXP Linux Team <linux-imx at nxp.com>
> > > Signed-off-by: Liu Ying <victor.liu at nxp.com>
> > > ---
> > >  drivers/gpu/drm/imx/imx-ldb.c | 5 -----
> > >  1 file changed, 5 deletions(-)
> > > 
> > > diff --git a/drivers/gpu/drm/imx/imx-ldb.c
> > > b/drivers/gpu/drm/imx/imx-
> > > ldb.c
> > > index 4da22a9..19a00eb 100644
> > > --- a/drivers/gpu/drm/imx/imx-ldb.c
> > > +++ b/drivers/gpu/drm/imx/imx-ldb.c
> > > @@ -477,11 +477,6 @@ static int imx_ldb_register(struct
> > > drm_device
> > > *drm,
> > >  	return 0;
> > >  }
> > >  
> > > -enum {
> > > -	LVDS_BIT_MAP_SPWG,
> > > -	LVDS_BIT_MAP_JEIDA
> > > -};
> > > -
> > >  struct imx_ldb_bit_mapping {
> > >  	u32 bus_format;
> > >  	u32 datawidth;
> > 
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel at lists.freedesktop.org
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Fdri-devel&data=02%7C01%7Cvictor.liu%40nxp.com%7C8da3e383727f4fe1e0dd08d802ea5458%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637262552748306435&sdata=isO%2Fp5kg9qJ8%2FLg5zqI8XazuxwHr1hPjiyTHFhy0n2o%3D&reserved=0
> >