Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enhancement: addition of /delete-label/ #64

Open
de-nordic opened this issue Apr 5, 2023 · 3 comments
Open

enhancement: addition of /delete-label/ #64

de-nordic opened this issue Apr 5, 2023 · 3 comments

Comments

@de-nordic
Copy link

Currently it is not possible to delete label independently from deleting entire node.

This is very inconvenient because it does not allow to move labels from node to node, when needed, without redefining
the node again.

References to labels may be used for property values (or as phandles), but when referenced node needs to be changed, due to lack of /delete-label/, it is required to remove property referencing the label for each node and then add it back referencing node with different label. This could be much easier if label could be removed from one node and added to another.

@robherring
Copy link
Member

The first step here is to implement this for dtc or propose the concept on the dtc maillist.

The spec is already not complete in terms of DTS directives. For example /bits/, /omit-if-no-ref/, and /incbin/ are all not in the spec. Perhaps that should be a requirement, but it is not ATM.

In any case, submitting a GH issue here is not going to get you a feature. There is no one working on spec enhancements.

@de-nordic
Copy link
Author

Thank you for response. I thought that specification states what DT compilers should implement, that is why I have proposed the enhancement here.

@robherring
Copy link
Member

Thank you for response. I thought that specification states what DT compilers should implement, that is why I have proposed the enhancement here.

While that makes sense, the reality is that the spec and dtc are not that decoupled. And proposing this for dtc will get wider review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants