mbox series

[00/10] gpio: msc313: Add gpio support for ssd20xd

Message ID 20210717045627.1739959-1-daniel@0x0f.com
Headers show
Series gpio: msc313: Add gpio support for ssd20xd | expand

Message

Daniel Palmer July 17, 2021, 4:56 a.m. UTC
This is a little series to add a compatible string for the Sigmastar
SSD201 and SSD202D to the msc313 gpio driver, add the specific offsets
for the pins on these chips, and then a bunch of DT wiring stuff so
that the LEDs on the M5 stack unitv2 work and it can control the
power switch for the USB connected.

Daniel Palmer (10):
  dt-bindings: gpio: msc313: Add compatible for ssd20xd
  dt-bindings: gpio: msc313: Add offsets for ssd20xd
  gpio: msc313: Code clean ups
  gpio: msc313: Add support for SSD201 and SSD202D
  ARM: dts: mstar: Set gpio compatible for ssd20xd
  ARM: dts: mstar: unitv2: Wire up LEDs
  ARM: dts: mstar: unitv2: Add core regulator
  ARM: dts: mstar: unitv2: Add io regulator
  ARM: dts: mstar: unitv2: Add DRAM regulator
  ARM: dts: mstar: unitv2: Add wifi switch

 .../bindings/gpio/mstar,msc313-gpio.yaml      |   4 +-
 .../dts/mstar-infinity2m-ssd202d-unitv2.dts   |  46 +++
 .../boot/dts/mstar-infinity2m-ssd20xd.dtsi    |   5 +
 drivers/gpio/gpio-msc313.c                    | 266 +++++++++++++++++-
 include/dt-bindings/gpio/msc313-gpio.h        |  71 +++++
 5 files changed, 387 insertions(+), 5 deletions(-)

Comments

Linus Walleij July 30, 2021, 10:12 a.m. UTC | #1
On Sat, Jul 17, 2021 at 6:56 AM Daniel Palmer <daniel@0x0f.com> wrote:

> This is a little series to add a compatible string for the Sigmastar
> SSD201 and SSD202D to the msc313 gpio driver, add the specific offsets
> for the pins on these chips, and then a bunch of DT wiring stuff so
> that the LEDs on the M5 stack unitv2 work and it can control the
> power switch for the USB connected.
>
> Daniel Palmer (10):
>   dt-bindings: gpio: msc313: Add compatible for ssd20xd
>   dt-bindings: gpio: msc313: Add offsets for ssd20xd
>   gpio: msc313: Code clean ups
>   gpio: msc313: Add support for SSD201 and SSD202D

I suppose Bartosz can just merge the 4 first patches into the
GPIO tree and you can take the rest into the SoC tree?

Yours,
Linus Walleij
Daniel Palmer July 30, 2021, 10:19 a.m. UTC | #2
Hi Linus,

On Fri, 30 Jul 2021 at 19:12, Linus Walleij <linus.walleij@linaro.org> wrote:
> On Sat, Jul 17, 2021 at 6:56 AM Daniel Palmer <daniel@0x0f.com> wrote:
> > Daniel Palmer (10):
> >   dt-bindings: gpio: msc313: Add compatible for ssd20xd
> >   dt-bindings: gpio: msc313: Add offsets for ssd20xd
> >   gpio: msc313: Code clean ups
> >   gpio: msc313: Add support for SSD201 and SSD202D
>
> I suppose Bartosz can just merge the 4 first patches into the
> GPIO tree

Yep.

>And you can take the rest into the SoC tree?

My plan is to take the rest into a "mstar dts for 5.15" branch along
with some other bits and send a PR to Arnd and Olof later on.

Thanks for checking out the patches.

Cheers,

Daniel
Daniel Palmer Aug. 25, 2021, 11:50 a.m. UTC | #3
+ Bartosz

Hi Linus, Bartosz,

On Fri, 30 Jul 2021 at 19:12, Linus Walleij <linus.walleij@linaro.org> wrote:
> I suppose Bartosz can just merge the 4 first patches into the
> GPIO tree and you can take the rest into the SoC tree?
>
> Yours,
> Linus Walleij

I just noticed I hadn't to'd Bartosz about these. Is there anything I
need to do for the first patches?

Cheers,

Daniel
Bartosz Golaszewski Sept. 22, 2021, 10:20 a.m. UTC | #4
On Wed, Aug 25, 2021 at 1:50 PM Daniel Palmer <daniel@0x0f.com> wrote:
>

> + Bartosz

>

> Hi Linus, Bartosz,

>

> On Fri, 30 Jul 2021 at 19:12, Linus Walleij <linus.walleij@linaro.org> wrote:

> > I suppose Bartosz can just merge the 4 first patches into the

