Re: [PATCH] arch: m68k: include: asm: the 3rd parameter of 'insl' and 'outsl' need '<< 2'

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

 



On 06/01/2013 08:38 AM, schmitz wrote:
All,
Geert ,

The related git number:
  for parport.h: "4914802 m68k,m68knommu: merge header files" in 2009
  for io_mm.h: "84b16b7 m68k/atari: ROM port ISA adapter support" in
Apr 6 2013

The related warning (make EXTRA_CFLAG=-W ARCH=m68k allmodconfig):
  arch/m68k/include/asm/parport.h:14:0: warning: "insl" redefined
[enabled by default]
  arch/m68k/include/asm/io_mm.h:403:0: note: this is the location of
the previous definition
  arch/m68k/include/asm/parport.h:15:0: warning: "outsl" redefined
[enabled by default]
  arch/m68k/include/asm/io_mm.h:406:0: note: this is the location of
the previous definition
    
Is that the same problem Thorsten reported recently? parport.h should
either use what the arch io.h include defined, or (in the case of Q40
on m68k) undef and redefine as needed.
It appears this is the same issue, see message ID
loom.20130511T171757-995@xxxxxxxxxxxxxx to linux-m68k (May 12th, by
Thorsten Glaser).

This _only_ applies to use of insl/outsl macros in parport_pc.h, which
is only used by Q40 on m68k. I see no reason to change anything in io.h
to cope with this warning.

It sounds reasonable.

And excuse me, could you provide the related link directly ?  I don't
know how to see the detail of 'loom.20130511T171757-995@xxxxxxxxxxxxxx'.


Thanks.
-- 
Chen Gang

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




[Index of Archives]     [Video for Linux]     [Yosemite News]     [Linux S/390]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux