Marc Dietrich wrote at Wednesday, October 26, 2011 1:59 PM:
> This adds device tree support to the nvec driver. By using this method
> it is no longer necessary to specify platform data through a board
> file.
You should document the binding in Documentation/devicetree/bindings.
> @@ -892,6 +915,17 @@ static int tegra_nvec_resume(struct platform_device *pdev)
> #define tegra_nvec_resume NULL
> #endif
>
> +#if defined(CONFIG_OF)
I think you can just remove the ifdef and always include this code. Yes, it'll
result in slightly more rodata when !CONFIG_OF, but !CONFIG_OF isn't going to
exist or be useful for Tegra for that much longer.
> +/* Match table for of_platform binding */
> +static const struct of_device_id nvidia_nvec_of_match[] __devinitconst = {
> + { .compatible = "nvidia,nvec", },
I'm not sure that nvidia,nvec is the right value, but need a little more
background.
It's my understanding that how this works is a little micro-controller
exists on the board, handles various devices like the keyboard, and sends
data to Tegra by making I2C master transactions. Isn't it the case that
the micro-controller (or at least the SW running on it) is board-specific,
and the same for the I2C protocol? If so, nvidia,nvec is a little generic;
we probably need to name it compal,paz00-ec or something like that?
Either way, we should probably include some kind of version number in
the compatible property so we can support upgrades to the protocol if
needed.
--
nvpublic
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