> > GPIO tree and you can take the rest into the SoC tree?

> >

> > Yours,

> > Linus Walleij

>

> I just noticed I hadn't to'd Bartosz about these. Is there anything I

> need to do for the first patches?

>

> Cheers,

>

> Daniel


I only have a part of this series in my inbox and patchwork doesn't
have it at all - can you resend it with me in Cc?

Bart
Daniel Palmer Sept. 22, 2021, 10:46 a.m. UTC | #5
Hi Bart,


On Wed, 22 Sept 2021 at 19:20, Bartosz Golaszewski
<bgolaszewski@baylibre.com> wrote:
> I only have a part of this series in my inbox and patchwork doesn't

> have it at all - can you resend it with me in Cc?


Sure. I want to do some fixes to the DTS bits at the end of the series
so I will send it as a v2. The gpio code parts will be unchanged.

Cheers,

Daniel
Andy Shevchenko Sept. 22, 2021, 10:51 a.m. UTC | #6
On Wed, Sep 22, 2021 at 1:21 PM Bartosz Golaszewski
<bgolaszewski@baylibre.com> wrote:
> On Wed, Aug 25, 2021 at 1:50 PM Daniel Palmer <daniel@0x0f.com> wrote:

> > Hi Linus, Bartosz,


...

> > I just noticed I hadn't to'd Bartosz about these. Is there anything I

> > need to do for the first patches?

>

> I only have a part of this series in my inbox and patchwork doesn't

> have it at all - can you resend it with me in Cc?


Have you tried to use the `b4` tool? It is capable of getting
everything from the lore archive.

-- 
With Best Regards,
Andy Shevchenko
Andy Shevchenko Sept. 22, 2021, 12:56 p.m. UTC | #7
On Wed, Sep 22, 2021 at 3:50 PM Bartosz Golaszewski
<bgolaszewski@baylibre.com> wrote:
> On Wed, Sep 22, 2021 at 12:52 PM Andy Shevchenko

> <andy.shevchenko@gmail.com> wrote:

> > On Wed, Sep 22, 2021 at 1:21 PM Bartosz Golaszewski

> > <bgolaszewski@baylibre.com> wrote:

> > > On Wed, Aug 25, 2021 at 1:50 PM Daniel Palmer <daniel@0x0f.com> wrote:


...

> > > I only have a part of this series in my inbox and patchwork doesn't

> > > have it at all - can you resend it with me in Cc?

> >

> > Have you tried to use the `b4` tool? It is capable of getting

> > everything from the lore archive.

>

> I haven't tried it because I have never heard of it. :) Thanks for the heads-up.


You're welcome!

> At the same time - is it too much to ask of people to just use

> scripts/get_maintainer.pl? It's so much easier to just actually get

> the patches in my inbox than to have to look for them manually.


Of course not. I completely agree with you on the point.
Btw, I advertise a script I wrote [1] exactly due to misses of this in the past.

[1]: https://github.com/andy-shev/home-bin-tools/blob/master/ge2maintainer.sh


--
With Best Regards,
Andy Shevchenko
Daniel Palmer Sept. 22, 2021, 2:52 p.m. UTC | #8
Hi Bartosz,

On Wed, 22 Sept 2021 at 21:50, Bartosz Golaszewski
<bgolaszewski@baylibre.com> wrote:
> > > I only have a part of this series in my inbox and patchwork doesn't

> > > have it at all - can you resend it with me in Cc?


I just checked and the series is in patchwork  -
https://patchwork.ozlabs.org/project/linux-gpio/patch/20210717045627.1739959-2-daniel@0x0f.com/
Unless I got the wrong patchwork?

> At the same time - is it too much to ask of people to just use

> scripts/get_maintainer.pl?


Sorry about that. scripts/get_maintainer.pl can return tons of people,
I get cc'd on stuff I have no idea about because I touched a line
somewhere, so I generally try to work out who needs to see a series
from the places touched and the get_maintainer.pl output instead of
just spamming everyone. In this case I thought Linus was the subsystem
maintainer as he took the initial series for this driver and I took
the mail recipients list from that series and pasted it in.

I'll be more careful next time.

Thanks,

Daniel
Linus Walleij Sept. 22, 2021, 11:50 p.m. UTC | #9
On Wed, Sep 22, 2021 at 2:50 PM Bartosz Golaszewski
<bgolaszewski@baylibre.com> wrote:
> On Wed, Sep 22, 2021 at 12:52 PM Andy Shevchenko

> <andy.shevchenko@gmail.com> wrote:


> > Have you tried to use the `b4` tool? It is capable of getting

> > everything from the lore archive.

>

> I haven't tried it because I have never heard of it. :) Thanks for the heads-up.


I use this all the time, it is awesome.

Yours,
Linus Walleij