Re: [PATCH 6/6] [media] s5c73m3: Remove redundant spi driver bus initialization

From: Andrzej Hajda
Date: Wed Jun 24 2015 - 01:54:13 EST


On 06/23/2015 04:53 PM, Antonio Borneo wrote:
> In ancient times it was necessary to manually initialize the bus
> field of an spi_driver to spi_bus_type. These days this is done in
> spi_register_driver(), so we can drop the manual assignment.
>
> Signed-off-by: Antonio Borneo <borneo.antonio@xxxxxxxxx>
> To: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxx>
> To: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> To: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
> To: linux-media@xxxxxxxxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx

Reviewed-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx>

Regards
Andrzej
> ---
> drivers/media/i2c/s5c73m3/s5c73m3-spi.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/media/i2c/s5c73m3/s5c73m3-spi.c b/drivers/media/i2c/s5c73m3/s5c73m3-spi.c
> index 63eb190..fa4a5eb 100644
> --- a/drivers/media/i2c/s5c73m3/s5c73m3-spi.c
> +++ b/drivers/media/i2c/s5c73m3/s5c73m3-spi.c
> @@ -149,7 +149,6 @@ int s5c73m3_register_spi_driver(struct s5c73m3 *state)
> spidrv->remove = s5c73m3_spi_remove;
> spidrv->probe = s5c73m3_spi_probe;
> spidrv->driver.name = S5C73M3_SPI_DRV_NAME;
> - spidrv->driver.bus = &spi_bus_type;
> spidrv->driver.owner = THIS_MODULE;
> spidrv->driver.of_match_table = s5c73m3_spi_ids;
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/