diff mbox

[next] bcma: add missing includes

Message ID 1425364377-12771-1-git-send-email-zajec5@gmail.com (mailing list archive)
State Accepted
Delegated to: Kalle Valo
Headers show

Commit Message

Rafał Miłecki March 3, 2015, 6:32 a.m. UTC
Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
---
 drivers/bcma/driver_pci_host.c | 1 +
 drivers/bcma/driver_pcie2.c    | 1 +
 2 files changed, 2 insertions(+)

Comments

Kalle Valo March 3, 2015, 12:14 p.m. UTC | #1
Rafa? Mi?ecki <zajec5@gmail.com> writes:

> Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>

Please, no empty commit logs. I don't (yet) require that with all
patches, but this definitely needs one. Is this ok:

bcma: add missing includes

kbuild found out that commit 804e27dee49e ("bcma: support bringing up
bus hosted on PCIe Gen 2") broke the build on m68k:

   drivers/bcma/driver_pcie2.c: In function 'bcma_core_pcie2_up':
>> drivers/bcma/driver_pcie2.c:196:2: error: implicit declaration of function 'pcie_set_readrq' [-Werror\
=implicit-function-declaration]
     err = pcie_set_readrq(dev, pcie2->reqsize);
     ^
   cc1: some warnings being treated as errors

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
Rafał Miłecki March 3, 2015, 1:18 p.m. UTC | #2
On 3 March 2015 at 13:14, Kalle Valo <kvalo@codeaurora.org> wrote:
> Rafa? Mi?ecki <zajec5@gmail.com> writes:
>
>> Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
>
> Please, no empty commit logs. I don't (yet) require that with all
> patches, but this definitely needs one. Is this ok:
>
> bcma: add missing includes
>
> kbuild found out that commit 804e27dee49e ("bcma: support bringing up
> bus hosted on PCIe Gen 2") broke the build on m68k:
>
>    drivers/bcma/driver_pcie2.c: In function 'bcma_core_pcie2_up':
>>> drivers/bcma/driver_pcie2.c:196:2: error: implicit declaration of function 'pcie_set_readrq' [-Werror\
> =implicit-function-declaration]
>      err = pcie_set_readrq(dev, pcie2->reqsize);
>      ^
>    cc1: some warnings being treated as errors
>
> Reported-by: kbuild test robot <fengguang.wu@intel.com>
> Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>

You're right, I could to better :| Yes, your version looks OK.
Kalle Valo March 3, 2015, 1:28 p.m. UTC | #3
Rafa? Mi?ecki <zajec5@gmail.com> writes:

> On 3 March 2015 at 13:14, Kalle Valo <kvalo@codeaurora.org> wrote:
>> Rafa? Mi?ecki <zajec5@gmail.com> writes:
>>
>>> Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
>>
>> Please, no empty commit logs. I don't (yet) require that with all
>> patches, but this definitely needs one. Is this ok:
>>
>> bcma: add missing includes
>>
>> kbuild found out that commit 804e27dee49e ("bcma: support bringing up
>> bus hosted on PCIe Gen 2") broke the build on m68k:
>>
>>    drivers/bcma/driver_pcie2.c: In function 'bcma_core_pcie2_up':
>>>> drivers/bcma/driver_pcie2.c:196:2: error: implicit declaration of function 'pcie_set_readrq' [-Werror\
>> =implicit-function-declaration]
>>      err = pcie_set_readrq(dev, pcie2->reqsize);
>>      ^
>>    cc1: some warnings being treated as errors
>>
>> Reported-by: kbuild test robot <fengguang.wu@intel.com>
>> Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
>
> You're right, I could to better :| Yes, your version looks OK.

Thanks, I applied with my modifications.
diff mbox

Patch

diff --git a/drivers/bcma/driver_pci_host.c b/drivers/bcma/driver_pci_host.c
index c8a6b74..c42cec7 100644
--- a/drivers/bcma/driver_pci_host.c
+++ b/drivers/bcma/driver_pci_host.c
@@ -11,6 +11,7 @@ 
 
 #include "bcma_private.h"
 #include <linux/pci.h>
+#include <linux/slab.h>
 #include <linux/export.h>
 #include <linux/bcma/bcma.h>
 #include <asm/paccess.h>
diff --git a/drivers/bcma/driver_pcie2.c b/drivers/bcma/driver_pcie2.c
index 4568bc7..b1a6e32 100644
--- a/drivers/bcma/driver_pcie2.c
+++ b/drivers/bcma/driver_pcie2.c
@@ -10,6 +10,7 @@ 
 
 #include "bcma_private.h"
 #include <linux/bcma/bcma.h>
+#include <linux/pci.h>
 
 /**************************************************
  * R/W ops.