search

devicetree-org / lopper

Other
33 stars 40 forks source link

A zero `cpu-mask` means "all of them" #166

Open mbolivar-nordic opened 1 year ago

mbolivar-nordic commented 1 year ago

As of 4fc81d45e35cd4cf3ff19d2a837c6f15c1afbf4b, the cpu-mask property in the "openamp,domain-v1" binding says:

- *cpu-mask* is a bitfield indicating the subset of CPUs in the cluster which
  the domain runs on

I'd like to propose that we extend this binding to say that a value of 0 means that the domain runs on all of the CPUs in the cluster.

This seems like a common case and it is a useful shorthand which may help users, especially since we could define a self-documenting macro like DOMAIN_CPU_MASK_ALL for this.

sstabellini commented 1 year ago

I don't have a strong opinion on this one. I think that's OK either way. If you'd like to add it to the spec, I am OK with it.