Don't hold the lock before unregistering the device, since when the
device is unregistered the datastruct containing the lock may be freed
(if the refcount went to 0).

Also fixed the framework documentation that erroneously suggested the
wrong locking order as well.
Reported-by: NDavid Ellingsworth <david@identd.dyndns.org>
Signed-off-by: NHans Verkuil <hverkuil@xs4all.nl>
Acked-by: NDavid Ellingsworth <david@identd.dyndns.org>
Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>