Just a minor comment
On Tue, 2010-03-09 at 13:01 -0700, Jason Gunthorpe wrote:
> @@ -703,7 +747,17 @@ static int __init init_tis(void)
> return rc;
> }
>
> - return pnp_register_driver(&tis_pnp_driver);
> +#ifdef CONFIG_OF
> + rc = of_register_platform_driver(&tis_of_
On Thu, Mar 25, 2010 at 01:20:41AM -0300, Rajiv Andrade wrote:
> Why not
>
> +#ifdef CONFIG_OF
> + return of_register_platform_driver(&tis_of_driver);
> +#elif CONFIG_PNP
> + return pnp_register_driver(&tis_pnp_driver);
> +#endif
>
> ?
AFAIK they are not exclusive options? I can't imag
