Allow frame instances to be passed to SkyCoord

@eteq @taldcroft - this is related to https://github.com/astropy/astropy/issues/2703

Basically at the moment one cannot pass frame instances to the ``frame=`` keyword argument of ``SkyCoord`` - this is not ideal because sometimes one might get a frame programmatically with all attribute sets, and decomposing it to pass to SkyCoord is a pain. With the following PR, ``frame=`` can be set to a frame instance, if and only if frame arguments are not also passed in the ``SkyCoord`` kwargs (to avoid conflict).

This is backward-compatible and also avoids the issue of having to merge frame attributes passed in the instance vs as keyword arguments.

cc @Cadair
Somewhat out of scope for this PR, but should we update the docs for the "best practice" of using `frame=..` when supplying a frame which is not providing the coordinate data?