Thread (3 messages) 3 messages, 2 authors, 2020-08-04

Re: [PATCH 2/2] speakup: only build serialio when ISA is enabled

From: Samuel Thibault <samuel.thibault@ens-lyon.org>
Date: 2020-08-04 16:12:34
Also in: linux-riscv, lkml

Greg KH, le mar. 04 août 2020 17:49:52 +0200, a ecrit:
quoted
#define inb(c)		({ u8  __v; __io_pbr(); __v = readb_cpu((void*)(PCI_IOBASE + (c))); __io_par(__v); __v; })

and thus yes it's arithmetic over a (void*) pointer, the caller cannot
do anything about it.
And that's fine, math with pointers, even void ones, is ok.
C++ doesn't like it, but in general compilers provide some semantic for
it, yes.
I wonder why riscv was complaining about that.  It's not nice, but it is
valid C.
Strictly C speaking, no, but GNU C yes.
Let's see if any of their developers care about it :)
Ok :) In the meanwhile I sent a v2 series that should fix the different
link cases, just not the particular riscv warning.

Samuel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help