On 1/7/19 8:22 AM, Thomas Huth wrote: > On 2019-01-04 18:58, Philippe Mathieu-Daudé wrote: >> Since uWireSlave is only used in this new header, there is no >> need to expose it via "qemu/typedefs.h". >> >> Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> >> --- >> MAINTAINERS | 2 ++ >> hw/arm/nseries.c | 2 +- >> hw/arm/palm.c | 2 +- >> hw/input/tsc2005.c | 2 +- >> hw/input/tsc210x.c | 4 ++-- >> include/hw/arm/omap.h | 6 +----- >> include/hw/devices.h | 14 -------------- >> include/hw/input/ti_uwire_tsc.h | 28 ++++++++++++++++++++++++++++ > > I'd maybe rather name the new file "tsc2xxx.h" instead, so that it > resembles the names of the .c files. Just my 0.02 €.
Good idea :)