On 02/12/2013 11:30 AM, Rhyland Klein wrote: > On 2/12/2013 11:33 AM, Rhyland Klein wrote: >> On 2/11/2013 6:45 PM, Stephen Warren wrote: >>> On 02/11/2013 03:25 PM, Rhyland Klein wrote: >>>> This patch adds the definition for the cldvfs function for Tegra114 >>>> pinctrl >>>> support. This is based on work by Pritesh Raithatha. >>> Sorry I forgot about this before, but >>> Documentation/devicetree/bindings/pinctrl/nvidia,tegra114-pinmux.txt >>> needs to be updated too. >>> >>> Aside from that, >>> >>> Reviewed-by: Stephen Warren <swarren@xxxxxxxxxx> >> thanks, I'll add that and send new rev. > > After looking, the devicetree binding doc don't seem to list the valid > functions. This change adds a function, and the other groups similar > (blink, cec, ...) don't seem to be listed either. Is there something > else that should be changed in the binding doc that I am missing? Oh right, so they don't. They really should though, so could you please file an internal bug to get the list of function added. If there are groups missing from the list of valid groups, they should be added too. However, all that is unrelated to this patch series, so consider my previous Reviewed-by to apply to the series as-is. LinusW, I assume you will apply patch 1/2 to your pinctrl tree, and I'll apply patch 2/2 to the Tegra tree later. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html