diff --git a/lib/db_connection.ex b/lib/db_connection.ex
index f9585d7..55d683b 100644
--- a/lib/db_connection.ex
+++ b/lib/db_connection.ex
@@ -171,7 +171,7 @@ defmodule DBConnection do
   to allow the checkout or `{:disconnect, exception, state}` to disconnect.
 
   This callback is called immediately after the connection is established
-  and the state is never effetively checked in again. That's because
+  and the state is never effectively checked in again. That's because
   DBConnection keeps the connection state in an ETS table that is moved
   between the different clients checking out connections. There is no
   `checkin` callback. The state is only handed back to the connection
@@ -1274,13 +1274,13 @@ defmodule DBConnection do
   end
 
   @doc """
-  Returns connection module used by the given connection pool.
+  Returns the connection module used by the given connection pool.
 
   When given a process that is not a connection pool, returns an `:error`.
   """
   @spec connection_module(conn) :: {:ok, module} | :error
   def connection_module(conn) do
-    with pid when pid != nil <- pool_pid(conn),
+    with pid when is_pid(pid) <- pool_pid(conn),
          {:dictionary, dictionary} <- Process.info(pid, :dictionary),
          {@connection_module_key, module} <- List.keyfind(dictionary, @connection_module_key, 0),
          do: {:ok, module},
diff --git a/lib/db_connection/ownership.ex b/lib/db_connection/ownership.ex
index 26b52f2..f61a379 100644
--- a/lib/db_connection/ownership.ex
+++ b/lib/db_connection/ownership.ex
@@ -94,8 +94,7 @@ defmodule DBConnection.Ownership do
 
   It may return `:ok` if the connection is checked out.
   `{:already, :owner | :allowed}` if the caller process already
-  has a connection, `:error` if it could be not checked out or
-  raise if there was an error.
+  has a connection, or raise if there was an error.
   """
   @spec ownership_checkout(GenServer.server(), Keyword.t()) ::
           :ok | {:already, :owner | :allowed}