I don't agree with unexporting pc8736x_gpio_ops. If anything the struct should be made global and kept exported (as _GPL perhaps). This lets other modules use the pc8736x_gpio module in a semi-independent way. See my comments about the scx200_gpio changes you submitted.