Re: [PATCH 1/2] SGI hotplug: fix build failure

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

 



On Mon, 27 Jul 2009 12:05:38 +0900
Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx> wrote:

> The commit bd3d99c17039fd05a29587db3f4a180c48da115a ("PCI: Remove
> untested Electromechanical Interlock (EMI) support in pciehp."), which
> removes the definition of "struct hotplug_slot_attr", broke SGI
> hotplug driver. By this commit, we get the following compile error.
> 
> 
> drivers/pci/hotplug/sgi_hotplug.c:106: error: variable
> 'sn_slot_path_attr' has initializer but incomplete type
> drivers/pci/hotplug/sgi_hotplug.c:106: error: unknown field 'attr'
> specified in initializer drivers/pci/hotplug/sgi_hotplug.c:106:
> error: extra brace group at end of initializer
> drivers/pci/hotplug/sgi_hotplug.c:106: error: (near initialization
> for 'sn_slot_path_attr') drivers/pci/hotplug/sgi_hotplug.c:106:
> warning: excess elements in struct initializer
> drivers/pci/hotplug/sgi_hotplug.c:106: warning: (near initialization
> for 'sn_slot_path_attr') drivers/pci/hotplug/sgi_hotplug.c:106:
> error: unknown field 'show' specified in initializer
> drivers/pci/hotplug/sgi_hotplug.c:106: warning: excess elements in
> struct initializer drivers/pci/hotplug/sgi_hotplug.c:106: warning:
> (near initialization for 'sn_slot_path_attr')
> drivers/pci/hotplug/sgi_hotplug.c: In function 'sn_hp_destroy':
> drivers/pci/hotplug/sgi_hotplug.c:203: error: invalid use of
> undefined type 'struct hotplug_slot_attribute'
> drivers/pci/hotplug/sgi_hotplug.c: In function
> 'sn_hotplug_slot_register': drivers/pci/hotplug/sgi_hotplug.c:655:
> error: invalid use of undefined type 'struct hotplug_slot_attribute'
> 
> 
> This patch fixes this regression by adding the definition of struct
> hotplug_slot_attr into sgi_hotplug.c.
> 
> Signed-off-by: Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx>
> 
> ---
>  drivers/pci/hotplug/sgi_hotplug.c |    6 ++++++
>  1 file changed, 6 insertions(+)
> 

Applied these two to linux-next for now, thanks.

-- 
Jesse Barnes, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux