> If ii is alright with you I will change the dev_err(...)messages to > dev_dbg(...) so that they do not spoil the logs. OK. > >> + rc = of_property_read_string(dev->device->of_node, "interrupt-names", > >> + &int_name); > > > > I haven't checked but is that really needed? of_irq_to_resource() seems > > to parse the "interrupt-names" property > > Since the driver also fall's back to polling it will not work in case > there is no irq domain assigned. The above approach works for both > cases. So we do need it. I still wonder if there isn't a better way since the name of the resource is set already, but if so we can fix this incrementally, so OK for now.
Attachment:
signature.asc
Description: Digital signature