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

udev rules for d4xx mipi dfu access #10736

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

dmipx
Copy link
Contributor

@dmipx dmipx commented Aug 1, 2022

  • set access for /dev/d4xx-dfu* to read/write by 'video' group

Signed-off-by: Dmitry Perchanov [email protected]

@dmipx dmipx force-pushed the d4xx_mipi_dfu_udev branch from 09723cb to 4d27333 Compare August 1, 2022 11:35
@dmipx dmipx requested a review from Nir-Az August 4, 2022 07:14
@Nir-Az Nir-Az requested review from remibettan and ev-mp August 4, 2022 07:14
Copy link
Contributor

@remibettan remibettan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Nir-Az
Copy link
Collaborator

Nir-Az commented Aug 4, 2022

@ev-mp / @remibettan / @dmipx don't we want this based and merged to the D4XX_MIPI branch?
merging to development will be done on a later stage no?

Currently it's origin from master and that's why we have testings failing..
We expect no failed tests.

@dmipx dmipx force-pushed the d4xx_mipi_dfu_udev branch from 4d27333 to 4276e1f Compare August 4, 2022 10:33
@remibettan
Copy link
Contributor

@Nir-Az I agree it can be "based and merged to the D4XX_MIPI branch".

- set access for /dev/d4xx-dfu* to read/write by 'video' group

Signed-off-by: Dmitry Perchanov <[email protected]>
@dmipx dmipx force-pushed the d4xx_mipi_dfu_udev branch from 4276e1f to fee7770 Compare August 4, 2022 11:35
@dmipx dmipx changed the base branch from development to D4XX_MIPI August 4, 2022 11:36
@dmipx
Copy link
Contributor Author

dmipx commented Aug 4, 2022

@Nir-Az I agree it can be "based and merged to the D4XX_MIPI branch".

so be it

Copy link
Collaborator

@ev-mp ev-mp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR looks good to me

@ev-mp ev-mp merged commit cd5dc9d into IntelRealSense:D4XX_MIPI Aug 8, 2022
@dmipx dmipx deleted the d4xx_mipi_dfu_udev branch February 26, 2023 07:58
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

Successfully merging this pull request may close these issues.

4 participants