Re: [PATCH 3/3] i2c: jz4780: Remove of_match_ptr()

[Paul Cercueil]

Hi Rob,

Le lun. 14 sept. 2020 à 16:12, Rob Herring <robh@xxxxxxxxxx> a écrit :
> On Fri, Sep 04, 2020 at 03:11:52PM +0200, Paul Cercueil wrote:
> >  CONFIG_OF is selected by CONFIG_MACH_INGENIC, therefore we don't 
> > need to
> >  handle the case where Device Tree is not supported.
>
> What about COMPILE_TEST? If not supported, why not?

What about it? It will still compile fine with COMPILE_TEST.

-Paul

> >  Signed-off-by: Paul Cercueil <paul@xxxxxxxxxxxxxxx>
> >  ---
> >   drivers/i2c/busses/i2c-jz4780.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >  diff --git a/drivers/i2c/busses/i2c-jz4780.c 
> > b/drivers/i2c/busses/i2c-jz4780.c
> >  index ed2ec86f6f1a..cb4a25ebb890 100644
> >  --- a/drivers/i2c/busses/i2c-jz4780.c
> >  +++ b/drivers/i2c/busses/i2c-jz4780.c
> >  @@ -857,7 +857,7 @@ static struct platform_driver jz4780_i2c_driver 
> > = {
> >   	.remove		= jz4780_i2c_remove,
> >   	.driver		= {
> >   		.name	= "jz4780-i2c",
> >  -		.of_match_table = of_match_ptr(jz4780_i2c_of_matches),
> >  +		.of_match_table = jz4780_i2c_of_matches,
> >   	},
> >   };
> >
> >  --
> >  2.28.0