]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "pata_octeon_cf: perform host detach, removal on exit"
authorJeff Garzik <jeff@garzik.org>
Fri, 14 Dec 2012 14:35:51 +0000 (09:35 -0500)
committerJeff Garzik <jgarzik@redhat.com>
Fri, 14 Dec 2012 14:35:51 +0000 (09:35 -0500)
commitdb609a71d5a3ead633932f998aa1ba90d9be13eb
tree669688b545614c0f2fb4edf271dd7d2b6ed2b64d
parent17db41c3f9df84028d8e058b8ba807a2929b71c2
Revert "pata_octeon_cf: perform host detach, removal on exit"

This reverts commit 3ed311f957706f84fb95c768d21172501633473a.

Brian Norris writes:
  > David Daney writes:

  > I can seem to find it.  Without knowing what that does, I would be inclined
  > to NACK the whole thing.

  A NACK is probably the right thing. I was mostly converting a few
  other drivers which used some simple, common patterns to use my new
  common code, but this driver was missing it altogether. It looks like
  there may be bigger issues, though, as you point out.

  > This patch is likely to be incomplete as the driver is also missing the
  > module_exit() things.
  >
  > It might be simpler to just make the driver "bool" instead of "tristate" in
  > the Kconfig.

  As noted earlier, I don't have much interest in this driver. I agree
  that there are some other issues with the driver; I think it leaks
  memory if it is ever allowed to unload, for one. Feel free to submit
  an alternative patch to prevent this driver from being built as a
  module.

Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
drivers/ata/pata_octeon_cf.c