On 09/19/2017 03:27 PM, Philippe Mathieu-Daudé wrote:
> On 09/19/2017 08:26 AM, Cédric Le Goater wrote:
>> Specs are available here :
>>
>>    https://www.nxp.com/docs/en/data-sheet/PCA9552.pdf
>>
>> This is a simple model supporting the basic registers for led and GPIO
>> mode. The device also supports two blinking rates but not the model
>> yet.
>>
>> Signed-off-by: Cédric Le Goater <[email protected]>
>> ---
>>   default-configs/arm-softmmu.mak |   1 +
>>   hw/misc/Makefile.objs           |   1 +
>>   hw/misc/pca9552.c               | 212 
>> ++++++++++++++++++++++++++++++++++++++++
>>   include/hw/misc/pca9552.h       |  32 ++++++
> 
> I'd rather move it to hw/display/

The device is not related to displays. It is just an I2C expander 
chip. I would leave it under misc/

C. 

> 
>>   4 files changed, 246 insertions(+)
>>   create mode 100644 hw/misc/pca9552.c
>>   create mode 100644 include/hw/misc/pca9552.h


Reply via email to