Your code is correct and should work. You should use latest version of openstreetmap library and modify code accordingly. Latest version is 5.6.5 at the moment.
Quickly crafted example is giving this result (after tap):
I had to comment out this line:
Because I don't have the drawable.
Classes you are using from osmbonuspack are no longer there because they were moved to osmdroid library. You will have to change your imports (e.g. to org.osmdroid.views.overlay.Marker etc). You can remove osmbonuspack dependency completely because code you provided does not need it.
You will have to change a construction of a tile source:
(But please keep in mind there is a usage policy for openstreetmap tiles.)
Final note: Common problem with markers is when you set custom onMarkerClickListener, than the default implementation isn't called. You would need to open marker window from your listener by calling marker.showInfoWindow(). But this is not your problem.