* [PATCH] pinctrl: sun4i: add spdif to pin description. @ 2015-08-11 16:19 codekipper at gmail.com 2015-08-11 16:48 ` [linux-sunxi] " Chen-Yu Tsai 0 siblings, 1 reply; 5+ messages in thread From: codekipper at gmail.com @ 2015-08-11 16:19 UTC (permalink / raw) To: linux-arm-kernel From: Marcus Cooper <codekipper@gmail.com> Signed-off-by: Marcus Cooper <codekipper@gmail.com> --- drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c index 7376a97..daf7dec 100644 --- a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c +++ b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c @@ -135,7 +135,8 @@ static const struct sunxi_desc_pin sun4i_a10_pins[] = { SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 3), SUNXI_FUNCTION(0x0, "gpio_in"), SUNXI_FUNCTION(0x1, "gpio_out"), - SUNXI_FUNCTION(0x2, "ir0")), /* TX */ + SUNXI_FUNCTION(0x2, "ir0"), /* TX */ + SUNXI_FUNCTION(0x4, "spdif")), /* MCLK */ SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 4), SUNXI_FUNCTION(0x0, "gpio_in"), SUNXI_FUNCTION(0x1, "gpio_out"), @@ -176,11 +177,13 @@ static const struct sunxi_desc_pin sun4i_a10_pins[] = { SUNXI_FUNCTION(0x0, "gpio_in"), SUNXI_FUNCTION(0x1, "gpio_out"), SUNXI_FUNCTION(0x2, "i2s"), /* DI */ - SUNXI_FUNCTION(0x3, "ac97")), /* DI */ + SUNXI_FUNCTION(0x3, "ac97"), /* DI */ + SUNXI_FUNCTION(0x4, "spdif")), /* DI */ SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 13), SUNXI_FUNCTION(0x0, "gpio_in"), SUNXI_FUNCTION(0x1, "gpio_out"), - SUNXI_FUNCTION(0x2, "spi2")), /* CS1 */ + SUNXI_FUNCTION(0x2, "spi2"), /* CS1 */ + SUNXI_FUNCTION(0x4, "spdif")), /* DO */ SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 14), SUNXI_FUNCTION(0x0, "gpio_in"), SUNXI_FUNCTION(0x1, "gpio_out"), -- 2.5.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [linux-sunxi] [PATCH] pinctrl: sun4i: add spdif to pin description. 2015-08-11 16:19 [PATCH] pinctrl: sun4i: add spdif to pin description codekipper at gmail.com @ 2015-08-11 16:48 ` Chen-Yu Tsai [not found] ` <CAEKpxBmS6YJMjREc3gmeSQOzBXFKw=Sb-xPK57oUn6BtmASErg@mail.gmail.com> 0 siblings, 1 reply; 5+ messages in thread From: Chen-Yu Tsai @ 2015-08-11 16:48 UTC (permalink / raw) To: linux-arm-kernel On Wed, Aug 12, 2015 at 12:19 AM, <codekipper@gmail.com> wrote: > From: Marcus Cooper <codekipper@gmail.com> > > Signed-off-by: Marcus Cooper <codekipper@gmail.com> > --- > drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c > index 7376a97..daf7dec 100644 > --- a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c > +++ b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c > @@ -135,7 +135,8 @@ static const struct sunxi_desc_pin sun4i_a10_pins[] = { > SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 3), > SUNXI_FUNCTION(0x0, "gpio_in"), > SUNXI_FUNCTION(0x1, "gpio_out"), > - SUNXI_FUNCTION(0x2, "ir0")), /* TX */ > + SUNXI_FUNCTION(0x2, "ir0"), /* TX */ > + SUNXI_FUNCTION(0x4, "spdif")), /* MCLK */ > SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 4), > SUNXI_FUNCTION(0x0, "gpio_in"), > SUNXI_FUNCTION(0x1, "gpio_out"), > @@ -176,11 +177,13 @@ static const struct sunxi_desc_pin sun4i_a10_pins[] = { > SUNXI_FUNCTION(0x0, "gpio_in"), > SUNXI_FUNCTION(0x1, "gpio_out"), > SUNXI_FUNCTION(0x2, "i2s"), /* DI */ > - SUNXI_FUNCTION(0x3, "ac97")), /* DI */ > + SUNXI_FUNCTION(0x3, "ac97"), /* DI */ > + SUNXI_FUNCTION(0x4, "spdif")), /* DI */ > SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 13), > SUNXI_FUNCTION(0x0, "gpio_in"), > SUNXI_FUNCTION(0x1, "gpio_out"), > - SUNXI_FUNCTION(0x2, "spi2")), /* CS1 */ > + SUNXI_FUNCTION(0x2, "spi2"), /* CS1 */ > + SUNXI_FUNCTION(0x4, "spdif")), /* DO */ The datasheet and manual list them as NC or Reserved. Maybe mention how you knew they were available? Given that the S/PDIF block is not documented either, you should do the same for the actual driver, like mention Allwinner kernel sources. ChenYu > SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 14), > SUNXI_FUNCTION(0x0, "gpio_in"), > SUNXI_FUNCTION(0x1, "gpio_out"), > -- > 2.5.0 > > -- > You received this message because you are subscribed to the Google Groups "linux-sunxi" group. > To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe at googlegroups.com. > For more options, visit https://groups.google.com/d/optout. ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <CAEKpxBmS6YJMjREc3gmeSQOzBXFKw=Sb-xPK57oUn6BtmASErg@mail.gmail.com>]
* [linux-sunxi] [PATCH] pinctrl: sun4i: add spdif to pin description. [not found] ` <CAEKpxBmS6YJMjREc3gmeSQOzBXFKw=Sb-xPK57oUn6BtmASErg@mail.gmail.com> @ 2015-08-12 8:34 ` Hans de Goede [not found] ` <CAEKpxB=UQYEnwXYAzMZA4noR9Xg8wnSjD+H0bNO5GV3dtZQtTg@mail.gmail.com> 0 siblings, 1 reply; 5+ messages in thread From: Hans de Goede @ 2015-08-12 8:34 UTC (permalink / raw) To: linux-arm-kernel Hi, On 12-08-15 07:31, Code Kipper wrote: > On 11 August 2015 at 18:48, Chen-Yu Tsai <wens@csie.org> wrote: > >> On Wed, Aug 12, 2015 at 12:19 AM, <codekipper@gmail.com> wrote: >>> From: Marcus Cooper <codekipper@gmail.com> >>> >>> Signed-off-by: Marcus Cooper <codekipper@gmail.com> >>> --- >>> drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c | 9 ++++++--- >>> 1 file changed, 6 insertions(+), 3 deletions(-) >>> >>> diff --git a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >> b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >>> index 7376a97..daf7dec 100644 >>> --- a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >>> +++ b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >>> @@ -135,7 +135,8 @@ static const struct sunxi_desc_pin sun4i_a10_pins[] >> = { >>> SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 3), >>> SUNXI_FUNCTION(0x0, "gpio_in"), >>> SUNXI_FUNCTION(0x1, "gpio_out"), >>> - SUNXI_FUNCTION(0x2, "ir0")), /* TX */ >>> + SUNXI_FUNCTION(0x2, "ir0"), /* TX */ >>> + SUNXI_FUNCTION(0x4, "spdif")), /* MCLK */ >>> SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 4), >>> SUNXI_FUNCTION(0x0, "gpio_in"), >>> SUNXI_FUNCTION(0x1, "gpio_out"), >>> @@ -176,11 +177,13 @@ static const struct sunxi_desc_pin >> sun4i_a10_pins[] = { >>> SUNXI_FUNCTION(0x0, "gpio_in"), >>> SUNXI_FUNCTION(0x1, "gpio_out"), >>> SUNXI_FUNCTION(0x2, "i2s"), /* DI */ >>> - SUNXI_FUNCTION(0x3, "ac97")), /* DI */ >>> + SUNXI_FUNCTION(0x3, "ac97"), /* DI */ >>> + SUNXI_FUNCTION(0x4, "spdif")), /* DI */ >>> SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 13), >>> SUNXI_FUNCTION(0x0, "gpio_in"), >>> SUNXI_FUNCTION(0x1, "gpio_out"), >>> - SUNXI_FUNCTION(0x2, "spi2")), /* CS1 */ >>> + SUNXI_FUNCTION(0x2, "spi2"), /* CS1 */ >>> + SUNXI_FUNCTION(0x4, "spdif")), /* DO */ >> >> The datasheet and manual list them as NC or Reserved. Maybe mention how >> you knew >> they were available? >> > Not sure if this deserves to be in the commit message but I can definitely > put together a covering letter with links etc. I dear to say it belongs right there in the source code, use a multi-line comment above the SUNXI_FUNCTION(0x4, "spdif")) to explain where the mux info comes from. someday sooner or later someone is going to compare the kernel mux table to the datasheet and think "huh, that is not right", that person will be saved an immense amount of time by simply having a comment there. And do NOT say that this will not happen, because I've already done such a comparison once in the past. Regards, Hans ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <CAEKpxB=UQYEnwXYAzMZA4noR9Xg8wnSjD+H0bNO5GV3dtZQtTg@mail.gmail.com>]
* [linux-sunxi] [PATCH] pinctrl: sun4i: add spdif to pin description. [not found] ` <CAEKpxB=UQYEnwXYAzMZA4noR9Xg8wnSjD+H0bNO5GV3dtZQtTg@mail.gmail.com> @ 2015-08-12 9:18 ` Chen-Yu Tsai 2015-08-12 10:18 ` Hans de Goede 0 siblings, 1 reply; 5+ messages in thread From: Chen-Yu Tsai @ 2015-08-12 9:18 UTC (permalink / raw) To: linux-arm-kernel On Wed, Aug 12, 2015 at 5:13 PM, Code Kipper <codekipper@gmail.com> wrote: > On 12 August 2015 at 10:34, Hans de Goede <hdegoede@redhat.com> wrote: >> >> Hi, >> >> >> On 12-08-15 07:31, Code Kipper wrote: >>> >>> On 11 August 2015 at 18:48, Chen-Yu Tsai <wens@csie.org> wrote: >>> >>>> On Wed, Aug 12, 2015 at 12:19 AM, <codekipper@gmail.com> wrote: >>>>> >>>>> From: Marcus Cooper <codekipper@gmail.com> >>>>> >>>>> Signed-off-by: Marcus Cooper <codekipper@gmail.com> >>>>> --- >>>>> drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c | 9 ++++++--- >>>>> 1 file changed, 6 insertions(+), 3 deletions(-) >>>>> >>>>> diff --git a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >>>> >>>> b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >>>>> >>>>> index 7376a97..daf7dec 100644 >>>>> --- a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >>>>> +++ b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >>>>> @@ -135,7 +135,8 @@ static const struct sunxi_desc_pin sun4i_a10_pins[] >>>> >>>> = { >>>>> >>>>> SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 3), >>>>> SUNXI_FUNCTION(0x0, "gpio_in"), >>>>> SUNXI_FUNCTION(0x1, "gpio_out"), >>>>> - SUNXI_FUNCTION(0x2, "ir0")), /* TX */ >>>>> + SUNXI_FUNCTION(0x2, "ir0"), /* TX */ >>>>> + SUNXI_FUNCTION(0x4, "spdif")), /* MCLK */ >>>>> SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 4), >>>>> SUNXI_FUNCTION(0x0, "gpio_in"), >>>>> SUNXI_FUNCTION(0x1, "gpio_out"), >>>>> @@ -176,11 +177,13 @@ static const struct sunxi_desc_pin >>>> >>>> sun4i_a10_pins[] = { >>>>> >>>>> SUNXI_FUNCTION(0x0, "gpio_in"), >>>>> SUNXI_FUNCTION(0x1, "gpio_out"), >>>>> SUNXI_FUNCTION(0x2, "i2s"), /* DI */ >>>>> - SUNXI_FUNCTION(0x3, "ac97")), /* DI */ >>>>> + SUNXI_FUNCTION(0x3, "ac97"), /* DI */ >>>>> + SUNXI_FUNCTION(0x4, "spdif")), /* DI */ >>>>> SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 13), >>>>> SUNXI_FUNCTION(0x0, "gpio_in"), >>>>> SUNXI_FUNCTION(0x1, "gpio_out"), >>>>> - SUNXI_FUNCTION(0x2, "spi2")), /* CS1 */ >>>>> + SUNXI_FUNCTION(0x2, "spi2"), /* CS1 */ >>>>> + SUNXI_FUNCTION(0x4, "spdif")), /* DO */ >>>> >>>> >>>> The datasheet and manual list them as NC or Reserved. Maybe mention how >>>> you knew >>>> they were available? >>>> >>> Not sure if this deserves to be in the commit message but I can >>> definitely >>> put together a covering letter with links etc. >> >> >> I dear to say it belongs right there in the source code, use a multi-line >> comment above the SUNXI_FUNCTION(0x4, "spdif")) to explain where the mux >> info comes from. someday sooner or later someone is going to compare the >> kernel mux table to the datasheet and think "huh, that is not right", >> that person will be saved an immense amount of time by simply having a >> comment there. > > Good point I was thinking something along the lines of this > /* > * The SPDIF block is not referenced at all in the A10 user > * manual. However it is described in the code leaked and the > * pin descriptions are declared in the A20 user manual which > * is pin compatible with this device. > */ > Thing is do I add this above every spdif pin, the first change(MCLK) or the > pin most likely to be used(SPDIF DO)? I'd say put it before the first pin, and then reference it (like "undocumented, see the comment for pin P??? above", for the rest. ChenYu > Thanks, > CK >> >> >> And do NOT say that this will not happen, because I've already done >> such a comparison once in the past. >> >> Regards, >> >> Hans > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* [linux-sunxi] [PATCH] pinctrl: sun4i: add spdif to pin description. 2015-08-12 9:18 ` Chen-Yu Tsai @ 2015-08-12 10:18 ` Hans de Goede 0 siblings, 0 replies; 5+ messages in thread From: Hans de Goede @ 2015-08-12 10:18 UTC (permalink / raw) To: linux-arm-kernel Hi, On 12-08-15 11:18, Chen-Yu Tsai wrote: > On Wed, Aug 12, 2015 at 5:13 PM, Code Kipper <codekipper@gmail.com> wrote: >> On 12 August 2015 at 10:34, Hans de Goede <hdegoede@redhat.com> wrote: >>> >>> Hi, >>> >>> >>> On 12-08-15 07:31, Code Kipper wrote: >>>> >>>> On 11 August 2015 at 18:48, Chen-Yu Tsai <wens@csie.org> wrote: >>>> >>>>> On Wed, Aug 12, 2015 at 12:19 AM, <codekipper@gmail.com> wrote: >>>>>> >>>>>> From: Marcus Cooper <codekipper@gmail.com> >>>>>> >>>>>> Signed-off-by: Marcus Cooper <codekipper@gmail.com> >>>>>> --- >>>>>> drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c | 9 ++++++--- >>>>>> 1 file changed, 6 insertions(+), 3 deletions(-) >>>>>> >>>>>> diff --git a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >>>>> >>>>> b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >>>>>> >>>>>> index 7376a97..daf7dec 100644 >>>>>> --- a/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >>>>>> +++ b/drivers/pinctrl/sunxi/pinctrl-sun4i-a10.c >>>>>> @@ -135,7 +135,8 @@ static const struct sunxi_desc_pin sun4i_a10_pins[] >>>>> >>>>> = { >>>>>> >>>>>> SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 3), >>>>>> SUNXI_FUNCTION(0x0, "gpio_in"), >>>>>> SUNXI_FUNCTION(0x1, "gpio_out"), >>>>>> - SUNXI_FUNCTION(0x2, "ir0")), /* TX */ >>>>>> + SUNXI_FUNCTION(0x2, "ir0"), /* TX */ >>>>>> + SUNXI_FUNCTION(0x4, "spdif")), /* MCLK */ >>>>>> SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 4), >>>>>> SUNXI_FUNCTION(0x0, "gpio_in"), >>>>>> SUNXI_FUNCTION(0x1, "gpio_out"), >>>>>> @@ -176,11 +177,13 @@ static const struct sunxi_desc_pin >>>>> >>>>> sun4i_a10_pins[] = { >>>>>> >>>>>> SUNXI_FUNCTION(0x0, "gpio_in"), >>>>>> SUNXI_FUNCTION(0x1, "gpio_out"), >>>>>> SUNXI_FUNCTION(0x2, "i2s"), /* DI */ >>>>>> - SUNXI_FUNCTION(0x3, "ac97")), /* DI */ >>>>>> + SUNXI_FUNCTION(0x3, "ac97"), /* DI */ >>>>>> + SUNXI_FUNCTION(0x4, "spdif")), /* DI */ >>>>>> SUNXI_PIN(SUNXI_PINCTRL_PIN(B, 13), >>>>>> SUNXI_FUNCTION(0x0, "gpio_in"), >>>>>> SUNXI_FUNCTION(0x1, "gpio_out"), >>>>>> - SUNXI_FUNCTION(0x2, "spi2")), /* CS1 */ >>>>>> + SUNXI_FUNCTION(0x2, "spi2"), /* CS1 */ >>>>>> + SUNXI_FUNCTION(0x4, "spdif")), /* DO */ >>>>> >>>>> >>>>> The datasheet and manual list them as NC or Reserved. Maybe mention how >>>>> you knew >>>>> they were available? >>>>> >>>> Not sure if this deserves to be in the commit message but I can >>>> definitely >>>> put together a covering letter with links etc. >>> >>> >>> I dear to say it belongs right there in the source code, use a multi-line >>> comment above the SUNXI_FUNCTION(0x4, "spdif")) to explain where the mux >>> info comes from. someday sooner or later someone is going to compare the >>> kernel mux table to the datasheet and think "huh, that is not right", >>> that person will be saved an immense amount of time by simply having a >>> comment there. >> >> Good point I was thinking something along the lines of this >> /* >> * The SPDIF block is not referenced at all in the A10 user >> * manual. However it is described in the code leaked and the >> * pin descriptions are declared in the A20 user manual which >> * is pin compatible with this device. >> */ >> Thing is do I add this above every spdif pin, the first change(MCLK) or the >> pin most likely to be used(SPDIF DO)? > > I'd say put it before the first pin, and then reference it (like > "undocumented, see the comment for pin P??? above", for the rest. Ack / +1 exactly what I was thinking :) Regards, Hans > > ChenYu > >> Thanks, >> CK >>> >>> >>> And do NOT say that this will not happen, because I've already done >>> such a comparison once in the past. >>> >>> Regards, >>> >>> Hans >> >> ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-08-12 10:18 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-11 16:19 [PATCH] pinctrl: sun4i: add spdif to pin description codekipper at gmail.com
2015-08-11 16:48 ` [linux-sunxi] " Chen-Yu Tsai
[not found] ` <CAEKpxBmS6YJMjREc3gmeSQOzBXFKw=Sb-xPK57oUn6BtmASErg@mail.gmail.com>
2015-08-12 8:34 ` Hans de Goede
[not found] ` <CAEKpxB=UQYEnwXYAzMZA4noR9Xg8wnSjD+H0bNO5GV3dtZQtTg@mail.gmail.com>
2015-08-12 9:18 ` Chen-Yu Tsai
2015-08-12 10:18 ` Hans de Goede
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).