Re: [PATCH] Documentation: Add examples to samsung-pinctrl device tree bindings documentation
Hi Tomasz, Thanks for reviewing the patch. Will post version 2 soon addressing your comments. Best Wishes, Leela Krishna. On Mon, Jun 17, 2013 at 5:40 PM, Tomasz Figa wrote: > Hi Leela, > > On Monday 17 of June 2013 16:14:55 Leela Krishna Amudala wrote: >> This patch adds examples to samsung-pinctrl.txt documentaion file >> on how to make gpio binding and gpio request >> >> Signed-off-by: Leela Krishna Amudala >> Reviewed-by: Doug Anderson >> --- >> .../bindings/pinctrl/samsung-pinctrl.txt | 48 >> +++- 1 file changed, 46 insertions(+), 2 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt >> b/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt index >> b2bc219..f1cf177 100644 >> --- a/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt >> +++ b/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt >> @@ -25,8 +25,18 @@ Required Properties: >> >>- gpio-controller: identifies the node as a gpio controller and pin bank. >> - #gpio-cells: number of cells in GPIO specifier. Since the generic GPIO - >> binding is used, the amount of cells must be specified as 2. See generic >> -GPIO binding documentation for description of particular cells. + >> binding is used, the amount of cells must be specified as 2. See the below >> +mentioned gpio binding representation for description of particular >> cells. + >> + Eg: <&gpx2 6 0> >> + <[phandle of the gpio controller node] >> + [pin number within the gpio controller] >> + [flags]> >> + >> + Values for gpio specifier: >> + - Pin number: is a value between 0 to 7. >> + - Flags: 0 - Active High >> + 1 - Active Low >> >> - Pin mux/config groups as child nodes: The pin mux (selecting pin function >> mode) and pin config (pull up/down, driver strength) settings are >> represented @@ -274,3 +284,37 @@ Example 4: Set up the default pin state >> for uart controller. >> >> pinctrl = devm_pinctrl_get_select_default(&pdev->dev); >> } >> + >> +Example 5: A display port client node that supports 'default' pinctrl state >> +and gpio binding. >> + >> + display-port-controller { >> + samsung,color-space = <0>; >> + samsung,dynamic-range = <0>; >> + samsung,ycbcr-coeff = <0>; >> + samsung,color-depth = <1>; >> + samsung,link-rate = <0x06>; >> + samsung,lane-count = <2>; > > Maybe it would be better to omit those properties above for readability of the > example? Just like you did with example 6. > > But generally it's a good addition to our documentation. Thanks. > > Best regards, > Tomasz > >> + samsung,hpd-gpio = <&gpx2 6 0>; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&dp_hpd>; >> + }; >> + >> +Example 6: Request the gpio for display port controller >> + >> + static int exynos_dp_probe(struct platform_device *pdev) >> + { >> + int hpd_gpio, ret; >> + struct device *dev = &pdev->dev; >> + struct device_node *dp_node = dev->of_node; >> + >> + /* ... */ >> + >> + hpd_gpio = of_get_named_gpio(dp_node, "samsung,hpd-gpio", 0); >> + >> + /* ... */ >> + >> + ret = devm_gpio_request_one(&pdev->dev, hpd_gpio, GPIOF_IN, >> + "hpd_gpio"); >> + /* ... */ >> + } > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" > in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] Documentation: Add examples to samsung-pinctrl device tree bindings documentation
Hi Leela, On Monday 17 of June 2013 16:14:55 Leela Krishna Amudala wrote: > This patch adds examples to samsung-pinctrl.txt documentaion file > on how to make gpio binding and gpio request > > Signed-off-by: Leela Krishna Amudala > Reviewed-by: Doug Anderson > --- > .../bindings/pinctrl/samsung-pinctrl.txt | 48 > +++- 1 file changed, 46 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt > b/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt index > b2bc219..f1cf177 100644 > --- a/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt > +++ b/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt > @@ -25,8 +25,18 @@ Required Properties: > >- gpio-controller: identifies the node as a gpio controller and pin bank. > - #gpio-cells: number of cells in GPIO specifier. Since the generic GPIO - > binding is used, the amount of cells must be specified as 2. See generic > -GPIO binding documentation for description of particular cells. + > binding is used, the amount of cells must be specified as 2. See the below > +mentioned gpio binding representation for description of particular > cells. + > + Eg: <&gpx2 6 0> > + <[phandle of the gpio controller node] > + [pin number within the gpio controller] > + [flags]> > + > + Values for gpio specifier: > + - Pin number: is a value between 0 to 7. > + - Flags: 0 - Active High > + 1 - Active Low > > - Pin mux/config groups as child nodes: The pin mux (selecting pin function > mode) and pin config (pull up/down, driver strength) settings are > represented @@ -274,3 +284,37 @@ Example 4: Set up the default pin state > for uart controller. > > pinctrl = devm_pinctrl_get_select_default(&pdev->dev); > } > + > +Example 5: A display port client node that supports 'default' pinctrl state > +and gpio binding. > + > + display-port-controller { > + samsung,color-space = <0>; > + samsung,dynamic-range = <0>; > + samsung,ycbcr-coeff = <0>; > + samsung,color-depth = <1>; > + samsung,link-rate = <0x06>; > + samsung,lane-count = <2>; Maybe it would be better to omit those properties above for readability of the example? Just like you did with example 6. But generally it's a good addition to our documentation. Thanks. Best regards, Tomasz > + samsung,hpd-gpio = <&gpx2 6 0>; > + pinctrl-names = "default"; > + pinctrl-0 = <&dp_hpd>; > + }; > + > +Example 6: Request the gpio for display port controller > + > + static int exynos_dp_probe(struct platform_device *pdev) > + { > + int hpd_gpio, ret; > + struct device *dev = &pdev->dev; > + struct device_node *dp_node = dev->of_node; > + > + /* ... */ > + > + hpd_gpio = of_get_named_gpio(dp_node, "samsung,hpd-gpio", 0); > + > + /* ... */ > + > + ret = devm_gpio_request_one(&pdev->dev, hpd_gpio, GPIOF_IN, > + "hpd_gpio"); > + /* ... */ > + } -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH] Documentation: Add examples to samsung-pinctrl device tree bindings documentation
Kukjin Kim wrote: > > Leela Krishna Amudala wrote: > > > > This patch adds examples to samsung-pinctrl.txt documentaion file > > on how to make gpio binding and gpio request > > > > Signed-off-by: Leela Krishna Amudala > > Reviewed-by: Doug Anderson > > --- > > .../bindings/pinctrl/samsung-pinctrl.txt | 48 > > +++- > > 1 file changed, 46 insertions(+), 2 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/pinctrl/samsung- > pinctrl.txt > > b/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt > > index b2bc219..f1cf177 100644 > > --- a/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt > > +++ b/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt > > @@ -25,8 +25,18 @@ Required Properties: > > > >- gpio-controller: identifies the node as a gpio controller and pin > > bank. > >- #gpio-cells: number of cells in GPIO specifier. Since the generic > > GPIO > > -binding is used, the amount of cells must be specified as 2. See > > generic > > -GPIO binding documentation for description of particular cells. > > +binding is used, the amount of cells must be specified as 2. See the > > below > > +mentioned gpio binding representation for description of particular > > cells. > > + > > + Eg: <&gpx2 6 0> > > + <[phandle of the gpio controller node] > > + [pin number within the gpio controller] > > + [flags]> > > + > > + Values for gpio specifier: > > + - Pin number: is a value between 0 to 7. > > + - Flags: 0 - Active High > > +1 - Active Low > > > > - Pin mux/config groups as child nodes: The pin mux (selecting pin > > function > >mode) and pin config (pull up/down, driver strength) settings are > > represented > > @@ -274,3 +284,37 @@ Example 4: Set up the default pin state for uart > > controller. > > > > pinctrl = devm_pinctrl_get_select_default(&pdev->dev); > > } > > + > > +Example 5: A display port client node that supports 'default' pinctrl > > state > > + and gpio binding. > > + > > + display-port-controller { > > + samsung,color-space = <0>; > > + samsung,dynamic-range = <0>; > > + samsung,ycbcr-coeff = <0>; > > + samsung,color-depth = <1>; > > + samsung,link-rate = <0x06>; > > + samsung,lane-count = <2>; > > + samsung,hpd-gpio = <&gpx2 6 0>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&dp_hpd>; > > + }; > > + > > +Example 6: Request the gpio for display port controller > > + > > + static int exynos_dp_probe(struct platform_device *pdev) > > + { > > + int hpd_gpio, ret; > > + struct device *dev = &pdev->dev; > > + struct device_node *dp_node = dev->of_node; > > + > > + /* ... */ > > + > > + hpd_gpio = of_get_named_gpio(dp_node, "samsung,hpd-gpio", 0); > > + > > + /* ... */ > > + > > + ret = devm_gpio_request_one(&pdev->dev, hpd_gpio, GPIOF_IN, > > + "hpd_gpio"); > > + /* ... */ > > + } > > -- > > 1.7.9.5 > > Looks good to me, applied. > (+ Linus Walleij) Hmm, probably I need Linus' ack on this ;-) - Kukjin -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH] Documentation: Add examples to samsung-pinctrl device tree bindings documentation
Leela Krishna Amudala wrote: > > This patch adds examples to samsung-pinctrl.txt documentaion file > on how to make gpio binding and gpio request > > Signed-off-by: Leela Krishna Amudala > Reviewed-by: Doug Anderson > --- > .../bindings/pinctrl/samsung-pinctrl.txt | 48 > +++- > 1 file changed, 46 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt > b/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt > index b2bc219..f1cf177 100644 > --- a/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt > +++ b/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt > @@ -25,8 +25,18 @@ Required Properties: > >- gpio-controller: identifies the node as a gpio controller and pin > bank. >- #gpio-cells: number of cells in GPIO specifier. Since the generic > GPIO > -binding is used, the amount of cells must be specified as 2. See > generic > -GPIO binding documentation for description of particular cells. > +binding is used, the amount of cells must be specified as 2. See the > below > +mentioned gpio binding representation for description of particular > cells. > + > + Eg: <&gpx2 6 0> > + <[phandle of the gpio controller node] > + [pin number within the gpio controller] > + [flags]> > + > + Values for gpio specifier: > + - Pin number: is a value between 0 to 7. > + - Flags: 0 - Active High > + 1 - Active Low > > - Pin mux/config groups as child nodes: The pin mux (selecting pin > function >mode) and pin config (pull up/down, driver strength) settings are > represented > @@ -274,3 +284,37 @@ Example 4: Set up the default pin state for uart > controller. > > pinctrl = devm_pinctrl_get_select_default(&pdev->dev); > } > + > +Example 5: A display port client node that supports 'default' pinctrl > state > +and gpio binding. > + > + display-port-controller { > + samsung,color-space = <0>; > + samsung,dynamic-range = <0>; > + samsung,ycbcr-coeff = <0>; > + samsung,color-depth = <1>; > + samsung,link-rate = <0x06>; > + samsung,lane-count = <2>; > + samsung,hpd-gpio = <&gpx2 6 0>; > + pinctrl-names = "default"; > + pinctrl-0 = <&dp_hpd>; > + }; > + > +Example 6: Request the gpio for display port controller > + > + static int exynos_dp_probe(struct platform_device *pdev) > + { > + int hpd_gpio, ret; > + struct device *dev = &pdev->dev; > + struct device_node *dp_node = dev->of_node; > + > + /* ... */ > + > + hpd_gpio = of_get_named_gpio(dp_node, "samsung,hpd-gpio", 0); > + > + /* ... */ > + > + ret = devm_gpio_request_one(&pdev->dev, hpd_gpio, GPIOF_IN, > + "hpd_gpio"); > + /* ... */ > + } > -- > 1.7.9.5 Looks good to me, applied. Thanks, - Kukjin -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] Documentation: Add examples to samsung-pinctrl device tree bindings documentation
This patch adds examples to samsung-pinctrl.txt documentaion file on how to make gpio binding and gpio request Signed-off-by: Leela Krishna Amudala Reviewed-by: Doug Anderson --- .../bindings/pinctrl/samsung-pinctrl.txt | 48 +++- 1 file changed, 46 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt b/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt index b2bc219..f1cf177 100644 --- a/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt +++ b/Documentation/devicetree/bindings/pinctrl/samsung-pinctrl.txt @@ -25,8 +25,18 @@ Required Properties: - gpio-controller: identifies the node as a gpio controller and pin bank. - #gpio-cells: number of cells in GPIO specifier. Since the generic GPIO -binding is used, the amount of cells must be specified as 2. See generic -GPIO binding documentation for description of particular cells. +binding is used, the amount of cells must be specified as 2. See the below +mentioned gpio binding representation for description of particular cells. + + Eg: <&gpx2 6 0> + <[phandle of the gpio controller node] + [pin number within the gpio controller] + [flags]> + + Values for gpio specifier: + - Pin number: is a value between 0 to 7. + - Flags: 0 - Active High +1 - Active Low - Pin mux/config groups as child nodes: The pin mux (selecting pin function mode) and pin config (pull up/down, driver strength) settings are represented @@ -274,3 +284,37 @@ Example 4: Set up the default pin state for uart controller. pinctrl = devm_pinctrl_get_select_default(&pdev->dev); } + +Example 5: A display port client node that supports 'default' pinctrl state + and gpio binding. + + display-port-controller { + samsung,color-space = <0>; + samsung,dynamic-range = <0>; + samsung,ycbcr-coeff = <0>; + samsung,color-depth = <1>; + samsung,link-rate = <0x06>; + samsung,lane-count = <2>; + samsung,hpd-gpio = <&gpx2 6 0>; + pinctrl-names = "default"; + pinctrl-0 = <&dp_hpd>; + }; + +Example 6: Request the gpio for display port controller + + static int exynos_dp_probe(struct platform_device *pdev) + { + int hpd_gpio, ret; + struct device *dev = &pdev->dev; + struct device_node *dp_node = dev->of_node; + + /* ... */ + + hpd_gpio = of_get_named_gpio(dp_node, "samsung,hpd-gpio", 0); + + /* ... */ + + ret = devm_gpio_request_one(&pdev->dev, hpd_gpio, GPIOF_IN, + "hpd_gpio"); + /* ... */ + } -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html