On 11/10/2018 14:29, P J P wrote: > +-- On Thu, 11 Oct 2018, Philippe Mathieu-Daudé wrote --+ > | The IOReadHandler does not have documentation. > | > | typedef void IOReadHandler(void *opaque, const uint8_t *buf, int size); > | > | Why is the 'size' argument signed? Does it makes sens to call it with a > | negative value? > > No, it doesn't IMO. I had first changed argument type 'int' to uint32_t'. > as > > typedef void IOReadHandler(void *opaque, const uint8_t *buf, uint32_t size); > > But 'IOReadHandler' is registered and called from multiple char devices, > which lead to compile time errors. As the function prototype changed. > > I'll update them all, if the above change is okay.
I started this change and already converted 40 files.
