Always call set_maple_port(), even when maple port isn't in config

[Tails86]

This is regarding discussion #1825

Problem: When no config is present, then a DreamConn connection isn't being made.

DreamConnGamepad::set_maple_port() needs to be called in order to establish connection to the device. It should always be called in GamepadDevice::Register(), even when cfgLoadInt() returns the default result. This is essentially just setting it to the same defaulted index it was already set to if it isn't in the config.

[kosekmi]

Fix confirmed

[flyinghead]

Can you do this call in the DreamConnGamepad constructor instead?
Like here.
set_maple_port() is already called by the SDLGamepad constructor but you can call it again if you need to.
No big deal if it's not possible/easy but better to keep device-specific stuff in their own source.

[flyinghead]

actually you certainly want dreamcastControllerType to be correctly set before so here is a better place.

[Tails86]

Can you do this call in the DreamConnGamepad constructor instead? Like here. set_maple_port() is already called by the SDLGamepad constructor but you can call it again if you need to. No big deal if it's not possible/easy but better to keep device-specific stuff in their own source.

I think it would make sense to prevent multiple connect() calls, so I would feel more comfortable adding something like virtual void registered(); to GamepadDevice so the DreamConnGamepad can react to registration and only do connection attempts once registered in set_maple_port(). Does that sound like an ok compromise?