* [PATCH v2] PCI: kirin: Fix section mismatch warning @ 2018-09-19 18:53 Nathan Chancellor 2018-09-19 21:08 ` Nick Desaulniers 0 siblings, 1 reply; 6+ messages in thread From: Nathan Chancellor @ 2018-09-19 18:53 UTC (permalink / raw) To: Xiaowei Song, Binghui Wang, Lorenzo Pieralisi, Bjorn Helgaas Cc: linux-pci, linux-kernel, Nick Desaulniers, Nathan Chancellor WARNING: vmlinux.o(.text+0x4758cc): Section mismatch in reference from the function kirin_pcie_probe() to the function .init.text:kirin_add_pcie_port() The function kirin_pcie_probe() references the function __init kirin_add_pcie_port(). This is often because kirin_pcie_probe lacks a __init annotation or the annotation of kirin_add_pcie_port is wrong. Add the __init annotation to both kirin_pcie_probe and kirin_pcie_add_msi then use builtin_platform_driver_probe instead of builtin_platform_driver + .probe to avoid a section mismatch warning with kirin_pcie_driver. Fixes: fc5165db245a ("PCI: kirin: Add HiSilicon Kirin SoC PCIe controller driver") Reported-by: Nick Desaulniers <ndesaulniers@google.com> Suggested-by: Nick Desaulniers <ndesaulniers@google.com> Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> --- v1 -> v2: Instead of removing the annotation on kirin_add_pcie_port, add it to kirin_pcie_add_msi and kirin_pcie_probe. To avoid a warning with this configuration, use builtin_platform_driver_probe. drivers/pci/controller/dwc/pcie-kirin.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/pci/controller/dwc/pcie-kirin.c b/drivers/pci/controller/dwc/pcie-kirin.c index 5352e0c3be82..f64fed12de51 100644 --- a/drivers/pci/controller/dwc/pcie-kirin.c +++ b/drivers/pci/controller/dwc/pcie-kirin.c @@ -448,8 +448,8 @@ static const struct dw_pcie_host_ops kirin_pcie_host_ops = { .host_init = kirin_pcie_host_init, }; -static int kirin_pcie_add_msi(struct dw_pcie *pci, - struct platform_device *pdev) +static int __init kirin_pcie_add_msi(struct dw_pcie *pci, + struct platform_device *pdev) { int irq; @@ -481,7 +481,7 @@ static int __init kirin_add_pcie_port(struct dw_pcie *pci, return dw_pcie_host_init(&pci->pp); } -static int kirin_pcie_probe(struct platform_device *pdev) +static int __init kirin_pcie_probe(struct platform_device *pdev) { struct device *dev = &pdev->dev; struct kirin_pcie *kirin_pcie; @@ -533,11 +533,10 @@ static const struct of_device_id kirin_pcie_match[] = { }; static struct platform_driver kirin_pcie_driver = { - .probe = kirin_pcie_probe, .driver = { .name = "kirin-pcie", .of_match_table = kirin_pcie_match, .suppress_bind_attrs = true, }, }; -builtin_platform_driver(kirin_pcie_driver); +builtin_platform_driver_probe(kirin_pcie_driver, kirin_pcie_probe); -- 2.19.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] PCI: kirin: Fix section mismatch warning 2018-09-19 18:53 [PATCH v2] PCI: kirin: Fix section mismatch warning Nathan Chancellor @ 2018-09-19 21:08 ` Nick Desaulniers 2018-09-20 10:58 ` Lorenzo Pieralisi 0 siblings, 1 reply; 6+ messages in thread From: Nick Desaulniers @ 2018-09-19 21:08 UTC (permalink / raw) To: Nathan Chancellor Cc: songxiaowei, wangbinghui, lorenzo.pieralisi, bhelgaas, linux-pci, LKML, Greg KH On Wed, Sep 19, 2018 at 11:55 AM Nathan Chancellor <natechancellor@gmail.com> wrote: > > WARNING: vmlinux.o(.text+0x4758cc): Section mismatch in reference from > the function kirin_pcie_probe() to the function > .init.text:kirin_add_pcie_port() > The function kirin_pcie_probe() references > the function __init kirin_add_pcie_port(). > This is often because kirin_pcie_probe lacks a __init > annotation or the annotation of kirin_add_pcie_port is wrong. > > Add the __init annotation to both kirin_pcie_probe and > kirin_pcie_add_msi then use builtin_platform_driver_probe > instead of builtin_platform_driver + .probe to avoid a section > mismatch warning with kirin_pcie_driver. > > Fixes: fc5165db245a ("PCI: kirin: Add HiSilicon Kirin SoC PCIe controller driver") > Reported-by: Nick Desaulniers <ndesaulniers@google.com> > Suggested-by: Nick Desaulniers <ndesaulniers@google.com> > Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> > --- > > v1 -> v2: > > Instead of removing the annotation on kirin_add_pcie_port, add it to > kirin_pcie_add_msi and kirin_pcie_probe. To avoid a warning with this > configuration, use builtin_platform_driver_probe. > > drivers/pci/controller/dwc/pcie-kirin.c | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/drivers/pci/controller/dwc/pcie-kirin.c b/drivers/pci/controller/dwc/pcie-kirin.c > index 5352e0c3be82..f64fed12de51 100644 > --- a/drivers/pci/controller/dwc/pcie-kirin.c > +++ b/drivers/pci/controller/dwc/pcie-kirin.c > @@ -448,8 +448,8 @@ static const struct dw_pcie_host_ops kirin_pcie_host_ops = { > .host_init = kirin_pcie_host_init, > }; > > -static int kirin_pcie_add_msi(struct dw_pcie *pci, > - struct platform_device *pdev) > +static int __init kirin_pcie_add_msi(struct dw_pcie *pci, > + struct platform_device *pdev) > { > int irq; > > @@ -481,7 +481,7 @@ static int __init kirin_add_pcie_port(struct dw_pcie *pci, > return dw_pcie_host_init(&pci->pp); > } > > -static int kirin_pcie_probe(struct platform_device *pdev) > +static int __init kirin_pcie_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev; > struct kirin_pcie *kirin_pcie; > @@ -533,11 +533,10 @@ static const struct of_device_id kirin_pcie_match[] = { > }; > > static struct platform_driver kirin_pcie_driver = { > - .probe = kirin_pcie_probe, > .driver = { > .name = "kirin-pcie", > .of_match_table = kirin_pcie_match, > .suppress_bind_attrs = true, > }, > }; > -builtin_platform_driver(kirin_pcie_driver); > +builtin_platform_driver_probe(kirin_pcie_driver, kirin_pcie_probe); It would be good to get additional review from someone who knows more about driver callback lifecycles, but I think this is the correct fix. Thanks Nathan. Reviewed-by: Nick Desaulniers <ndesaulniers@google.com> -- Thanks, ~Nick Desaulniers ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] PCI: kirin: Fix section mismatch warning 2018-09-19 21:08 ` Nick Desaulniers @ 2018-09-20 10:58 ` Lorenzo Pieralisi 2018-10-01 14:53 ` Lorenzo Pieralisi 0 siblings, 1 reply; 6+ messages in thread From: Lorenzo Pieralisi @ 2018-09-20 10:58 UTC (permalink / raw) To: Nick Desaulniers Cc: Nathan Chancellor, songxiaowei, wangbinghui, bhelgaas, linux-pci, LKML, Greg KH, paul.gortmaker [+Paul] On Wed, Sep 19, 2018 at 02:08:08PM -0700, Nick Desaulniers wrote: > On Wed, Sep 19, 2018 at 11:55 AM Nathan Chancellor > <natechancellor@gmail.com> wrote: > > > > WARNING: vmlinux.o(.text+0x4758cc): Section mismatch in reference from > > the function kirin_pcie_probe() to the function > > .init.text:kirin_add_pcie_port() > > The function kirin_pcie_probe() references > > the function __init kirin_add_pcie_port(). > > This is often because kirin_pcie_probe lacks a __init > > annotation or the annotation of kirin_add_pcie_port is wrong. > > > > Add the __init annotation to both kirin_pcie_probe and > > kirin_pcie_add_msi then use builtin_platform_driver_probe > > instead of builtin_platform_driver + .probe to avoid a section > > mismatch warning with kirin_pcie_driver. > > > > Fixes: fc5165db245a ("PCI: kirin: Add HiSilicon Kirin SoC PCIe controller driver") > > Reported-by: Nick Desaulniers <ndesaulniers@google.com> > > Suggested-by: Nick Desaulniers <ndesaulniers@google.com> > > Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> > > --- > > > > v1 -> v2: > > > > Instead of removing the annotation on kirin_add_pcie_port, add it to > > kirin_pcie_add_msi and kirin_pcie_probe. To avoid a warning with this > > configuration, use builtin_platform_driver_probe. > > > > drivers/pci/controller/dwc/pcie-kirin.c | 9 ++++----- > > 1 file changed, 4 insertions(+), 5 deletions(-) > > > > diff --git a/drivers/pci/controller/dwc/pcie-kirin.c b/drivers/pci/controller/dwc/pcie-kirin.c > > index 5352e0c3be82..f64fed12de51 100644 > > --- a/drivers/pci/controller/dwc/pcie-kirin.c > > +++ b/drivers/pci/controller/dwc/pcie-kirin.c > > @@ -448,8 +448,8 @@ static const struct dw_pcie_host_ops kirin_pcie_host_ops = { > > .host_init = kirin_pcie_host_init, > > }; > > > > -static int kirin_pcie_add_msi(struct dw_pcie *pci, > > - struct platform_device *pdev) > > +static int __init kirin_pcie_add_msi(struct dw_pcie *pci, > > + struct platform_device *pdev) > > { > > int irq; > > > > @@ -481,7 +481,7 @@ static int __init kirin_add_pcie_port(struct dw_pcie *pci, > > return dw_pcie_host_init(&pci->pp); > > } > > > > -static int kirin_pcie_probe(struct platform_device *pdev) > > +static int __init kirin_pcie_probe(struct platform_device *pdev) > > { > > struct device *dev = &pdev->dev; > > struct kirin_pcie *kirin_pcie; > > @@ -533,11 +533,10 @@ static const struct of_device_id kirin_pcie_match[] = { > > }; > > > > static struct platform_driver kirin_pcie_driver = { > > - .probe = kirin_pcie_probe, > > .driver = { > > .name = "kirin-pcie", > > .of_match_table = kirin_pcie_match, > > .suppress_bind_attrs = true, > > }, > > }; > > -builtin_platform_driver(kirin_pcie_driver); > > +builtin_platform_driver_probe(kirin_pcie_driver, kirin_pcie_probe); > > It would be good to get additional review from someone who knows more > about driver callback lifecycles, but I think this is the correct fix. > Thanks Nathan. > Reviewed-by: Nick Desaulniers <ndesaulniers@google.com> AFAICS we can't use builtin_platform_driver_probe() if the probe() itself may need deferral (and I *reckon* it may need that, see eg clk_get()) so I suspect we have to go back to v1 for the patch, apologies if so. Lorenzo ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] PCI: kirin: Fix section mismatch warning 2018-09-20 10:58 ` Lorenzo Pieralisi @ 2018-10-01 14:53 ` Lorenzo Pieralisi 2018-10-01 15:12 ` Nathan Chancellor 0 siblings, 1 reply; 6+ messages in thread From: Lorenzo Pieralisi @ 2018-10-01 14:53 UTC (permalink / raw) To: Nick Desaulniers Cc: Nathan Chancellor, songxiaowei, wangbinghui, bhelgaas, linux-pci, LKML, Greg KH, paul.gortmaker On Thu, Sep 20, 2018 at 11:58:28AM +0100, Lorenzo Pieralisi wrote: > [+Paul] > > On Wed, Sep 19, 2018 at 02:08:08PM -0700, Nick Desaulniers wrote: > > On Wed, Sep 19, 2018 at 11:55 AM Nathan Chancellor > > <natechancellor@gmail.com> wrote: > > > > > > WARNING: vmlinux.o(.text+0x4758cc): Section mismatch in reference from > > > the function kirin_pcie_probe() to the function > > > .init.text:kirin_add_pcie_port() > > > The function kirin_pcie_probe() references > > > the function __init kirin_add_pcie_port(). > > > This is often because kirin_pcie_probe lacks a __init > > > annotation or the annotation of kirin_add_pcie_port is wrong. > > > > > > Add the __init annotation to both kirin_pcie_probe and > > > kirin_pcie_add_msi then use builtin_platform_driver_probe > > > instead of builtin_platform_driver + .probe to avoid a section > > > mismatch warning with kirin_pcie_driver. > > > > > > Fixes: fc5165db245a ("PCI: kirin: Add HiSilicon Kirin SoC PCIe controller driver") > > > Reported-by: Nick Desaulniers <ndesaulniers@google.com> > > > Suggested-by: Nick Desaulniers <ndesaulniers@google.com> > > > Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> > > > --- > > > > > > v1 -> v2: > > > > > > Instead of removing the annotation on kirin_add_pcie_port, add it to > > > kirin_pcie_add_msi and kirin_pcie_probe. To avoid a warning with this > > > configuration, use builtin_platform_driver_probe. > > > > > > drivers/pci/controller/dwc/pcie-kirin.c | 9 ++++----- > > > 1 file changed, 4 insertions(+), 5 deletions(-) > > > > > > diff --git a/drivers/pci/controller/dwc/pcie-kirin.c b/drivers/pci/controller/dwc/pcie-kirin.c > > > index 5352e0c3be82..f64fed12de51 100644 > > > --- a/drivers/pci/controller/dwc/pcie-kirin.c > > > +++ b/drivers/pci/controller/dwc/pcie-kirin.c > > > @@ -448,8 +448,8 @@ static const struct dw_pcie_host_ops kirin_pcie_host_ops = { > > > .host_init = kirin_pcie_host_init, > > > }; > > > > > > -static int kirin_pcie_add_msi(struct dw_pcie *pci, > > > - struct platform_device *pdev) > > > +static int __init kirin_pcie_add_msi(struct dw_pcie *pci, > > > + struct platform_device *pdev) > > > { > > > int irq; > > > > > > @@ -481,7 +481,7 @@ static int __init kirin_add_pcie_port(struct dw_pcie *pci, > > > return dw_pcie_host_init(&pci->pp); > > > } > > > > > > -static int kirin_pcie_probe(struct platform_device *pdev) > > > +static int __init kirin_pcie_probe(struct platform_device *pdev) > > > { > > > struct device *dev = &pdev->dev; > > > struct kirin_pcie *kirin_pcie; > > > @@ -533,11 +533,10 @@ static const struct of_device_id kirin_pcie_match[] = { > > > }; > > > > > > static struct platform_driver kirin_pcie_driver = { > > > - .probe = kirin_pcie_probe, > > > .driver = { > > > .name = "kirin-pcie", > > > .of_match_table = kirin_pcie_match, > > > .suppress_bind_attrs = true, > > > }, > > > }; > > > -builtin_platform_driver(kirin_pcie_driver); > > > +builtin_platform_driver_probe(kirin_pcie_driver, kirin_pcie_probe); > > > > It would be good to get additional review from someone who knows more > > about driver callback lifecycles, but I think this is the correct fix. > > Thanks Nathan. > > Reviewed-by: Nick Desaulniers <ndesaulniers@google.com> > > AFAICS we can't use builtin_platform_driver_probe() if the probe() > itself may need deferral (and I *reckon* it may need that, see eg > clk_get()) so I suspect we have to go back to v1 for the patch, > apologies if so. I think that the best course of action consists in merging v1 since we have not reached a definitive conclusion on v2, please let me know. Thanks, Lorenzo ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] PCI: kirin: Fix section mismatch warning 2018-10-01 14:53 ` Lorenzo Pieralisi @ 2018-10-01 15:12 ` Nathan Chancellor 2018-10-01 16:44 ` Lorenzo Pieralisi 0 siblings, 1 reply; 6+ messages in thread From: Nathan Chancellor @ 2018-10-01 15:12 UTC (permalink / raw) To: Lorenzo Pieralisi Cc: Nick Desaulniers, songxiaowei, wangbinghui, bhelgaas, linux-pci, LKML, Greg KH, paul.gortmaker On Mon, Oct 01, 2018 at 03:53:41PM +0100, Lorenzo Pieralisi wrote: > On Thu, Sep 20, 2018 at 11:58:28AM +0100, Lorenzo Pieralisi wrote: > > [+Paul] > > > > On Wed, Sep 19, 2018 at 02:08:08PM -0700, Nick Desaulniers wrote: > > > On Wed, Sep 19, 2018 at 11:55 AM Nathan Chancellor > > > <natechancellor@gmail.com> wrote: > > > > > > > > WARNING: vmlinux.o(.text+0x4758cc): Section mismatch in reference from > > > > the function kirin_pcie_probe() to the function > > > > .init.text:kirin_add_pcie_port() > > > > The function kirin_pcie_probe() references > > > > the function __init kirin_add_pcie_port(). > > > > This is often because kirin_pcie_probe lacks a __init > > > > annotation or the annotation of kirin_add_pcie_port is wrong. > > > > > > > > Add the __init annotation to both kirin_pcie_probe and > > > > kirin_pcie_add_msi then use builtin_platform_driver_probe > > > > instead of builtin_platform_driver + .probe to avoid a section > > > > mismatch warning with kirin_pcie_driver. > > > > > > > > Fixes: fc5165db245a ("PCI: kirin: Add HiSilicon Kirin SoC PCIe controller driver") > > > > Reported-by: Nick Desaulniers <ndesaulniers@google.com> > > > > Suggested-by: Nick Desaulniers <ndesaulniers@google.com> > > > > Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> > > > > --- > > > > > > > > v1 -> v2: > > > > > > > > Instead of removing the annotation on kirin_add_pcie_port, add it to > > > > kirin_pcie_add_msi and kirin_pcie_probe. To avoid a warning with this > > > > configuration, use builtin_platform_driver_probe. > > > > > > > > drivers/pci/controller/dwc/pcie-kirin.c | 9 ++++----- > > > > 1 file changed, 4 insertions(+), 5 deletions(-) > > > > > > > > diff --git a/drivers/pci/controller/dwc/pcie-kirin.c b/drivers/pci/controller/dwc/pcie-kirin.c > > > > index 5352e0c3be82..f64fed12de51 100644 > > > > --- a/drivers/pci/controller/dwc/pcie-kirin.c > > > > +++ b/drivers/pci/controller/dwc/pcie-kirin.c > > > > @@ -448,8 +448,8 @@ static const struct dw_pcie_host_ops kirin_pcie_host_ops = { > > > > .host_init = kirin_pcie_host_init, > > > > }; > > > > > > > > -static int kirin_pcie_add_msi(struct dw_pcie *pci, > > > > - struct platform_device *pdev) > > > > +static int __init kirin_pcie_add_msi(struct dw_pcie *pci, > > > > + struct platform_device *pdev) > > > > { > > > > int irq; > > > > > > > > @@ -481,7 +481,7 @@ static int __init kirin_add_pcie_port(struct dw_pcie *pci, > > > > return dw_pcie_host_init(&pci->pp); > > > > } > > > > > > > > -static int kirin_pcie_probe(struct platform_device *pdev) > > > > +static int __init kirin_pcie_probe(struct platform_device *pdev) > > > > { > > > > struct device *dev = &pdev->dev; > > > > struct kirin_pcie *kirin_pcie; > > > > @@ -533,11 +533,10 @@ static const struct of_device_id kirin_pcie_match[] = { > > > > }; > > > > > > > > static struct platform_driver kirin_pcie_driver = { > > > > - .probe = kirin_pcie_probe, > > > > .driver = { > > > > .name = "kirin-pcie", > > > > .of_match_table = kirin_pcie_match, > > > > .suppress_bind_attrs = true, > > > > }, > > > > }; > > > > -builtin_platform_driver(kirin_pcie_driver); > > > > +builtin_platform_driver_probe(kirin_pcie_driver, kirin_pcie_probe); > > > > > > It would be good to get additional review from someone who knows more > > > about driver callback lifecycles, but I think this is the correct fix. > > > Thanks Nathan. > > > Reviewed-by: Nick Desaulniers <ndesaulniers@google.com> > > > > AFAICS we can't use builtin_platform_driver_probe() if the probe() > > itself may need deferral (and I *reckon* it may need that, see eg > > clk_get()) so I suspect we have to go back to v1 for the patch, > > apologies if so. > > I think that the best course of action consists in merging v1 since > we have not reached a definitive conclusion on v2, please let me > know. > > Thanks, > Lorenzo Hi Lorenzo, I am perfectly fine with v1 being merged as either patch solves the warning. I agree with Nick that if this version works it would be more proper but it also carries more risk with regards to deferral like you stated. Please let me know if I need to resend it. Thanks, Nathan ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] PCI: kirin: Fix section mismatch warning 2018-10-01 15:12 ` Nathan Chancellor @ 2018-10-01 16:44 ` Lorenzo Pieralisi 0 siblings, 0 replies; 6+ messages in thread From: Lorenzo Pieralisi @ 2018-10-01 16:44 UTC (permalink / raw) To: Nathan Chancellor Cc: Nick Desaulniers, songxiaowei, wangbinghui, bhelgaas, linux-pci, LKML, Greg KH, paul.gortmaker On Mon, Oct 01, 2018 at 08:12:05AM -0700, Nathan Chancellor wrote: > On Mon, Oct 01, 2018 at 03:53:41PM +0100, Lorenzo Pieralisi wrote: > > On Thu, Sep 20, 2018 at 11:58:28AM +0100, Lorenzo Pieralisi wrote: > > > [+Paul] > > > > > > On Wed, Sep 19, 2018 at 02:08:08PM -0700, Nick Desaulniers wrote: > > > > On Wed, Sep 19, 2018 at 11:55 AM Nathan Chancellor > > > > <natechancellor@gmail.com> wrote: > > > > > > > > > > WARNING: vmlinux.o(.text+0x4758cc): Section mismatch in reference from > > > > > the function kirin_pcie_probe() to the function > > > > > .init.text:kirin_add_pcie_port() > > > > > The function kirin_pcie_probe() references > > > > > the function __init kirin_add_pcie_port(). > > > > > This is often because kirin_pcie_probe lacks a __init > > > > > annotation or the annotation of kirin_add_pcie_port is wrong. > > > > > > > > > > Add the __init annotation to both kirin_pcie_probe and > > > > > kirin_pcie_add_msi then use builtin_platform_driver_probe > > > > > instead of builtin_platform_driver + .probe to avoid a section > > > > > mismatch warning with kirin_pcie_driver. > > > > > > > > > > Fixes: fc5165db245a ("PCI: kirin: Add HiSilicon Kirin SoC PCIe controller driver") > > > > > Reported-by: Nick Desaulniers <ndesaulniers@google.com> > > > > > Suggested-by: Nick Desaulniers <ndesaulniers@google.com> > > > > > Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> > > > > > --- > > > > > > > > > > v1 -> v2: > > > > > > > > > > Instead of removing the annotation on kirin_add_pcie_port, add it to > > > > > kirin_pcie_add_msi and kirin_pcie_probe. To avoid a warning with this > > > > > configuration, use builtin_platform_driver_probe. > > > > > > > > > > drivers/pci/controller/dwc/pcie-kirin.c | 9 ++++----- > > > > > 1 file changed, 4 insertions(+), 5 deletions(-) > > > > > > > > > > diff --git a/drivers/pci/controller/dwc/pcie-kirin.c b/drivers/pci/controller/dwc/pcie-kirin.c > > > > > index 5352e0c3be82..f64fed12de51 100644 > > > > > --- a/drivers/pci/controller/dwc/pcie-kirin.c > > > > > +++ b/drivers/pci/controller/dwc/pcie-kirin.c > > > > > @@ -448,8 +448,8 @@ static const struct dw_pcie_host_ops kirin_pcie_host_ops = { > > > > > .host_init = kirin_pcie_host_init, > > > > > }; > > > > > > > > > > -static int kirin_pcie_add_msi(struct dw_pcie *pci, > > > > > - struct platform_device *pdev) > > > > > +static int __init kirin_pcie_add_msi(struct dw_pcie *pci, > > > > > + struct platform_device *pdev) > > > > > { > > > > > int irq; > > > > > > > > > > @@ -481,7 +481,7 @@ static int __init kirin_add_pcie_port(struct dw_pcie *pci, > > > > > return dw_pcie_host_init(&pci->pp); > > > > > } > > > > > > > > > > -static int kirin_pcie_probe(struct platform_device *pdev) > > > > > +static int __init kirin_pcie_probe(struct platform_device *pdev) > > > > > { > > > > > struct device *dev = &pdev->dev; > > > > > struct kirin_pcie *kirin_pcie; > > > > > @@ -533,11 +533,10 @@ static const struct of_device_id kirin_pcie_match[] = { > > > > > }; > > > > > > > > > > static struct platform_driver kirin_pcie_driver = { > > > > > - .probe = kirin_pcie_probe, > > > > > .driver = { > > > > > .name = "kirin-pcie", > > > > > .of_match_table = kirin_pcie_match, > > > > > .suppress_bind_attrs = true, > > > > > }, > > > > > }; > > > > > -builtin_platform_driver(kirin_pcie_driver); > > > > > +builtin_platform_driver_probe(kirin_pcie_driver, kirin_pcie_probe); > > > > > > > > It would be good to get additional review from someone who knows more > > > > about driver callback lifecycles, but I think this is the correct fix. > > > > Thanks Nathan. > > > > Reviewed-by: Nick Desaulniers <ndesaulniers@google.com> > > > > > > AFAICS we can't use builtin_platform_driver_probe() if the probe() > > > itself may need deferral (and I *reckon* it may need that, see eg > > > clk_get()) so I suspect we have to go back to v1 for the patch, > > > apologies if so. > > > > I think that the best course of action consists in merging v1 since > > we have not reached a definitive conclusion on v2, please let me > > know. > > > > Thanks, > > Lorenzo > > Hi Lorenzo, > > I am perfectly fine with v1 being merged as either patch solves the > warning. I agree with Nick that if this version works it would be more > proper but it also carries more risk with regards to deferral like you > stated. Please let me know if I need to resend it. I have applied v1 to my pci/dwc for v4.20 and dropped this patch, thanks for bearing with me and if there is any objection please let me know. Thanks, Lorenzo ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-10-01 16:44 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-09-19 18:53 [PATCH v2] PCI: kirin: Fix section mismatch warning Nathan Chancellor 2018-09-19 21:08 ` Nick Desaulniers 2018-09-20 10:58 ` Lorenzo Pieralisi 2018-10-01 14:53 ` Lorenzo Pieralisi 2018-10-01 15:12 ` Nathan Chancellor 2018-10-01 16:44 ` Lorenzo Pieralisi
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.