Linux clock framework development
 help / color / mirror / Atom feed
From: "Uwe Kleine-König" <ukleinek@kernel.org>
To: James Hilliard <james.hilliard1@gmail.com>
Cc: Richard Genoud <richard.genoud@bootlin.com>,
	 Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	 Conor Dooley <conor+dt@kernel.org>,
	Chen-Yu Tsai <wens@kernel.org>,
	 Jernej Skrabec <jernej.skrabec@gmail.com>,
	Samuel Holland <samuel@sholland.org>,
	 Philipp Zabel <p.zabel@pengutronix.de>,
	Michael Turquette <mturquette@baylibre.com>,
	 Stephen Boyd <sboyd@kernel.org>,
	Brian Masney <bmasney@redhat.com>,
	 Paul Kocialkowski <paulk@sys-base.io>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	 John Stultz <jstultz@google.com>,
	Joao Schim <joao@schimsalabim.eu>,
	bigunclemax@gmail.com,  linux-pwm@vger.kernel.org,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	 linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org,
	linux-clk@vger.kernel.org
Subject: Re: [PATCH v7 2/4] pwm: sun8i: Add H616 PWM support
Date: Tue, 18 Aug 2026 23:26:21 +0200	[thread overview]
Message-ID: <aoTMMAGbWelnjJTo@monoceros> (raw)
In-Reply-To: <CADvTj4r=uBJyJdmK3T+jD_Nitg2NBs7fZgtE5p6wt9zwWgj6DQ@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1050 bytes --]

Hello,

On Tue, Aug 18, 2026 at 08:12:43AM -0600, James Hilliard wrote:
> On Tue, Aug 18, 2026 at 5:00 AM Uwe Kleine-König <ukleinek@kernel.org> wrote:
> > I didn't look at Sashiko's feedback
> > (https://lore.kernel.org/all/20260703153337.B5AFF1F000E9@smtp.kernel.org)
> > but I would expect that not all 9 high-severity issues identified are
> > slop. Please look through these for the next revision.
> 
> I sent a v8 that should have fixed all the sashiko issues along with a
> bunch of other issues:
> https://lore.kernel.org/all/20260804-h616-pwm-v8-v8-0-db37ab8624ae@gmail.com/T/

Damn, I missed that because my usual check for newer revisions is using
the sender as key (because subjects occasionally change). You didn't
write about your patch series being coordinated with Richard, I hope it
is.

Just looking at your changelog I think most of my manual feedback still
needs addressing. Can you (or Richard?) please go through my feedback
and address the things still relevant for your v8 please?

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

  reply	other threads:[~2026-08-18 21:26 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-03 15:22 [PATCH v7 0/4] Introduce Allwinner H616 PWM controller Richard Genoud
2026-07-03 15:22 ` [PATCH v7 1/4] dt-bindings: pwm: allwinner: add h616 pwm compatible Richard Genoud
2026-07-03 15:22 ` [PATCH v7 2/4] pwm: sun8i: Add H616 PWM support Richard Genoud
2026-07-03 16:00   ` Philipp Zabel
2026-08-18 10:58   ` Uwe Kleine-König
2026-08-18 14:12     ` James Hilliard
2026-08-18 21:26       ` Uwe Kleine-König [this message]
2026-08-18 21:36         ` James Hilliard
2026-07-03 15:22 ` [PATCH v7 3/4] arm64: dts: allwinner: h616: add PWM controller Richard Genoud
2026-07-03 15:22 ` [PATCH v7 4/4] MAINTAINERS: Add entry on Allwinner sun8i/H616 PWM driver Richard Genoud

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=aoTMMAGbWelnjJTo@monoceros \
    --to=ukleinek@kernel.org \
    --cc=bigunclemax@gmail.com \
    --cc=bmasney@redhat.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=james.hilliard1@gmail.com \
    --cc=jernej.skrabec@gmail.com \
    --cc=joao@schimsalabim.eu \
    --cc=jstultz@google.com \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pwm@vger.kernel.org \
    --cc=linux-sunxi@lists.linux.dev \
    --cc=mturquette@baylibre.com \
    --cc=p.zabel@pengutronix.de \
    --cc=paulk@sys-base.io \
    --cc=richard.genoud@bootlin.com \
    --cc=robh@kernel.org \
    --cc=samuel@sholland.org \
    --cc=sboyd@kernel.org \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=wens@kernel.org \
    /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