From: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
To: Masahiro Yamada <yamada.masahiro@socionext.com>
Cc: Linus Walleij <linus.walleij@linaro.org>,
"open list:GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Masami Hiramatsu <masami.hiramatsu@linaro.org>,
Jassi Brar <jaswinder.singh@linaro.org>
Subject: Re: [PATCH 2/5] pinctrl: uniphier: Add another audio I/O pin-mux settings for LD20
Date: Tue, 30 Jul 2019 11:22:05 +0900 [thread overview]
Message-ID: <20190730112205.F397.4A936039@socionext.com> (raw)
In-Reply-To: <CAK7LNASuNMij8Fttup6T6hd=vyKSEu=B7HCPMAezWK6T2b0Gfg@mail.gmail.com>
Hello,
On Mon, 29 Jul 2019 22:44:27 +0900 <yamada.masahiro@socionext.com> wrote:
> On Tue, Jul 9, 2019 at 7:29 PM Kunihiko Hayashi
> <hayashi.kunihiko@socionext.com> wrote:
> >
> > This adds support for pinmux settings of aout1b groups. This group includes
> > aout1 signals derived from xirq pins.
> >
> > Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
> > ---
> > drivers/pinctrl/uniphier/pinctrl-uniphier-ld20.c | 5 +++++
> > 1 file changed, 5 insertions(+)
> >
> > diff --git a/drivers/pinctrl/uniphier/pinctrl-uniphier-ld20.c b/drivers/pinctrl/uniphier/pinctrl-uniphier-ld20.c
> > index 28e54b3..2c66e70 100644
> > --- a/drivers/pinctrl/uniphier/pinctrl-uniphier-ld20.c
> > +++ b/drivers/pinctrl/uniphier/pinctrl-uniphier-ld20.c
> > @@ -544,6 +544,8 @@ static const struct pinctrl_pin_desc uniphier_ld20_pins[] = {
> >
> > static const unsigned aout1_pins[] = {137, 138, 139, 140, 141, 142};
> > static const int aout1_muxvals[] = {0, 0, 0, 0, 0, 0};
> > +static const unsigned aout1b_pins[] = {150, 151, 152, 153, 154, 155, 156};
> > +static const int aout1b_muxvals[] = {1, 1, 1, 1, 1, 1, 1};
> > static const unsigned aoutiec1_pins[] = {135, 136};
> > static const int aoutiec1_muxvals[] = {0, 0};
> > static const unsigned int emmc_pins[] = {19, 20, 21, 22, 23, 24, 25};
> > @@ -664,6 +666,7 @@ static const unsigned int gpio_range2_pins[] = {
> >
> > static const struct uniphier_pinctrl_group uniphier_ld20_groups[] = {
> > UNIPHIER_PINCTRL_GROUP(aout1),
> > + UNIPHIER_PINCTRL_GROUP(aout1b),
> > UNIPHIER_PINCTRL_GROUP(aoutiec1),
> > UNIPHIER_PINCTRL_GROUP(emmc),
> > UNIPHIER_PINCTRL_GROUP(emmc_dat8),
> > @@ -708,6 +711,7 @@ static const struct uniphier_pinctrl_group uniphier_ld20_groups[] = {
> > };
> >
> > static const char * const aout1_groups[] = {"aout1"};
> > +static const char * const aout1b_groups[] = {"aout1b"};
>
> If this has the same functionality as "aout1",
> shouldn't it be a part of aout1_groups?
Indeed.
I'll merge "aout1b" with aout1_groups.
>
>
>
>
> --
> Best Regards
> Masahiro Yamada
---
Best Regards,
Kunihiko Hayashi
next prev parent reply other threads:[~2019-07-30 2:22 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-09 10:29 [PATCH 0/5] pinctrl: uniphier: Add some improvements and new settings Kunihiko Hayashi
2019-07-09 10:29 ` [PATCH 1/5] pinctrl: uniphier: Separate modem group from UART ctsrts group Kunihiko Hayashi
2019-07-29 13:45 ` Masahiro Yamada
2019-07-30 2:21 ` Kunihiko Hayashi
2019-07-09 10:29 ` [PATCH 2/5] pinctrl: uniphier: Add another audio I/O pin-mux settings for LD20 Kunihiko Hayashi
2019-07-29 13:44 ` Masahiro Yamada
2019-07-30 2:22 ` Kunihiko Hayashi [this message]
2019-07-09 10:29 ` [PATCH 3/5] pinctrl: uniphier: Add 4th LD20 MPEG2-TS input pin-mux setting Kunihiko Hayashi
2019-07-29 13:45 ` Masahiro Yamada
2019-07-09 10:29 ` [PATCH 4/5] pinctrl: uniphier: Add Pro5 PCIe pin-mux settings Kunihiko Hayashi
2019-07-29 13:46 ` Masahiro Yamada
2019-07-30 2:22 ` Kunihiko Hayashi
2019-07-09 10:29 ` [PATCH 5/5] pinctrl: uniphier: Fix Pro5 SD pin-mux setting Kunihiko Hayashi
2019-07-29 13:46 ` Masahiro Yamada
2019-07-28 23:07 ` [PATCH 0/5] pinctrl: uniphier: Add some improvements and new settings Linus Walleij
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190730112205.F397.4A936039@socionext.com \
--to=hayashi.kunihiko@socionext.com \
--cc=jaswinder.singh@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=masami.hiramatsu@linaro.org \
--cc=yamada.masahiro@socionext.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).