Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1471390 > unrolled thread

Re: [PATCH v3 1/4] pwm: Add support for Meson PWM Controller

Started byMartin Blumenstingl <martin.blumenstingl@googlemail.com>
First post2016-08-28 18:40 +0200
Last post2016-09-06 23:30 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v3 1/4] pwm: Add support for Meson PWM Controller Martin Blumenstingl <martin.blumenstingl@googlemail.com> - 2016-08-28 18:40 +0200
    Re: [PATCH v3 1/4] pwm: Add support for Meson PWM Controller Thierry Reding <thierry.reding@gmail.com> - 2016-09-05 12:00 +0200
      Re: [PATCH v3 1/4] pwm: Add support for Meson PWM Controller Martin Blumenstingl <martin.blumenstingl@googlemail.com> - 2016-09-06 23:30 +0200

#1471390 — Re: [PATCH v3 1/4] pwm: Add support for Meson PWM Controller

FromMartin Blumenstingl <martin.blumenstingl@googlemail.com>
Date2016-08-28 18:40 +0200
SubjectRe: [PATCH v3 1/4] pwm: Add support for Meson PWM Controller
Message-ID<sbbhM-44E-37@gated-at.bofh.it>
On Mon, Aug 22, 2016 at 5:36 PM, Neil Armstrong <narmstrong@baylibre.com> wrote:
> Add support for the PWM controller found in the Amlogic SoCs.
> This driver supports the Meson8b and GXBB SoCs.
>
> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
I could enable the AP6330 SDIO wifi module with this on my
meson-gxbb-vega-s95-meta clone, which uses pwm_e to generate the LPO
clock for the wifi module (see [0] for the corresponding patches).
So thank you Neil, and:
Tested-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>


[0] https://github.com/xdarklight/linux/tree/meson-gxbb-integration-4.8-20160828

[toc] | [next] | [standalone]


#1476245

FromThierry Reding <thierry.reding@gmail.com>
Date2016-09-05 12:00 +0200
Message-ID<sdYR4-209-35@gated-at.bofh.it>
In reply to#1471390

[Multipart message — attachments visible in raw view] — view raw

On Sun, Aug 28, 2016 at 06:33:03PM +0200, Martin Blumenstingl wrote:
> On Mon, Aug 22, 2016 at 5:36 PM, Neil Armstrong <narmstrong@baylibre.com> wrote:
> > Add support for the PWM controller found in the Amlogic SoCs.
> > This driver supports the Meson8b and GXBB SoCs.
> >
> > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> I could enable the AP6330 SDIO wifi module with this on my
> meson-gxbb-vega-s95-meta clone, which uses pwm_e to generate the LPO
> clock for the wifi module (see [0] for the corresponding patches).
> So thank you Neil, and:
> Tested-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>

Hi Martin,

As mentioned in another reply in this thread I ended up restructuring
the driver a little before applying it to the PWM tree. I don't /think/
there are any functional changes, so I left your Tested-by in place, but
it would be great if you could retest and confirm, just to make sure.

The driver is in the for-4.9/drivers branch of the PWM tree:

	https://git.kernel.org/cgit/linux/kernel/git/thierry.reding/linux-pwm.git/log/?h=for-4.9/drivers

Thanks,
Thierry

[toc] | [prev] | [next] | [standalone]


#1477830

FromMartin Blumenstingl <martin.blumenstingl@googlemail.com>
Date2016-09-06 23:30 +0200
Message-ID<sew6l-7po-5@gated-at.bofh.it>
In reply to#1476245
Hi Thierry,

On Mon, Sep 5, 2016 at 11:53 AM, Thierry Reding
<thierry.reding@gmail.com> wrote:
> As mentioned in another reply in this thread I ended up restructuring
> the driver a little before applying it to the PWM tree. I don't /think/
> there are any functional changes, so I left your Tested-by in place, but
> it would be great if you could retest and confirm, just to make sure.
>
> The driver is in the for-4.9/drivers branch of the PWM tree:
>
>         https://git.kernel.org/cgit/linux/kernel/git/thierry.reding/linux-pwm.git/log/?h=for-4.9/drivers
I have tested it with the updated Neil sent today: [0]
This is still working fine for me (driving the LPO clock of an AP6330
SDIO wifi chip).

Please note that I don't have a scope to measure the actual signal,
but I guess it's fine since my AP6330 is happy.

So feel free to keep my:
Tested-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>


Regards,
Martin


[0] https://github.com/superna9999/linux/releases/tag/amlogic/v4.8/pwm-for-next-for-v4

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web