Hi Andrew, On 20/11/2023 14:56, Andrew Cooper wrote:
@@ -1662,7 +1662,7 @@ int dt_device_get_irq(const struct dt_device_node *device, unsigned int index, return dt_irq_translate(&raw, out_irq); }-bool_t dt_device_is_available(const struct dt_device_node *device)+bool dt_device_is_available(const struct dt_device_node *device) { const char *status; u32 statlen; @@ -1680,7 +1680,7 @@ bool_t dt_device_is_available(const struct dt_device_node *device) return 0;
Just because it is in context, shouldn't this now be 'return false'? There might be others, but they can be modified afterwards (this patch is already large).
Acked-by: Julien Grall <[email protected]> Cheers, -- Julien Grall
