On Mon, 2019-01-21 at 11:17 -0500, Nicolas Dufresne wrote: [...] > > > +Example: > > > + al5e: al5e@a0009000 { > > > > Should the node names be "vpu" or "video-codec"? > > Xilinx calls this IP the "vcu", so "vpu" would be even more confusing. > Was this just a typo ? I was just going by what is already commonly used in other device trees. Given that "vpu" doesn't seem to be as universally accepted as for example "gpu", maybe "video-codec" is the better choice? In any case, it should not matter what the the IP vendor or the SoC vendor call this, the node name should be common and descriptive of its function, same as ethernet controller nodes should be called "ethernet". > That being said, is this referring to the actual > HW or the firmware that runs on a microblaze (the firmware being > Allegro specific) ? As I understand it this binding is referring to the whole [microblaze + attached encoder/decoder hardware] setup that is supposed to work with the Allegro specific firmware. regards Philipp