Re: [PATCH 05/11] hw/gpio/avr_gpio: Add 'id' field in AVRGPIOState

[Richard Henderson]

On 3/13/21 10:54 AM, Philippe Mathieu-Daudé wrote:
+    if (s->id == UINT8_MAX) {
+        error_setg(errp, "property 'id' not set");
+        return;
+    }

This error message would be a tad confusing if one set the id to 255.  What's 
the point?


r~