On Tue, Oct 12, 2021 at 08:14:14PM +0200, Geert Uytterhoeven wrote:
> Hi Christian, Greg,
>
> CC devicetree, linux-renesas-soc
>
> On Tue, 8 May 2018, Christian Gromm wrote:
> > This patch removes the dependency to platform specific source files
> > that do platform specific initialization and supply the IRQ number.
> > Instead DT code is added
> >
> > Signed-off-by: Christian Gromm <christian.gromm@xxxxxxxxxxxxx>
>
> This patch bypassed review by the DT people, and ended up in v4.18 as
> commit 21e57ff086056c01 ("staging: most: dim2: use device tree").
Yeah, staging-only dt changes are not usually run by the dt maintainers.
>
> > --- a/drivers/staging/most/dim2/dim2.c
> > +++ b/drivers/staging/most/dim2/dim2.c
> > +static const struct of_device_id dim2_of_match[] = {
> > + {
> > + .compatible = "fsl,imx6q-mlb150",
> > + .data = plat_data + FSL_MX6
> > + },
> > + {
> > + .compatible = "renesas,mlp",
> > + .data = plat_data + RCAR_H2
> > + },
> > + {
> > + .compatible = "rcar,medialb-dim2",
> > + .data = plat_data + RCAR_M3
> > + },
> > + {
> > + .compatible = "xlnx,axi4-os62420_3pin-1.00.a",
> > + },
> > + {
> > + .compatible = "xlnx,axi4-os62420_6pin-1.00.a",
> > + },
> > + {},
> > };
>
> There are no documented DT bindings for this hardware block, nor any
> upstream example users. Given some compatible values do not follow
> standard practises (no idea about the other parts), it's very likely
> these de facto bindings, and all their out-of-tree users, will have to
> be changed.
Great, fix the bindings and anything in-kernel here please. We don't
care about out-of-kernel stuff for drivers/staging/ at all.
thanks,
greg k-h
