Re: linux-next: build failure after merge of the scsi tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/23/2012 06:26 AM, Alexey Dobriyan wrote:
On Fri, Mar 23, 2012 at 7:50 AM, Benjamin Herrenschmidt
<benh@xxxxxxxxxxxxxxxxxxx>  wrote:
On Fri, 2012-03-23 at 13:00 +1100, Stephen Rothwell wrote:
Hi James,

After merging the scsi tree, today's linux-next build (powerpc
ppc64_defconfig) failed like this:

drivers/ata/sata_svw.c: In function 'k2_sata_proc_open':
drivers/ata/sata_svw.c:327:2: error: implicit declaration of function 'PDE' [-Werror=implicit-function-declaration]
drivers/ata/sata_svw.c:327:56: error: invalid type argument of '->' (have 'int')
drivers/ata/sata_svw.c:328:1: warning: control reaches end of non-void function [-Wreturn-type]

Caused by commit 343ac4b031a7 ("[SCSI] sata_svw: convert to scsi proc_ops").

I have used the scsi tree from next-20120322 for today.

Additionally, I am the author and maintainer of that driver and I don't
remember ever being CCed on that patch ... or did I miss it ?

Your email was not mentioned in changelogs for 6 years nor the driver
has specific MAINTAINERS entry.


At the top of drivers/ata/sata_svw.c, we find:

 *  Maintained by: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> and
 *                 Jeff Garzik <jgarzik@xxxxxxxxx>
 *                  Please ALWAYS copy linux-ide@xxxxxxxxxxxxxxx
 *                  on emails.

and in MAINTAINERS we also find

SERIAL ATA (SATA) SUBSYSTEM
M:      Jeff Garzik <jgarzik@xxxxxxxxx>
L:      linux-ide@xxxxxxxxxxxxxxx
T: git git://git.kernel.org/pub/scm/linux/kernel/git/jgarzik/libata-dev.git
S:      Supported
F:      drivers/ata/

None of these addresses, nor LKML, were copied.  Please do so in the future.

	Jeff



--
To unsubscribe from this list: send the line "unsubscribe linux-next" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel]     [Linux USB Development]     [Yosemite News]     [Linux SCSI]

  Powered by Linux