-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[rotel] Fix request of play status and track number #13148
Conversation
Impacted channels: track and playControl for models CD11, CD14 and RCD1572 Signed-off-by: Laurent Garnier <lg.hc@free.fr>
14b1a1a
to
a230217
Compare
@lolodomo - can you describe the problem which is fixed? Code looks simple, but I have a hard time assessing it without any context. For the last two commands I can guess that question mark might be needed for querying values? For |
As "?" was missing, the commands to retrieve the current playback status and current track number were certainly not working and corresponding channels were not set at Initialization. Regarding the command to get the DSP mode, this is different. The 3 models use the ASCII V1 protocol and so the command get_dsp_mode. The command for ASCII V2 protocol is just not yet known. |
Are some models still using the ASCII V1 protocol? Since that enum seems not model-specific, will |
Ah, sorry, now I get it. Last argument is for V2 protocol, and it's not known, so you removed it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fix! Can you update the PR title to mention that requesting play status and track has been fixed, as this seems like a bugfix that could actually affect users?
Impacted channels: track and playControl for models CD11, CD14 and RCD1572 Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Impacted channels: track and playControl for models CD11, CD14 and RCD1572 Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Impacted channels: track and playControl for models CD11, CD14 and RCD1572 Signed-off-by: Laurent Garnier <lg.hc@free.fr> Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
Impacted channels: track and playControl for models CD11, CD14 and RCD1572 Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Impacted channels: track and playControl for models CD11, CD14 and RCD1572 Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Impacted channels: track and playControl for models CD11, CD14 and RCD1572
Signed-off-by: Laurent Garnier lg.hc@free.fr