From: Li Zefan <lizf@cn.fujitsu.com>
To: "Miguel Botón" <mboton@gmail.com>
Cc: LKML <linux-kernel@vger.kernel.org>,
netdev@vger.kernel.org, David Miller <davem@davemloft.net>,
zambrano@broadcom.com, linville@tuxdriver.com
Subject: Re: linux-2.6.24 compile error in drivers/net/b44.c
Date: Sun, 03 Feb 2008 09:53:10 +0800 [thread overview]
Message-ID: <47A51E86.8080305@cn.fujitsu.com> (raw)
In-Reply-To: <47A4CB7C.3010507@gmail.com>
Miguel Botón 写道:
> Li Zefan wrote:
>> Add CCs:
>>
>> CC: mboton@gmail.com
>> CC: zambrano@broadcom.com
>> CC: linville@tuxdriver.com
>>
>> Li Zefan wrote:
>>> drivers/net/b44.c: In function 'b44_remove_one':
>>> drivers/net/b44.c:2231: error: implicit declaration of function 'ssb_pcihost_set_power_state'
>>> make[2]: *** [drivers/net/b44.o] Error 1
>>> make[1]: *** [drivers/net] Error 2
>>>
>>> I think it is caused by:
>>>
>>> CONFIG_SSB_PCIHOST=n
>>> CONFIG_B44=y
>>>
>>
>
> This patch should fix b44 errors when 'SSB_PCIHOST' is not enabled.
>
I tried the patch, but it does't work, same compile error. :(
If .config is needed, tell me and I will attach it.
> Signed-off-by: Miguel Botón <mboton@gmail.com>
>
> diff --git a/drivers/net/b44.c b/drivers/net/b44.c
> index ea2a2b5..d081234 100644
> --- a/drivers/net/b44.c
> +++ b/drivers/net/b44.c
> @@ -2228,7 +2228,9 @@ static void __devexit b44_remove_one(struct ssb_device *sdev)
> unregister_netdev(dev);
> ssb_bus_may_powerdown(sdev->bus);
> free_netdev(dev);
> +#ifdef B44_PCI
> ssb_pcihost_set_power_state(sdev, PCI_D3hot);
> +#endif
> ssb_set_drvdata(sdev, NULL);
> }
>
> @@ -2257,7 +2259,9 @@ static int b44_suspend(struct ssb_device *sdev, pm_message_t state)
> b44_setup_wol(bp);
> }
>
> +#ifdef B44_PCI
> ssb_pcihost_set_power_state(sdev, PCI_D3hot);
> +#endif
> return 0;
> }
>
> --
> Miguel Botón
>
>
next prev parent reply other threads:[~2008-02-03 1:55 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-02 1:50 linux-2.6.24 compile error in drivers/net/b44.c Li Zefan
2008-02-02 4:51 ` Li Zefan
2008-02-02 19:58 ` Miguel Botón
2008-02-03 1:53 ` Li Zefan [this message]
2008-02-03 1:57 ` Li Zefan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=47A51E86.8080305@cn.fujitsu.com \
--to=lizf@cn.fujitsu.com \
--cc=davem@davemloft.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=mboton@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=zambrano@broadcom.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.