[PATCH v3] pwm: bcm2835: Support apply function for atomic configuration

Fri Dec 11 04:53:05 EST 2020

On Fri, Dec 11, 2020 at 10:28:35AM +0100, Lino Sanfilippo wrote:
> Hi Uwe,
> 
> > Gesendet: Donnerstag, 10. Dezember 2020 um 12:43 Uhr
> > Von: "Uwe Kleine-König" <u.kleine-koenig at pengutronix.de>
> > An: "Lino Sanfilippo" <LinoSanfilippo at gmx.de>
> > Cc: thierry.reding at gmail.com, lee.jones at linaro.org, nsaenzjulienne at suse.de, f.fainelli at gmail.com, rjui at broadcom.com, sean at mess.org, sbranden at broadcom.com, bcm-kernel-feedback-list at broadcom.com, linux-pwm at vger.kernel.org, linux-rpi-kernel at lists.infradead.org, linux-arm-kernel at lists.infradead.org, linux-kernel at vger.kernel.org
> > Betreff: Re: [PATCH v3] pwm: bcm2835: Support apply function for atomic configuration
> 
> > 
> > Side note: I'm a bit surprised about the output of
> > 
> > 	b4 diff 1607464905-16630-1-git-send-email-LinoSanfilippo at gmx.de
> > 
> > This is probably due to the fact that compared to v3 you also rebased.
> > Still the diff is quite big.
> 
> You are right, I made a rebase before I created the last patch, sorry for the confusion this caused.
> Anyway, thanks for the review(s)!

You did everything good enough. (To further improve, you could use
git-format-patch's --base option and mention a rebase in the series'
changelog; note this is quite high level critic.)

This was more me wondering the output is not easier to use. (And note I
also showed the wrong commandline, but that doesn't resolve the issue.
The right command is:

	b4 diff 1607546905-20549-1-git-send-email-LinoSanfilippo at gmx.de

.)

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20201211/4ac3b2c0/attachment.sig>