[PATCH 0/6] Remove redundant spi driver bus initialization

[PATCH 0/6] Remove redundant spi driver bus initialization

[Antonio Borneo]

This cleanup was already completed between end 2011 and early 2012
with a patch series from Lars-Peter Clausen:
https://lkml.org/lkml/2011/11/24/190

Later on new redundant initialization re-appeared here and there.
Time to cleanup again.

And, yes, I'm lazy! I copy-paste the exact same commit message from
Lars-Peter; only minor reformat to stay in 75 char/line and fix the
name of spi_register_driver().

Regards,
Antonio

Antonio Borneo (6):
  ASoC: wm0010: Remove redundant spi driver bus initialization
  iio: ssp_sensors: Remove redundant spi driver bus initialization
  staging: mt29f_spinand: Remove redundant spi driver bus initialization
  net: ieee802154: Remove redundant spi driver bus initialization
  wireless: cw1200: Remove redundant spi driver bus initialization
  [media] s5c73m3: Remove redundant spi driver bus initialization

 drivers/iio/common/ssp_sensors/ssp_dev.c      | 1 -
 drivers/media/i2c/s5c73m3/s5c73m3-spi.c       | 1 -
 drivers/net/ieee802154/cc2520.c               | 1 -
 drivers/net/ieee802154/mrf24j40.c             | 1 -
 drivers/net/wireless/cw1200/cw1200_spi.c      | 1 -
 drivers/staging/mt29f_spinand/mt29f_spinand.c | 1 -
 sound/soc/codecs/wm0010.c                     | 1 -
 7 files changed, 7 deletions(-)

-- 
2.4.4

[PATCH 2/6] iio: ssp_sensors: Remove redundant spi driver bus initialization

[Antonio Borneo]

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@gmail.com>
To: Jonathan Cameron <jic23@kernel.org>
To: Lars-Peter Clausen <lars@metafoo.de>
To: Karol Wrona <k.wrona@samsung.com>
To: linux-iio@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
---
 drivers/iio/common/ssp_sensors/ssp_dev.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/iio/common/ssp_sensors/ssp_dev.c b/drivers/iio/common/ssp_sensors/ssp_dev.c
index 9a40097..d338bb5 100644
--- a/drivers/iio/common/ssp_sensors/ssp_dev.c
+++ b/drivers/iio/common/ssp_sensors/ssp_dev.c
@@ -700,7 +700,6 @@ static struct spi_driver ssp_driver = {
 	.remove = ssp_remove,
 	.driver = {
 		.pm = &ssp_pm_ops,
-		.bus = &spi_bus_type,
 		.owner = THIS_MODULE,
 		.of_match_table = of_match_ptr(ssp_of_match),
 		.name = "sensorhub"
-- 
2.4.4

Re: [PATCH 2/6] iio: ssp_sensors: Remove redundant spi driver bus initialization

[Karol Wrona]

On 06/23/2015 04:52 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@gmail.com>
> To: Jonathan Cameron <jic23@kernel.org>
> To: Lars-Peter Clausen <lars@metafoo.de>
> To: Karol Wrona <k.wrona@samsung.com>
> To: linux-iio@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> ---
>  drivers/iio/common/ssp_sensors/ssp_dev.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/iio/common/ssp_sensors/ssp_dev.c b/drivers/iio/common/ssp_sensors/ssp_dev.c
> index 9a40097..d338bb5 100644
> --- a/drivers/iio/common/ssp_sensors/ssp_dev.c
> +++ b/drivers/iio/common/ssp_sensors/ssp_dev.c
> @@ -700,7 +700,6 @@ static struct spi_driver ssp_driver = {
>  	.remove = ssp_remove,
>  	.driver = {
>  		.pm = &ssp_pm_ops,
> -		.bus = &spi_bus_type,
>  		.owner = THIS_MODULE,
>  		.of_match_table = of_match_ptr(ssp_of_match),
>  		.name = "sensorhub"
> 
Acked-by: Karol Wrona <k.wrona@samsung.com>

Re: [PATCH 2/6] iio: ssp_sensors: Remove redundant spi driver bus initialization

[Jonathan Cameron]

On 29/06/15 11:05, Karol Wrona wrote:
> On 06/23/2015 04:52 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@gmail.com>
>> To: Jonathan Cameron <jic23@kernel.org>
>> To: Lars-Peter Clausen <lars@metafoo.de>
>> To: Karol Wrona <k.wrona@samsung.com>
>> To: linux-iio@vger.kernel.org
>> Cc: linux-kernel@vger.kernel.org
>> ---
>>  drivers/iio/common/ssp_sensors/ssp_dev.c | 1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/drivers/iio/common/ssp_sensors/ssp_dev.c b/drivers/iio/common/ssp_sensors/ssp_dev.c
>> index 9a40097..d338bb5 100644
>> --- a/drivers/iio/common/ssp_sensors/ssp_dev.c
>> +++ b/drivers/iio/common/ssp_sensors/ssp_dev.c
>> @@ -700,7 +700,6 @@ static struct spi_driver ssp_driver = {
>>  	.remove = ssp_remove,
>>  	.driver = {
>>  		.pm = &ssp_pm_ops,
>> -		.bus = &spi_bus_type,
>>  		.owner = THIS_MODULE,
>>  		.of_match_table = of_match_ptr(ssp_of_match),
>>  		.name = "sensorhub"
>>
> Acked-by: Karol Wrona <k.wrona@samsung.com>
Applied to the togreg branch of iio.git

Thanks,

J
>