* [PATCH 0/5] of_platform_driver and OF_DEVICE removal @ 2013-04-22 2:13 Rob Herring 2013-04-22 2:13 ` [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver Rob Herring 2013-04-22 7:03 ` [PATCH 0/5] of_platform_driver and OF_DEVICE removal Arnd Bergmann 0 siblings, 2 replies; 12+ messages in thread From: Rob Herring @ 2013-04-22 2:13 UTC (permalink / raw) To: linux-kernel, devicetree-discuss Cc: Roland Dreier, Sean Hefty, Arnd Bergmann, Alexander Shishkin, linux-usb, Rob Herring, linux-rdma, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, Greg Kroah-Hartman, Grant Likely, Paul Mackerras, netdev, linuxppc-dev, Hal Rosenstock From: Rob Herring <rob.herring@calxeda.com> This series is a relatively straight-forward removal of the last remaining user of of_platform_driver (ibmebus) and removal of CONFIG_OF_DEVICE which is always enabled when CONFIG_OF is enabled. Compile tested on powerpc and sparc. Rob Rob Herring (5): ibmebus: convert of_platform_driver to platform_driver driver core: move to_platform_driver to platform_device.h of: remove of_platform_driver usb: chipidea: depend on CONFIG_OF instead of CONFIG_OF_DEVICE of: remove CONFIG_OF_DEVICE arch/powerpc/include/asm/ibmebus.h | 4 ++-- arch/powerpc/kernel/ibmebus.c | 22 ++++++++++----------- drivers/base/platform.c | 3 --- drivers/infiniband/hw/ehca/ehca_main.c | 5 ++--- drivers/net/ethernet/ibm/ehea/ehea_main.c | 8 +++----- drivers/of/Kconfig | 3 --- drivers/of/Makefile | 3 +-- drivers/usb/chipidea/Makefile | 2 +- include/linux/of_device.h | 6 +++--- include/linux/of_platform.h | 30 +---------------------------- include/linux/platform_device.h | 3 +++ 11 files changed, 26 insertions(+), 63 deletions(-) -- 1.7.10.4 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver 2013-04-22 2:13 [PATCH 0/5] of_platform_driver and OF_DEVICE removal Rob Herring @ 2013-04-22 2:13 ` Rob Herring 2013-04-25 15:23 ` Rob Herring 2013-04-22 7:03 ` [PATCH 0/5] of_platform_driver and OF_DEVICE removal Arnd Bergmann 1 sibling, 1 reply; 12+ messages in thread From: Rob Herring @ 2013-04-22 2:13 UTC (permalink / raw) To: linux-kernel, devicetree-discuss Cc: Roland Dreier, Arnd Bergmann, linux-rdma, Rob Herring, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, netdev, Grant Likely, Paul Mackerras, Sean Hefty, linuxppc-dev, Hal Rosenstock From: Rob Herring <rob.herring@calxeda.com> ibmebus is the last remaining user of of_platform_driver and the conversion to a regular platform driver is trivial. Signed-off-by: Rob Herring <rob.herring@calxeda.com> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> Cc: Paul Mackerras <paulus@samba.org> Cc: Hoang-Nam Nguyen <hnguyen@de.ibm.com> Cc: Christoph Raisch <raisch@de.ibm.com> Cc: Roland Dreier <roland@kernel.org> Cc: Sean Hefty <sean.hefty@intel.com> Cc: Hal Rosenstock <hal.rosenstock@gmail.com> Cc: Thadeu Lima de Souza Cascardo <cascardo@linux.vnet.ibm.com> Cc: Grant Likely <grant.likely@linaro.org> Cc: linuxppc-dev@lists.ozlabs.org Cc: linux-rdma@vger.kernel.org Cc: netdev@vger.kernel.org --- arch/powerpc/include/asm/ibmebus.h | 4 ++-- arch/powerpc/kernel/ibmebus.c | 22 ++++++++++------------ drivers/infiniband/hw/ehca/ehca_main.c | 5 ++--- drivers/net/ethernet/ibm/ehea/ehea_main.c | 8 +++----- 4 files changed, 17 insertions(+), 22 deletions(-) diff --git a/arch/powerpc/include/asm/ibmebus.h b/arch/powerpc/include/asm/ibmebus.h index 1a9d9ae..088f95b 100644 --- a/arch/powerpc/include/asm/ibmebus.h +++ b/arch/powerpc/include/asm/ibmebus.h @@ -48,8 +48,8 @@ extern struct bus_type ibmebus_bus_type; -int ibmebus_register_driver(struct of_platform_driver *drv); -void ibmebus_unregister_driver(struct of_platform_driver *drv); +int ibmebus_register_driver(struct platform_driver *drv); +void ibmebus_unregister_driver(struct platform_driver *drv); int ibmebus_request_irq(u32 ist, irq_handler_t handler, unsigned long irq_flags, const char *devname, diff --git a/arch/powerpc/kernel/ibmebus.c b/arch/powerpc/kernel/ibmebus.c index 8220baa..16a7c23 100644 --- a/arch/powerpc/kernel/ibmebus.c +++ b/arch/powerpc/kernel/ibmebus.c @@ -205,7 +205,7 @@ static int ibmebus_create_devices(const struct of_device_id *matches) return ret; } -int ibmebus_register_driver(struct of_platform_driver *drv) +int ibmebus_register_driver(struct platform_driver *drv) { /* If the driver uses devices that ibmebus doesn't know, add them */ ibmebus_create_devices(drv->driver.of_match_table); @@ -215,7 +215,7 @@ int ibmebus_register_driver(struct of_platform_driver *drv) } EXPORT_SYMBOL(ibmebus_register_driver); -void ibmebus_unregister_driver(struct of_platform_driver *drv) +void ibmebus_unregister_driver(struct platform_driver *drv) { driver_unregister(&drv->driver); } @@ -338,11 +338,10 @@ static int ibmebus_bus_bus_match(struct device *dev, struct device_driver *drv) static int ibmebus_bus_device_probe(struct device *dev) { int error = -ENODEV; - struct of_platform_driver *drv; + struct platform_driver *drv; struct platform_device *of_dev; - const struct of_device_id *match; - drv = to_of_platform_driver(dev->driver); + drv = to_platform_driver(dev->driver); of_dev = to_platform_device(dev); if (!drv->probe) @@ -350,9 +349,8 @@ static int ibmebus_bus_device_probe(struct device *dev) of_dev_get(of_dev); - match = of_match_device(drv->driver.of_match_table, dev); - if (match) - error = drv->probe(of_dev, match); + if (of_driver_match_device(dev, dev->driver)) + error = drv->probe(of_dev); if (error) of_dev_put(of_dev); @@ -362,7 +360,7 @@ static int ibmebus_bus_device_probe(struct device *dev) static int ibmebus_bus_device_remove(struct device *dev) { struct platform_device *of_dev = to_platform_device(dev); - struct of_platform_driver *drv = to_of_platform_driver(dev->driver); + struct platform_driver *drv = to_platform_driver(dev->driver); if (dev->driver && drv->remove) drv->remove(of_dev); @@ -372,7 +370,7 @@ static int ibmebus_bus_device_remove(struct device *dev) static void ibmebus_bus_device_shutdown(struct device *dev) { struct platform_device *of_dev = to_platform_device(dev); - struct of_platform_driver *drv = to_of_platform_driver(dev->driver); + struct platform_driver *drv = to_platform_driver(dev->driver); if (dev->driver && drv->shutdown) drv->shutdown(of_dev); @@ -419,7 +417,7 @@ struct device_attribute ibmebus_bus_device_attrs[] = { static int ibmebus_bus_legacy_suspend(struct device *dev, pm_message_t mesg) { struct platform_device *of_dev = to_platform_device(dev); - struct of_platform_driver *drv = to_of_platform_driver(dev->driver); + struct platform_driver *drv = to_platform_driver(dev->driver); int ret = 0; if (dev->driver && drv->suspend) @@ -430,7 +428,7 @@ static int ibmebus_bus_legacy_suspend(struct device *dev, pm_message_t mesg) static int ibmebus_bus_legacy_resume(struct device *dev) { struct platform_device *of_dev = to_platform_device(dev); - struct of_platform_driver *drv = to_of_platform_driver(dev->driver); + struct platform_driver *drv = to_platform_driver(dev->driver); int ret = 0; if (dev->driver && drv->resume) diff --git a/drivers/infiniband/hw/ehca/ehca_main.c b/drivers/infiniband/hw/ehca/ehca_main.c index f8a6291..982e3ef 100644 --- a/drivers/infiniband/hw/ehca/ehca_main.c +++ b/drivers/infiniband/hw/ehca/ehca_main.c @@ -713,8 +713,7 @@ static struct attribute_group ehca_dev_attr_grp = { .attrs = ehca_dev_attrs }; -static int ehca_probe(struct platform_device *dev, - const struct of_device_id *id) +static int ehca_probe(struct platform_device *dev) { struct ehca_shca *shca; const u64 *handle; @@ -937,7 +936,7 @@ static struct of_device_id ehca_device_table[] = }; MODULE_DEVICE_TABLE(of, ehca_device_table); -static struct of_platform_driver ehca_driver = { +static struct platform_driver ehca_driver = { .probe = ehca_probe, .remove = ehca_remove, .driver = { diff --git a/drivers/net/ethernet/ibm/ehea/ehea_main.c b/drivers/net/ethernet/ibm/ehea/ehea_main.c index 328f47c..c6ecf70 100644 --- a/drivers/net/ethernet/ibm/ehea/ehea_main.c +++ b/drivers/net/ethernet/ibm/ehea/ehea_main.c @@ -98,8 +98,7 @@ static struct ehea_fw_handle_array ehea_fw_handles; static struct ehea_bcmc_reg_array ehea_bcmc_regs; -static int ehea_probe_adapter(struct platform_device *dev, - const struct of_device_id *id); +static int ehea_probe_adapter(struct platform_device *dev); static int ehea_remove(struct platform_device *dev); @@ -112,7 +111,7 @@ static struct of_device_id ehea_device_table[] = { }; MODULE_DEVICE_TABLE(of, ehea_device_table); -static struct of_platform_driver ehea_driver = { +static struct platform_driver ehea_driver = { .driver = { .name = "ehea", .owner = THIS_MODULE, @@ -3255,8 +3254,7 @@ static void ehea_remove_device_sysfs(struct platform_device *dev) device_remove_file(&dev->dev, &dev_attr_remove_port); } -static int ehea_probe_adapter(struct platform_device *dev, - const struct of_device_id *id) +static int ehea_probe_adapter(struct platform_device *dev) { struct ehea_adapter *adapter; const u64 *adapter_handle; -- 1.7.10.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver 2013-04-22 2:13 ` [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver Rob Herring @ 2013-04-25 15:23 ` Rob Herring 2013-04-25 17:35 ` Benjamin Herrenschmidt 0 siblings, 1 reply; 12+ messages in thread From: Rob Herring @ 2013-04-25 15:23 UTC (permalink / raw) To: linux-kernel@vger.kernel.org, devicetree-discuss@lists.ozlabs.org, Benjamin Herrenschmidt Cc: Roland Dreier, Arnd Bergmann, linux-rdma, netdev, Rob Herring, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, Grant Likely, Paul Mackerras, Sean Hefty, linuxppc-dev, Hal Rosenstock On Sun, Apr 21, 2013 at 9:13 PM, Rob Herring <robherring2@gmail.com> wrote: > From: Rob Herring <rob.herring@calxeda.com> > > ibmebus is the last remaining user of of_platform_driver and the > conversion to a regular platform driver is trivial. > > Signed-off-by: Rob Herring <rob.herring@calxeda.com> > Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> > Cc: Paul Mackerras <paulus@samba.org> > Cc: Hoang-Nam Nguyen <hnguyen@de.ibm.com> > Cc: Christoph Raisch <raisch@de.ibm.com> > Cc: Roland Dreier <roland@kernel.org> > Cc: Sean Hefty <sean.hefty@intel.com> > Cc: Hal Rosenstock <hal.rosenstock@gmail.com> > Cc: Thadeu Lima de Souza Cascardo <cascardo@linux.vnet.ibm.com> > Cc: Grant Likely <grant.likely@linaro.org> > Cc: linuxppc-dev@lists.ozlabs.org > Cc: linux-rdma@vger.kernel.org > Cc: netdev@vger.kernel.org Ben, Can I have your Ack for this? The change is straightforward and neither of the 2 drivers used the id parameter that is removed. Rob > --- > arch/powerpc/include/asm/ibmebus.h | 4 ++-- > arch/powerpc/kernel/ibmebus.c | 22 ++++++++++------------ > drivers/infiniband/hw/ehca/ehca_main.c | 5 ++--- > drivers/net/ethernet/ibm/ehea/ehea_main.c | 8 +++----- > 4 files changed, 17 insertions(+), 22 deletions(-) > > diff --git a/arch/powerpc/include/asm/ibmebus.h b/arch/powerpc/include/asm/ibmebus.h > index 1a9d9ae..088f95b 100644 > --- a/arch/powerpc/include/asm/ibmebus.h > +++ b/arch/powerpc/include/asm/ibmebus.h > @@ -48,8 +48,8 @@ > > extern struct bus_type ibmebus_bus_type; > > -int ibmebus_register_driver(struct of_platform_driver *drv); > -void ibmebus_unregister_driver(struct of_platform_driver *drv); > +int ibmebus_register_driver(struct platform_driver *drv); > +void ibmebus_unregister_driver(struct platform_driver *drv); > > int ibmebus_request_irq(u32 ist, irq_handler_t handler, > unsigned long irq_flags, const char *devname, > diff --git a/arch/powerpc/kernel/ibmebus.c b/arch/powerpc/kernel/ibmebus.c > index 8220baa..16a7c23 100644 > --- a/arch/powerpc/kernel/ibmebus.c > +++ b/arch/powerpc/kernel/ibmebus.c > @@ -205,7 +205,7 @@ static int ibmebus_create_devices(const struct of_device_id *matches) > return ret; > } > > -int ibmebus_register_driver(struct of_platform_driver *drv) > +int ibmebus_register_driver(struct platform_driver *drv) > { > /* If the driver uses devices that ibmebus doesn't know, add them */ > ibmebus_create_devices(drv->driver.of_match_table); > @@ -215,7 +215,7 @@ int ibmebus_register_driver(struct of_platform_driver *drv) > } > EXPORT_SYMBOL(ibmebus_register_driver); > > -void ibmebus_unregister_driver(struct of_platform_driver *drv) > +void ibmebus_unregister_driver(struct platform_driver *drv) > { > driver_unregister(&drv->driver); > } > @@ -338,11 +338,10 @@ static int ibmebus_bus_bus_match(struct device *dev, struct device_driver *drv) > static int ibmebus_bus_device_probe(struct device *dev) > { > int error = -ENODEV; > - struct of_platform_driver *drv; > + struct platform_driver *drv; > struct platform_device *of_dev; > - const struct of_device_id *match; > > - drv = to_of_platform_driver(dev->driver); > + drv = to_platform_driver(dev->driver); > of_dev = to_platform_device(dev); > > if (!drv->probe) > @@ -350,9 +349,8 @@ static int ibmebus_bus_device_probe(struct device *dev) > > of_dev_get(of_dev); > > - match = of_match_device(drv->driver.of_match_table, dev); > - if (match) > - error = drv->probe(of_dev, match); > + if (of_driver_match_device(dev, dev->driver)) > + error = drv->probe(of_dev); > if (error) > of_dev_put(of_dev); > > @@ -362,7 +360,7 @@ static int ibmebus_bus_device_probe(struct device *dev) > static int ibmebus_bus_device_remove(struct device *dev) > { > struct platform_device *of_dev = to_platform_device(dev); > - struct of_platform_driver *drv = to_of_platform_driver(dev->driver); > + struct platform_driver *drv = to_platform_driver(dev->driver); > > if (dev->driver && drv->remove) > drv->remove(of_dev); > @@ -372,7 +370,7 @@ static int ibmebus_bus_device_remove(struct device *dev) > static void ibmebus_bus_device_shutdown(struct device *dev) > { > struct platform_device *of_dev = to_platform_device(dev); > - struct of_platform_driver *drv = to_of_platform_driver(dev->driver); > + struct platform_driver *drv = to_platform_driver(dev->driver); > > if (dev->driver && drv->shutdown) > drv->shutdown(of_dev); > @@ -419,7 +417,7 @@ struct device_attribute ibmebus_bus_device_attrs[] = { > static int ibmebus_bus_legacy_suspend(struct device *dev, pm_message_t mesg) > { > struct platform_device *of_dev = to_platform_device(dev); > - struct of_platform_driver *drv = to_of_platform_driver(dev->driver); > + struct platform_driver *drv = to_platform_driver(dev->driver); > int ret = 0; > > if (dev->driver && drv->suspend) > @@ -430,7 +428,7 @@ static int ibmebus_bus_legacy_suspend(struct device *dev, pm_message_t mesg) > static int ibmebus_bus_legacy_resume(struct device *dev) > { > struct platform_device *of_dev = to_platform_device(dev); > - struct of_platform_driver *drv = to_of_platform_driver(dev->driver); > + struct platform_driver *drv = to_platform_driver(dev->driver); > int ret = 0; > > if (dev->driver && drv->resume) > diff --git a/drivers/infiniband/hw/ehca/ehca_main.c b/drivers/infiniband/hw/ehca/ehca_main.c > index f8a6291..982e3ef 100644 > --- a/drivers/infiniband/hw/ehca/ehca_main.c > +++ b/drivers/infiniband/hw/ehca/ehca_main.c > @@ -713,8 +713,7 @@ static struct attribute_group ehca_dev_attr_grp = { > .attrs = ehca_dev_attrs > }; > > -static int ehca_probe(struct platform_device *dev, > - const struct of_device_id *id) > +static int ehca_probe(struct platform_device *dev) > { > struct ehca_shca *shca; > const u64 *handle; > @@ -937,7 +936,7 @@ static struct of_device_id ehca_device_table[] = > }; > MODULE_DEVICE_TABLE(of, ehca_device_table); > > -static struct of_platform_driver ehca_driver = { > +static struct platform_driver ehca_driver = { > .probe = ehca_probe, > .remove = ehca_remove, > .driver = { > diff --git a/drivers/net/ethernet/ibm/ehea/ehea_main.c b/drivers/net/ethernet/ibm/ehea/ehea_main.c > index 328f47c..c6ecf70 100644 > --- a/drivers/net/ethernet/ibm/ehea/ehea_main.c > +++ b/drivers/net/ethernet/ibm/ehea/ehea_main.c > @@ -98,8 +98,7 @@ static struct ehea_fw_handle_array ehea_fw_handles; > static struct ehea_bcmc_reg_array ehea_bcmc_regs; > > > -static int ehea_probe_adapter(struct platform_device *dev, > - const struct of_device_id *id); > +static int ehea_probe_adapter(struct platform_device *dev); > > static int ehea_remove(struct platform_device *dev); > > @@ -112,7 +111,7 @@ static struct of_device_id ehea_device_table[] = { > }; > MODULE_DEVICE_TABLE(of, ehea_device_table); > > -static struct of_platform_driver ehea_driver = { > +static struct platform_driver ehea_driver = { > .driver = { > .name = "ehea", > .owner = THIS_MODULE, > @@ -3255,8 +3254,7 @@ static void ehea_remove_device_sysfs(struct platform_device *dev) > device_remove_file(&dev->dev, &dev_attr_remove_port); > } > > -static int ehea_probe_adapter(struct platform_device *dev, > - const struct of_device_id *id) > +static int ehea_probe_adapter(struct platform_device *dev) > { > struct ehea_adapter *adapter; > const u64 *adapter_handle; > -- > 1.7.10.4 > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver 2013-04-25 15:23 ` Rob Herring @ 2013-04-25 17:35 ` Benjamin Herrenschmidt 2013-04-25 19:14 ` Rob Herring 0 siblings, 1 reply; 12+ messages in thread From: Benjamin Herrenschmidt @ 2013-04-25 17:35 UTC (permalink / raw) To: Rob Herring Cc: Roland Dreier, Arnd Bergmann, linux-rdma, netdev, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, Grant Likely, Paul Mackerras, Sean Hefty, linuxppc-dev, Hal Rosenstock On Thu, 2013-04-25 at 10:23 -0500, Rob Herring wrote: > Ben, Can I have your Ack for this? The change is straightforward and > neither of the 2 drivers used the id parameter that is removed. Didn't you get my mail about a compile failure caused by this patch ? Or did you send an update that I missed ? (Copy of the original email below) Cheers, Ben. ---- On Sun, 2013-04-21 at 21:13 -0500, Rob Herring wrote: > From: Rob Herring <rob.herring@calxeda.com> > > ibmebus is the last remaining user of of_platform_driver and the > conversion to a regular platform driver is trivial. A quick build test leads to: /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c: In function 'ibmebus_bus_device_probe': /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:344:2: error: implicit declaration of function 'to_platform_driver' [-Werror=implicit-function-declaration] /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:344:6: error: assignment makes pointer from integer without a cast [-Werror] /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c: In function 'ibmebus_bus_device_remove': /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:363:32: error: initialization makes pointer from integer without a cast [-Werror] /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c: In function 'ibmebus_bus_device_shutdown': /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:373:32: error: initialization makes pointer from integer without a cast [-Werror] /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c: In function 'ibmebus_bus_legacy_suspend': /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:420:32: error: initialization makes pointer from integer without a cast [-Werror] /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c: In function 'ibmebus_bus_legacy_resume': /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:431:32: error: initialization makes pointer from integer without a cast [-Werror] cc1: all warnings being treated as errors make[2]: *** [arch/powerpc/kernel/ibmebus.o] Error 1 make[2]: *** Waiting for unfinished jobs.... Haven't had a chance to look too closely today -EJETLAG :-) ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver 2013-04-25 17:35 ` Benjamin Herrenschmidt @ 2013-04-25 19:14 ` Rob Herring 2013-04-25 20:12 ` Benjamin Herrenschmidt 0 siblings, 1 reply; 12+ messages in thread From: Rob Herring @ 2013-04-25 19:14 UTC (permalink / raw) To: Benjamin Herrenschmidt Cc: Roland Dreier, Arnd Bergmann, linux-rdma, netdev, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, Grant Likely, Paul Mackerras, Sean Hefty, linuxppc-dev, Hal Rosenstock On 04/25/2013 12:35 PM, Benjamin Herrenschmidt wrote: > On Thu, 2013-04-25 at 10:23 -0500, Rob Herring wrote: >> Ben, Can I have your Ack for this? The change is straightforward and >> neither of the 2 drivers used the id parameter that is removed. > > Didn't you get my mail about a compile failure caused by this patch ? No, and I can't find any evidence of a mail in my inbox or in the list archives. > > A quick build test leads to: > > /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c: In function 'ibmebus_bus_device_probe': > /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:344:2: error: implicit declaration of function 'to_platform_driver' [-Werror=implicit-function-declaration] > /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:344:6: error: assignment makes pointer from integer without a cast [-Werror] > /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c: In function 'ibmebus_bus_device_remove': > /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:363:32: error: initialization makes pointer from integer without a cast [-Werror] > /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c: In function 'ibmebus_bus_device_shutdown': > /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:373:32: error: initialization makes pointer from integer without a cast [-Werror] > /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c: In function 'ibmebus_bus_legacy_suspend': > /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:420:32: error: initialization makes pointer from integer without a cast [-Werror] > /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c: In function 'ibmebus_bus_legacy_resume': > /home/benh/linux-powerpc-test/arch/powerpc/kernel/ibmebus.c:431:32: error: initialization makes pointer from integer without a cast [-Werror] > cc1: all warnings being treated as errors > make[2]: *** [arch/powerpc/kernel/ibmebus.o] Error 1 > make[2]: *** Waiting for unfinished jobs.... > > Haven't had a chance to look too closely today -EJETLAG :-) You need patch 2 of this series to fix this: driver core: move to_platform_driver to platform_device.h which as Arnd pointed out needs to come first. I've fixed that in my tree, but I don't think it warrants another post. Rob ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver 2013-04-25 19:14 ` Rob Herring @ 2013-04-25 20:12 ` Benjamin Herrenschmidt 2013-04-25 20:45 ` Rob Herring 0 siblings, 1 reply; 12+ messages in thread From: Benjamin Herrenschmidt @ 2013-04-25 20:12 UTC (permalink / raw) To: Rob Herring Cc: Roland Dreier, Arnd Bergmann, linux-rdma, netdev, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, Grant Likely, Paul Mackerras, Sean Hefty, linuxppc-dev, Hal Rosenstock On Thu, 2013-04-25 at 14:14 -0500, Rob Herring wrote: > On 04/25/2013 12:35 PM, Benjamin Herrenschmidt wrote: > > On Thu, 2013-04-25 at 10:23 -0500, Rob Herring wrote: > >> Ben, Can I have your Ack for this? The change is straightforward and > >> neither of the 2 drivers used the id parameter that is removed. > > > > Didn't you get my mail about a compile failure caused by this patch ? > > No, and I can't find any evidence of a mail in my inbox or in the list > archives. Odd... > > .../... > You need patch 2 of this series to fix this: > > driver core: move to_platform_driver to platform_device.h > > which as Arnd pointed out needs to come first. I've fixed that in my > tree, but I don't think it warrants another post. Can I pull you tree from somewhere to test ? Cheers, Ben. > Rob > -- > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver 2013-04-25 20:12 ` Benjamin Herrenschmidt @ 2013-04-25 20:45 ` Rob Herring 2013-05-22 12:26 ` Rob Herring 0 siblings, 1 reply; 12+ messages in thread From: Rob Herring @ 2013-04-25 20:45 UTC (permalink / raw) To: Benjamin Herrenschmidt Cc: Roland Dreier, Arnd Bergmann, linux-rdma, netdev, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, Grant Likely, Paul Mackerras, Sean Hefty, linuxppc-dev, Hal Rosenstock On 04/25/2013 03:12 PM, Benjamin Herrenschmidt wrote: > On Thu, 2013-04-25 at 14:14 -0500, Rob Herring wrote: >> On 04/25/2013 12:35 PM, Benjamin Herrenschmidt wrote: [...] >> You need patch 2 of this series to fix this: >> >> driver core: move to_platform_driver to platform_device.h >> >> which as Arnd pointed out needs to come first. I've fixed that in my >> tree, but I don't think it warrants another post. > > Can I pull you tree from somewhere to test ? Here you go: git://sources.calxeda.com/kernel/linux.git of-platform-removal Rob ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver 2013-04-25 20:45 ` Rob Herring @ 2013-05-22 12:26 ` Rob Herring 2013-05-22 12:32 ` Benjamin Herrenschmidt 2013-06-12 5:29 ` Benjamin Herrenschmidt 0 siblings, 2 replies; 12+ messages in thread From: Rob Herring @ 2013-05-22 12:26 UTC (permalink / raw) To: Benjamin Herrenschmidt Cc: Roland Dreier, Arnd Bergmann, linux-rdma, netdev, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, Grant Likely, Paul Mackerras, Sean Hefty, linuxppc-dev, Hal Rosenstock On Thu, Apr 25, 2013 at 3:45 PM, Rob Herring <robherring2@gmail.com> wrote: > On 04/25/2013 03:12 PM, Benjamin Herrenschmidt wrote: >> On Thu, 2013-04-25 at 14:14 -0500, Rob Herring wrote: >>> On 04/25/2013 12:35 PM, Benjamin Herrenschmidt wrote: > > [...] > >>> You need patch 2 of this series to fix this: >>> >>> driver core: move to_platform_driver to platform_device.h >>> >>> which as Arnd pointed out needs to come first. I've fixed that in my >>> tree, but I don't think it warrants another post. >> >> Can I pull you tree from somewhere to test ? > > Here you go: > > git://sources.calxeda.com/kernel/linux.git of-platform-removal Ben, Did you have a chance to test this? I want to get this into -next. Rob ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver 2013-05-22 12:26 ` Rob Herring @ 2013-05-22 12:32 ` Benjamin Herrenschmidt 2013-06-12 5:29 ` Benjamin Herrenschmidt 1 sibling, 0 replies; 12+ messages in thread From: Benjamin Herrenschmidt @ 2013-05-22 12:32 UTC (permalink / raw) To: Rob Herring Cc: Roland Dreier, Arnd Bergmann, linux-rdma, netdev, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, Grant Likely, Paul Mackerras, Sean Hefty, linuxppc-dev, Hal Rosenstock On Wed, 2013-05-22 at 07:26 -0500, Rob Herring wrote: > Did you have a chance to test this? I want to get this into -next. Ah sorry, skipped out of my mind, I'll get to it asap... Cheers, Ben. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver 2013-05-22 12:26 ` Rob Herring 2013-05-22 12:32 ` Benjamin Herrenschmidt @ 2013-06-12 5:29 ` Benjamin Herrenschmidt 2013-06-12 11:40 ` Grant Likely 1 sibling, 1 reply; 12+ messages in thread From: Benjamin Herrenschmidt @ 2013-06-12 5:29 UTC (permalink / raw) To: Rob Herring Cc: Roland Dreier, Arnd Bergmann, linux-rdma, netdev, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, Grant Likely, Paul Mackerras, Sean Hefty, linuxppc-dev, Hal Rosenstock On Wed, 2013-05-22 at 07:26 -0500, Rob Herring wrote: > > git://sources.calxeda.com/kernel/linux.git of-platform-removal > > Ben, > > Did you have a chance to test this? I want to get this into -next. I tested the one in for-next (sorry for the high latency). Works fine. Ack. Cheers, Ben. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver 2013-06-12 5:29 ` Benjamin Herrenschmidt @ 2013-06-12 11:40 ` Grant Likely 0 siblings, 0 replies; 12+ messages in thread From: Grant Likely @ 2013-06-12 11:40 UTC (permalink / raw) To: Benjamin Herrenschmidt, Rob Herring Cc: Roland Dreier, Arnd Bergmann, linux-rdma, netdev, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, Paul Mackerras, Sean Hefty, linuxppc-dev, Hal Rosenstock On Wed, 12 Jun 2013 15:29:38 +1000, Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote: > On Wed, 2013-05-22 at 07:26 -0500, Rob Herring wrote: > > > git://sources.calxeda.com/kernel/linux.git of-platform-removal > > > > Ben, > > > > Did you have a chance to test this? I want to get this into -next. > > I tested the one in for-next (sorry for the high latency). Works fine. > > Ack. Added acks and applied to my for-next branch. Thanks! g. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 0/5] of_platform_driver and OF_DEVICE removal 2013-04-22 2:13 [PATCH 0/5] of_platform_driver and OF_DEVICE removal Rob Herring 2013-04-22 2:13 ` [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver Rob Herring @ 2013-04-22 7:03 ` Arnd Bergmann 1 sibling, 0 replies; 12+ messages in thread From: Arnd Bergmann @ 2013-04-22 7:03 UTC (permalink / raw) To: Rob Herring Cc: Roland Dreier, Sean Hefty, Alexander Shishkin, devicetree-discuss, linux-usb, linux-kernel, Rob Herring, linux-rdma, Christoph Raisch, Hoang-Nam Nguyen, Thadeu Lima de Souza Cascardo, Greg Kroah-Hartman, Grant Likely, Paul Mackerras, netdev, linuxppc-dev, Hal Rosenstock On Monday 22 April 2013, Rob Herring wrote: > From: Rob Herring <rob.herring@calxeda.com> > > This series is a relatively straight-forward removal of the last remaining > user of of_platform_driver (ibmebus) and removal of CONFIG_OF_DEVICE which > is always enabled when CONFIG_OF is enabled. > > Compile tested on powerpc and sparc. > Acked-by: Arnd Bergmann <arnd@arndb.de> ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2013-06-12 11:40 UTC | newest] Thread overview: 12+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-04-22 2:13 [PATCH 0/5] of_platform_driver and OF_DEVICE removal Rob Herring 2013-04-22 2:13 ` [PATCH 1/5] ibmebus: convert of_platform_driver to platform_driver Rob Herring 2013-04-25 15:23 ` Rob Herring 2013-04-25 17:35 ` Benjamin Herrenschmidt 2013-04-25 19:14 ` Rob Herring 2013-04-25 20:12 ` Benjamin Herrenschmidt 2013-04-25 20:45 ` Rob Herring 2013-05-22 12:26 ` Rob Herring 2013-05-22 12:32 ` Benjamin Herrenschmidt 2013-06-12 5:29 ` Benjamin Herrenschmidt 2013-06-12 11:40 ` Grant Likely 2013-04-22 7:03 ` [PATCH 0/5] of_platform_driver and OF_DEVICE removal Arnd Bergmann
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).