* linux-next: build failure after merge of the libata tree
@ 2010-08-18 0:40 Stephen Rothwell
2010-08-18 0:46 ` Gwendal Grignou
0 siblings, 1 reply; 22+ messages in thread
From: Stephen Rothwell @ 2010-08-18 0:40 UTC (permalink / raw)
To: Jeff Garzik; +Cc: linux-next, linux-kernel, Gwendal Grignou, Grant Grundler
[-- Attachment #1: Type: text/plain, Size: 877 bytes --]
Hi Jeff,
After merging the libata tree, today's linux-next build (powerpc
ppc64_defconfig) failed like this:
drivers/ata/libata-transport.c: In function 'ata_attach_transport':
drivers/ata/libata-transport.c:678: error: implicit declaration of function 'kzalloc'
drivers/ata/libata-transport.c:678: warning: assignment makes pointer from integer without a cast
drivers/ata/libata-transport.c: In function 'ata_release_transport':
drivers/ata/libata-transport.c:741: error: implicit declaration of function 'kfree'
Caused by commit 97395d39931160a171959ac2e1fbff1caba3374b ("[libata] Add
ATA transport class") neglecting to include linux/slab.h. See Rule 1 in
Documentation/SubmitChecklist.
I have used the libata tree form next-20100817 for today.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 490 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2010-08-18 0:40 linux-next: build failure after merge of the libata tree Stephen Rothwell
@ 2010-08-18 0:46 ` Gwendal Grignou
2010-08-19 20:15 ` Jeff Garzik
0 siblings, 1 reply; 22+ messages in thread
From: Gwendal Grignou @ 2010-08-18 0:46 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: Jeff Garzik, linux-next, linux-kernel, Grant Grundler
Let me fix it and send a patch to Jeff.
Gwendal.
On Tue, Aug 17, 2010 at 5:40 PM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> Hi Jeff,
>
> After merging the libata tree, today's linux-next build (powerpc
> ppc64_defconfig) failed like this:
>
> drivers/ata/libata-transport.c: In function 'ata_attach_transport':
> drivers/ata/libata-transport.c:678: error: implicit declaration of function 'kzalloc'
> drivers/ata/libata-transport.c:678: warning: assignment makes pointer from integer without a cast
> drivers/ata/libata-transport.c: In function 'ata_release_transport':
> drivers/ata/libata-transport.c:741: error: implicit declaration of function 'kfree'
>
> Caused by commit 97395d39931160a171959ac2e1fbff1caba3374b ("[libata] Add
> ATA transport class") neglecting to include linux/slab.h. See Rule 1 in
> Documentation/SubmitChecklist.
>
> I have used the libata tree form next-20100817 for today.
> --
> Cheers,
> Stephen Rothwell sfr@canb.auug.org.au
> http://www.canb.auug.org.au/~sfr/
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2010-08-18 0:46 ` Gwendal Grignou
@ 2010-08-19 20:15 ` Jeff Garzik
0 siblings, 0 replies; 22+ messages in thread
From: Jeff Garzik @ 2010-08-19 20:15 UTC (permalink / raw)
To: Gwendal Grignou, Stephen Rothwell
Cc: linux-next, linux-kernel, Grant Grundler
On 08/17/2010 08:46 PM, Gwendal Grignou wrote:
> Let me fix it and send a patch to Jeff.
>
> Gwendal.
>
> On Tue, Aug 17, 2010 at 5:40 PM, Stephen Rothwell<sfr@canb.auug.org.au> wrote:
>> Hi Jeff,
>>
>> After merging the libata tree, today's linux-next build (powerpc
>> ppc64_defconfig) failed like this:
>>
>> drivers/ata/libata-transport.c: In function 'ata_attach_transport':
>> drivers/ata/libata-transport.c:678: error: implicit declaration of function 'kzalloc'
>> drivers/ata/libata-transport.c:678: warning: assignment makes pointer from integer without a cast
>> drivers/ata/libata-transport.c: In function 'ata_release_transport':
>> drivers/ata/libata-transport.c:741: error: implicit declaration of function 'kfree'
>>
>> Caused by commit 97395d39931160a171959ac2e1fbff1caba3374b ("[libata] Add
>> ATA transport class") neglecting to include linux/slab.h. See Rule 1 in
>> Documentation/SubmitChecklist.
>>
>> I have used the libata tree form next-20100817 for today.
Never received a patch; pushed out the obvious fix myself.
^ permalink raw reply [flat|nested] 22+ messages in thread
* linux-next: build failure after merge of the libata tree
@ 2012-07-02 2:10 Stephen Rothwell
2012-07-02 2:19 ` huang ying
0 siblings, 1 reply; 22+ messages in thread
From: Stephen Rothwell @ 2012-07-02 2:10 UTC (permalink / raw)
To: Jeff Garzik; +Cc: linux-next, linux-kernel, Lin Ming, Huang Ying, Bjorn Helgaas
[-- Attachment #1: Type: text/plain, Size: 1640 bytes --]
Hi Jeff,
After merging the libata tree, today's linux-next build (x86_64
allmodconfig) failed like this:
drivers/ata/libata-acpi.c: In function 'ata_acpi_set_state':
drivers/ata/libata-acpi.c:872:5: error: too few arguments to function 'acpi_pm_device_sleep_state'
include/acpi/acpi_bus.h:418:5: note: declared here
Caused by commit 3bd46600a7a7 ("libata-acpi: add ata port runtime D3Cold
support") from the libata tree interacting with commit ee85f543710d
("ACPI/PM: specify lowest allowed state for device sleep state") from the
pci tree.
I added the following merge fix patch (and can carry it as necessary):
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Mon, 2 Jul 2012 12:07:25 +1000
Subject: [PATCH] libata-acpi: fix up for acpi_pm_device_sleep_state API
change
Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
---
drivers/ata/libata-acpi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/ata/libata-acpi.c b/drivers/ata/libata-acpi.c
index 4112eaa..a2b16c9 100644
--- a/drivers/ata/libata-acpi.c
+++ b/drivers/ata/libata-acpi.c
@@ -869,7 +869,7 @@ void ata_acpi_set_state(struct ata_port *ap, pm_message_t state)
if (state.event != PM_EVENT_ON) {
acpi_state = acpi_pm_device_sleep_state(
- &dev->sdev->sdev_gendev, NULL);
+ &dev->sdev->sdev_gendev, NULL, ACPI_STATE_D3);
if (acpi_state > 0)
acpi_bus_set_power(handle, acpi_state);
/* TBD: need to check if it's runtime pm request */
--
1.7.10.280.gaa39
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2012-07-02 2:10 Stephen Rothwell
@ 2012-07-02 2:19 ` huang ying
0 siblings, 0 replies; 22+ messages in thread
From: huang ying @ 2012-07-02 2:19 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Jeff Garzik, linux-next, linux-kernel, Lin Ming, Huang Ying,
Bjorn Helgaas
Hi, Stephen,
On Mon, Jul 2, 2012 at 10:10 AM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> Hi Jeff,
>
> After merging the libata tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> drivers/ata/libata-acpi.c: In function 'ata_acpi_set_state':
> drivers/ata/libata-acpi.c:872:5: error: too few arguments to function 'acpi_pm_device_sleep_state'
> include/acpi/acpi_bus.h:418:5: note: declared here
>
> Caused by commit 3bd46600a7a7 ("libata-acpi: add ata port runtime D3Cold
> support") from the libata tree interacting with commit ee85f543710d
> ("ACPI/PM: specify lowest allowed state for device sleep state") from the
> pci tree.
>
> I added the following merge fix patch (and can carry it as necessary):
>
> From: Stephen Rothwell <sfr@canb.auug.org.au>
> Date: Mon, 2 Jul 2012 12:07:25 +1000
> Subject: [PATCH] libata-acpi: fix up for acpi_pm_device_sleep_state API
> change
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
> ---
> drivers/ata/libata-acpi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/ata/libata-acpi.c b/drivers/ata/libata-acpi.c
> index 4112eaa..a2b16c9 100644
> --- a/drivers/ata/libata-acpi.c
> +++ b/drivers/ata/libata-acpi.c
> @@ -869,7 +869,7 @@ void ata_acpi_set_state(struct ata_port *ap, pm_message_t state)
>
> if (state.event != PM_EVENT_ON) {
> acpi_state = acpi_pm_device_sleep_state(
> - &dev->sdev->sdev_gendev, NULL);
> + &dev->sdev->sdev_gendev, NULL, ACPI_STATE_D3);
> if (acpi_state > 0)
> acpi_bus_set_power(handle, acpi_state);
> /* TBD: need to check if it's runtime pm request */
The fix is good for me. Thanks!
Best Regards,
Huang Ying
^ permalink raw reply [flat|nested] 22+ messages in thread
* linux-next: build failure after merge of the libata tree
@ 2013-08-19 1:54 Stephen Rothwell
2013-08-19 3:18 ` Terry Suereth
0 siblings, 1 reply; 22+ messages in thread
From: Stephen Rothwell @ 2013-08-19 1:54 UTC (permalink / raw)
To: Tejun Heo; +Cc: linux-next, linux-kernel, Terry Suereth, stable
[-- Attachment #1: Type: text/plain, Size: 581 bytes --]
Hi Tejun,
After merging the libata tree, today's linux-next build (powerpc
ppc64_defconfig) failed like this:
drivers/ata/libata-pmp.c: In function 'sata_pmp_quirks':
drivers/ata/libata-pmp.c:386:36: error: invalid suffix "x26" on integer constant
if (vendor == 0x1095 && (devid == 0x3x26 || devid == 0x3826)) {
^
Caused by commit f1a313ad86b9 ("libata: apply behavioral quirks to sil3826 PMP").
I have used the libata tree from next-20130816 for today.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2013-08-19 1:54 Stephen Rothwell
@ 2013-08-19 3:18 ` Terry Suereth
2013-08-19 13:40 ` Tejun Heo
0 siblings, 1 reply; 22+ messages in thread
From: Terry Suereth @ 2013-08-19 3:18 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: Tejun Heo, linux-next, linux-kernel, stable
Per my original patch, "0x3x26" in code should be "0x3726" (as in the
unpatched version). I refer to "x26" in comments as a forward-looking
assumption, but AFAIK only 3726 and 3826 exist at this time.
terry.suereth@gmail.com
On Sun, Aug 18, 2013 at 6:54 PM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> Hi Tejun,
>
> After merging the libata tree, today's linux-next build (powerpc
> ppc64_defconfig) failed like this:
>
> drivers/ata/libata-pmp.c: In function 'sata_pmp_quirks':
> drivers/ata/libata-pmp.c:386:36: error: invalid suffix "x26" on integer constant
> if (vendor == 0x1095 && (devid == 0x3x26 || devid == 0x3826)) {
> ^
>
> Caused by commit f1a313ad86b9 ("libata: apply behavioral quirks to sil3826 PMP").
>
> I have used the libata tree from next-20130816 for today.
> --
> Cheers,
> Stephen Rothwell sfr@canb.auug.org.au
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2013-08-19 3:18 ` Terry Suereth
@ 2013-08-19 13:40 ` Tejun Heo
0 siblings, 0 replies; 22+ messages in thread
From: Tejun Heo @ 2013-08-19 13:40 UTC (permalink / raw)
To: Terry Suereth; +Cc: Stephen Rothwell, linux-next, linux-kernel, stable
On Sun, Aug 18, 2013 at 08:18:55PM -0700, Terry Suereth wrote:
> Per my original patch, "0x3x26" in code should be "0x3726" (as in the
> unpatched version). I refer to "x26" in comments as a forward-looking
> assumption, but AFAIK only 3726 and 3826 exist at this time.
Heh... No idea how that happened. Fixed. Sorry about the trouble.
--
tejun
^ permalink raw reply [flat|nested] 22+ messages in thread
* linux-next: build failure after merge of the libata tree
@ 2013-08-27 2:50 Stephen Rothwell
2013-08-27 5:52 ` Aaron Lu
0 siblings, 1 reply; 22+ messages in thread
From: Stephen Rothwell @ 2013-08-27 2:50 UTC (permalink / raw)
To: Tejun Heo; +Cc: linux-next, linux-kernel, Aaron Lu
[-- Attachment #1: Type: text/plain, Size: 808 bytes --]
Hi Tejun,
After merging the libata tree, today's linux-next build (powerpc
ppc64_defconfig) failed like this:
In file included from drivers/ata/libata-pmp.c:14:0:
drivers/ata/libata.h:136:1: error: unknown type name 'acpi_handle'
static inline acpi_handle ata_dev_acpi_handle(struct ata_device *dev) { return NULL; }
^
And several others.
Caused by commit f1bc1e4c44b1 ("ata: acpi: rework the ata acpi bind
support"). All the world does not have the ACPI include files
included via some other path ... nor wants them, I suspect.
I have used the libata tree from next-20130822 for today.
P.S. drivers/ata/libata.h seems to assume that a lot of other stuff is
included that it does not include explicitly ...
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2013-08-27 2:50 Stephen Rothwell
@ 2013-08-27 5:52 ` Aaron Lu
2013-08-27 18:09 ` Tejun Heo
0 siblings, 1 reply; 22+ messages in thread
From: Aaron Lu @ 2013-08-27 5:52 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: Tejun Heo, linux-next, linux-kernel, linux-ide
On 08/27/2013 10:50 AM, Stephen Rothwell wrote:
> Hi Tejun,
>
> After merging the libata tree, today's linux-next build (powerpc
> ppc64_defconfig) failed like this:
>
> In file included from drivers/ata/libata-pmp.c:14:0:
> drivers/ata/libata.h:136:1: error: unknown type name 'acpi_handle'
> static inline acpi_handle ata_dev_acpi_handle(struct ata_device *dev) { return NULL; }
> ^
>
> And several others.
>
> Caused by commit f1bc1e4c44b1 ("ata: acpi: rework the ata acpi bind
> support"). All the world does not have the ACPI include files
> included via some other path ... nor wants them, I suspect.
Sorry for the trouble, my bad.
Does the below patch fix the problem?
From: Aaron Lu <aaron.lu@intel.com>
Subject: [PATCH] ata: acpi: Remove ata_dev_acpi_handle stub in libata.h
The ata_dev_acpi_handle is defined in libata-acpi.c and the only
external user is libata-zpodd.c, which is built when CONFIG_ATA_ACPI
is set, so there is no need to make an empty stub function for
!CONFIG_ATA_ACPI case in libata.h. It also causes compile errors due to
acpi_handle is not defined when !CONFIG_ACPI. This patch fixes this
problem by removing the empty stub.
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Aaron Lu <aaron.lu@intel.com>
---
drivers/ata/libata.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/ata/libata.h b/drivers/ata/libata.h
index 3e9cc95..eeeb778 100644
--- a/drivers/ata/libata.h
+++ b/drivers/ata/libata.h
@@ -133,7 +133,6 @@ static inline void ata_acpi_set_state(struct ata_port *ap,
pm_message_t state) { }
static inline void ata_acpi_bind_port(struct ata_port *ap) {}
static inline void ata_acpi_bind_dev(struct ata_device *dev) {}
-static inline acpi_handle ata_dev_acpi_handle(struct ata_device *dev) { return NULL; }
static inline void ata_scsi_acpi_bind(struct ata_device *dev) {}
static inline void ata_scsi_acpi_unbind(struct ata_device *dev) {}
#endif
--
1.8.3.2.10.g43d11f4
Thanks,
Aaron
>
> I have used the libata tree from next-20130822 for today.
>
> P.S. drivers/ata/libata.h seems to assume that a lot of other stuff is
> included that it does not include explicitly ...
>
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2013-08-27 5:52 ` Aaron Lu
@ 2013-08-27 18:09 ` Tejun Heo
0 siblings, 0 replies; 22+ messages in thread
From: Tejun Heo @ 2013-08-27 18:09 UTC (permalink / raw)
To: Aaron Lu; +Cc: Stephen Rothwell, linux-next, linux-kernel, linux-ide
On Tue, Aug 27, 2013 at 01:52:09PM +0800, Aaron Lu wrote:
> From: Aaron Lu <aaron.lu@intel.com>
> Subject: [PATCH] ata: acpi: Remove ata_dev_acpi_handle stub in libata.h
>
> The ata_dev_acpi_handle is defined in libata-acpi.c and the only
> external user is libata-zpodd.c, which is built when CONFIG_ATA_ACPI
> is set, so there is no need to make an empty stub function for
> !CONFIG_ATA_ACPI case in libata.h. It also causes compile errors due to
> acpi_handle is not defined when !CONFIG_ACPI. This patch fixes this
> problem by removing the empty stub.
>
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Signed-off-by: Aaron Lu <aaron.lu@intel.com>
Yeap, tested and works. Applied to libata/for-3.12. Thanks.
Stephen, sorry about the trouble.
--
tejun
^ permalink raw reply [flat|nested] 22+ messages in thread
* linux-next: build failure after merge of the libata tree
@ 2014-02-26 2:55 Stephen Rothwell
2014-02-26 13:49 ` Tejun Heo
0 siblings, 1 reply; 22+ messages in thread
From: Stephen Rothwell @ 2014-02-26 2:55 UTC (permalink / raw)
To: Tejun Heo; +Cc: linux-next, linux-kernel, Lee Jones
[-- Attachment #1: Type: text/plain, Size: 658 bytes --]
Hi Tejun,
After merging the libata tree, today's linux-next build (x86_64
allmodconfig) failed like this:
drivers/ata/ahci_st.c: In function 'st_ahci_probe':
drivers/ata/ahci_st.c:159:8: error: implicit declaration of function 'ahci_platform_put_resources' [-Werror=implicit-function-declaration]
ahci_platform_put_resources(&pdev->dev, hpriv);
^
Caused by commit 0d8d213703ff ("ahci: st: Add support for ST's SATA
IP"). That function is defined as "static" in another file ...
I have used the version of the libata tree from next-20140225 for today.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2014-02-26 2:55 Stephen Rothwell
@ 2014-02-26 13:49 ` Tejun Heo
0 siblings, 0 replies; 22+ messages in thread
From: Tejun Heo @ 2014-02-26 13:49 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: linux-next, linux-kernel, Lee Jones
On Wed, Feb 26, 2014 at 01:55:21PM +1100, Stephen Rothwell wrote:
> Hi Tejun,
>
> After merging the libata tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> drivers/ata/ahci_st.c: In function 'st_ahci_probe':
> drivers/ata/ahci_st.c:159:8: error: implicit declaration of function 'ahci_platform_put_resources' [-Werror=implicit-function-declaration]
> ahci_platform_put_resources(&pdev->dev, hpriv);
> ^
>
> Caused by commit 0d8d213703ff ("ahci: st: Add support for ST's SATA
> IP"). That function is defined as "static" in another file ...
>
> I have used the version of the libata tree from next-20140225 for today.
Right, that patch was against the previous version which didn't use
devres. The put_resources calls are no longer necessary. Will revert
the patch for now.
Thanks.
--
tejun
^ permalink raw reply [flat|nested] 22+ messages in thread
* linux-next: build failure after merge of the libata tree
@ 2014-04-24 1:17 Stephen Rothwell
2014-04-24 12:25 ` Tejun Heo
0 siblings, 1 reply; 22+ messages in thread
From: Stephen Rothwell @ 2014-04-24 1:17 UTC (permalink / raw)
To: Tejun Heo; +Cc: linux-next, linux-kernel, Kefeng Wang, Hans de Goede
[-- Attachment #1: Type: text/plain, Size: 758 bytes --]
Hi Tejun,
After merging the libata tree, today's linux-next build (x86_64
allmodconfig) failed like this:
drivers/ata/libahci_platform.c: In function 'ahci_platform_init_host':
drivers/ata/libahci_platform.c:317:20: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
pi.private_data = (void *)host_flags
^
drivers/ata/libahci_platform.c:318:2: error: expected ';' before 'hpriv'
hpriv->flags |= host_flags;
^
Please test build this stuff ...
Caused by commit ac795a7ec2d2 ("libahci_platform: add host_flags parameter
in ahci_platform_init_host()").
I have used the libata tree from next-20140423 for today.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2014-04-24 1:17 Stephen Rothwell
@ 2014-04-24 12:25 ` Tejun Heo
0 siblings, 0 replies; 22+ messages in thread
From: Tejun Heo @ 2014-04-24 12:25 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: linux-next, linux-kernel, Kefeng Wang, Hans de Goede
Hello,
On Thu, Apr 24, 2014 at 11:17:54AM +1000, Stephen Rothwell wrote:
> After merging the libata tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
>
> drivers/ata/libahci_platform.c: In function 'ahci_platform_init_host':
> drivers/ata/libahci_platform.c:317:20: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
> pi.private_data = (void *)host_flags
> ^
> drivers/ata/libahci_platform.c:318:2: error: expected ';' before 'hpriv'
> hpriv->flags |= host_flags;
> ^
>
> Please test build this stuff ...
My apologies.
> Caused by commit ac795a7ec2d2 ("libahci_platform: add host_flags parameter
> in ahci_platform_init_host()").
>
> I have used the libata tree from next-20140423 for today.
Reverted.
Thanks.
--
tejun
^ permalink raw reply [flat|nested] 22+ messages in thread
* linux-next: build failure after merge of the libata tree
@ 2015-09-17 4:03 Stephen Rothwell
2015-09-17 11:14 ` Luis de Bethencourt
0 siblings, 1 reply; 22+ messages in thread
From: Stephen Rothwell @ 2015-09-17 4:03 UTC (permalink / raw)
To: Tejun Heo; +Cc: linux-next, linux-kernel, Luis de Bethencourt
Hi Tejun,
After merging the libata tree, today's linux-next build (powercp
allmodconfig) failed like this:
drivers/ata/pata_macio.c:1347:1: error: 'pata_macio_patch' undeclared here (not in a function)
drivers/ata/pata_macio.c:1347:1: error: '__mod_of__pata_macio_patch_device_table' aliased to undefined symbol 'pata_macio_patch'
Caused by commit
5d8619769cf5 ("ata: pata_macio: Fix module autoload for OF platform driver")
A simple typo ("patch" v "match"), but clearly never build tested.
Only found after the release.
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2015-09-17 4:03 Stephen Rothwell
@ 2015-09-17 11:14 ` Luis de Bethencourt
0 siblings, 0 replies; 22+ messages in thread
From: Luis de Bethencourt @ 2015-09-17 11:14 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: Tejun Heo, linux-next, linux-kernel
On Thu, Sep 17, 2015 at 02:03:36PM +1000, Stephen Rothwell wrote:
> Hi Tejun,
>
> After merging the libata tree, today's linux-next build (powercp
> allmodconfig) failed like this:
>
> drivers/ata/pata_macio.c:1347:1: error: 'pata_macio_patch' undeclared here (not in a function)
> drivers/ata/pata_macio.c:1347:1: error: '__mod_of__pata_macio_patch_device_table' aliased to undefined symbol 'pata_macio_patch'
>
> Caused by commit
>
> 5d8619769cf5 ("ata: pata_macio: Fix module autoload for OF platform driver")
>
> A simple typo ("patch" v "match"), but clearly never build tested.
>
> Only found after the release.
>
> --
> Cheers,
> Stephen Rothwell sfr@canb.auug.org.au
Sorry, this was my fault.
When I created the second version of the patch, based on -stable instead of
-next the typo creeped in and I didn't test properly.
Sending a correct version of the patch now.
Apologies,
Luis
^ permalink raw reply [flat|nested] 22+ messages in thread
* linux-next: build failure after merge of the libata tree
@ 2016-05-09 0:14 Stephen Rothwell
0 siblings, 0 replies; 22+ messages in thread
From: Stephen Rothwell @ 2016-05-09 0:14 UTC (permalink / raw)
To: Tejun Heo; +Cc: linux-next, linux-kernel, Mans Rullgard, Christian Lamparter
Hi Tejun,
After merging the libata tree, today's linux-next build (x86_64
allmodconfig) failed like this:
drivers/ata/sata_dwc_460ex.c:203:2: error: unknown field 'm_master' specified in initializer
.m_master = 1,
^
/home/sfr/next/next/drivers/ata/sata_dwc_460ex.c:204:2: error: unknown field 'p_master' specified in initializer
.p_master = 0,
^
/home/sfr/next/next/drivers/ata/sata_dwc_460ex.c: In function 'sata_dwc_dma_init_old':
/home/sfr/next/next/drivers/ata/sata_dwc_460ex.c:269:9: error: too few arguments to function 'dw_dma_probe'
return dw_dma_probe(hsdev->dma);
^
Caused by commit
30ee9d52f7a7 ("ata: sata_dwc_460ex: use "dmas" DT property to find dma channel")
I have used the libata tree from next-20160506 for today.
--
Cheers,
Stephen Rothwell
^ permalink raw reply [flat|nested] 22+ messages in thread
* linux-next: build failure after merge of the libata tree
@ 2022-01-06 14:08 Stephen Rothwell
2022-01-06 23:58 ` Damien Le Moal
0 siblings, 1 reply; 22+ messages in thread
From: Stephen Rothwell @ 2022-01-06 14:08 UTC (permalink / raw)
To: Damien Le Moal
Cc: Hannes Reinecke, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 417 bytes --]
Hi all,
After merging the libata tree, today's linux-next build (powerpc
allyesconfig) failed like this:
drivers/ata/pata_cs5535.c:26:10: fatal error: asm/msr.h: No such file or directory
26 | #include <asm/msr.h>
| ^~~~~~~~~~~
Caused by commit
39d0a5fc445a ("ata: pata_cs5535: add compile test support")
I have reverted that commit for today.
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2022-01-06 14:08 Stephen Rothwell
@ 2022-01-06 23:58 ` Damien Le Moal
2022-01-07 0:17 ` Stephen Rothwell
0 siblings, 1 reply; 22+ messages in thread
From: Damien Le Moal @ 2022-01-06 23:58 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Hannes Reinecke, Linux Kernel Mailing List, Linux Next Mailing List
On 2022/01/06 23:08, Stephen Rothwell wrote:
> Hi all,
>
> After merging the libata tree, today's linux-next build (powerpc
> allyesconfig) failed like this:
>
> drivers/ata/pata_cs5535.c:26:10: fatal error: asm/msr.h: No such file or directory
> 26 | #include <asm/msr.h>
> | ^~~~~~~~~~~
>
> Caused by commit
>
> 39d0a5fc445a ("ata: pata_cs5535: add compile test support")
>
> I have reverted that commit for today.
>
I fixed that yesterday afternoon my time (Japan), so before your morning in the
US (assuming you are based there).
--
Damien Le Moal
Western Digital Research
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2022-01-06 23:58 ` Damien Le Moal
@ 2022-01-07 0:17 ` Stephen Rothwell
2022-01-07 1:28 ` Damien Le Moal
0 siblings, 1 reply; 22+ messages in thread
From: Stephen Rothwell @ 2022-01-07 0:17 UTC (permalink / raw)
To: Damien Le Moal
Cc: Hannes Reinecke, Linux Kernel Mailing List, Linux Next Mailing List
[-- Attachment #1: Type: text/plain, Size: 613 bytes --]
Hi Damien,
On Fri, 7 Jan 2022 08:58:48 +0900 Damien Le Moal <damien.lemoal@opensource.wdc.com> wrote:
>
> I fixed that yesterday afternoon my time (Japan), so before your morning in the
> US (assuming you are based there).
No, Australia (so +1100) at the moment (see my email headers :-)).
I do fetch trees regularly during the day, but I merged your tree at
about 13:28 yesterday (11:28 your time), but the build failure did not
happen until I did the powerpc allyesconfig build at about 1am this
morning.
Thanks for fixing this, it will be in today's release.
--
Cheers,
Stephen Rothwell
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: linux-next: build failure after merge of the libata tree
2022-01-07 0:17 ` Stephen Rothwell
@ 2022-01-07 1:28 ` Damien Le Moal
0 siblings, 0 replies; 22+ messages in thread
From: Damien Le Moal @ 2022-01-07 1:28 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Hannes Reinecke, Linux Kernel Mailing List, Linux Next Mailing List
On 2022/01/07 9:17, Stephen Rothwell wrote:
> Hi Damien,
>
> On Fri, 7 Jan 2022 08:58:48 +0900 Damien Le Moal <damien.lemoal@opensource.wdc.com> wrote:
>>
>> I fixed that yesterday afternoon my time (Japan), so before your morning in the
>> US (assuming you are based there).
>
> No, Australia (so +1100) at the moment (see my email headers :-)).
Indeed. So you are only two hour ahead of me :)
>
> I do fetch trees regularly during the day, but I merged your tree at
> about 13:28 yesterday (11:28 your time), but the build failure did not
> happen until I did the powerpc allyesconfig build at about 1am this
> morning.
OK. My update went in later in the day. I tested only on my x86_64 host native
compile. I did not setup cross compile for all arch.
>
> Thanks for fixing this, it will be in today's release.
Great ! Thanks !
--
Damien Le Moal
Western Digital Research
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2022-01-07 1:28 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-08-18 0:40 linux-next: build failure after merge of the libata tree Stephen Rothwell
2010-08-18 0:46 ` Gwendal Grignou
2010-08-19 20:15 ` Jeff Garzik
2012-07-02 2:10 Stephen Rothwell
2012-07-02 2:19 ` huang ying
2013-08-19 1:54 Stephen Rothwell
2013-08-19 3:18 ` Terry Suereth
2013-08-19 13:40 ` Tejun Heo
2013-08-27 2:50 Stephen Rothwell
2013-08-27 5:52 ` Aaron Lu
2013-08-27 18:09 ` Tejun Heo
2014-02-26 2:55 Stephen Rothwell
2014-02-26 13:49 ` Tejun Heo
2014-04-24 1:17 Stephen Rothwell
2014-04-24 12:25 ` Tejun Heo
2015-09-17 4:03 Stephen Rothwell
2015-09-17 11:14 ` Luis de Bethencourt
2016-05-09 0:14 Stephen Rothwell
2022-01-06 14:08 Stephen Rothwell
2022-01-06 23:58 ` Damien Le Moal
2022-01-07 0:17 ` Stephen Rothwell
2022-01-07 1:28 ` Damien Le Moal
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).