On Tue, Nov 25, 2014 at 12:14:48PM +0100, Jean-Francois Moine wrote:
As Russell King's explained it, there should not be pointers to struct device_node:
"struct device_node is a ref-counted structure. That means if you store a reference to it, you should "get" it, and you should "put" it once you've done. The act of "put"ing the pointed-to structure involves writing to that structure, so it is totally unappropriate to store a device_node structure as a const pointer. It forces you to have to cast it back to a non-const pointer at various points in time to use various OF function calls."
So, we're not holding references here (we're just doing comparisons, the references need to be owned before we get into the core) and I'm not seeing anything here removing casts?