Add card number option for when multiple cards are on same bus

[chipfreitag]

No description provided.

[clbr]

Is it possible for two pcie cards to be on the same bus? Where does this happen?

Notes on the patch:

• this breaks the short -b option
• please no distro-specific instructions in the readme, not everyone uses Ubuntu/Debian
• leftover debug printfs, and wrongly indented too
• function param should be const
• when adding new options, please also update radeontop.asc

[chipfreitag]

Thanks for the quick review. Yes sorry about the sloppy printfs and the notes in the READM. I should not have included those. In the past I have used the –b option to select from multiple adaptors. I was not able to do so recently, but this morning discovered that your interface does work, you just have to enter a decimal number for the device you want. I will withdraw the pull request. One enhancement I have made (locally) Is to include the busid string in the printcenter for the screen header. This way I can see exactly which device has been chosen. From: clbr [mailto:notifications@github.com] Sent: Tuesday, January 23, 2018 3:00 AM To: clbr/radeontop <radeontop@noreply.github.com> Cc: Freitag, Chip <chip.freitag@amd.com>; Author <author@noreply.github.com> Subject: Re: [clbr/radeontop] Add card number option for when multiple cards are on same bus (#56) Is it possible for two pcie cards to be on the same bus? Where does this happen? Notes on the patch: * this breaks the short -b option * please no distro-specific instructions in the readme, not everyone uses Ubuntu/Debian * leftover debug printfs, and wrongly indented too * function param should be const * when adding new options, please also update radeontop.asc — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#56 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AE-2BirDGsF83nnkr6aCLp8RdpqJ4pYNks5tNZ99gaJpZM4Roped>.