Message ID | cover.1595572867.git.frank@allwinnertech.com (mailing list archive) |
---|---|
Headers | show |
Series | Allwinner A100 Initial support | expand |
HI Maxime, Any comments on thermal and other patches? I hope these patches can be integrated into the mainline as soon as possible. Thx, Yangtao
ping......
On Mon, 24 Aug 2020, Frank Lee wrote:
> ping......
"Please don't send content free pings and please allow a reasonable
time for review. People get busy, go on holiday, attend conferences
and so on so unless there is some reason for urgency (like critical
bug fixes) please allow at least a couple of weeks for review. If
there have been review comments then people may be waiting for those
to be addressed. Sending content free pings just adds to the mail
volume (if they are seen at all) and if something has gone wrong
you'll have to resend the patches anyway so [RESEND]ing with any
comments addressed is generally a much better approach."
On Mon, Aug 24, 2020 at 09:03:27AM +0100, Lee Jones wrote: > On Mon, 24 Aug 2020, Frank Lee wrote: > > > ping...... > > "Please don't send content free pings and please allow a reasonable > time for review. People get busy, go on holiday, attend conferences > and so on so unless there is some reason for urgency (like critical > bug fixes) please allow at least a couple of weeks for review. If > there have been review comments then people may be waiting for those > to be addressed. Sending content free pings just adds to the mail > volume (if they are seen at all) and if something has gone wrong > you'll have to resend the patches anyway so [RESEND]ing with any > comments addressed is generally a much better approach." This is true to some extent, but pinging after a month doesn't seem unreasonable either. Maxime
On Tue, Aug 25, 2020 at 10:55:32AM +0200, Maxime Ripard wrote: > On Mon, Aug 24, 2020 at 09:03:27AM +0100, Lee Jones wrote: > > On Mon, 24 Aug 2020, Frank Lee wrote: > > > > > ping...... > > > > "Please don't send content free pings and please allow a reasonable > > time for review. People get busy, go on holiday, attend conferences > > and so on so unless there is some reason for urgency (like critical > > bug fixes) please allow at least a couple of weeks for review. If > > there have been review comments then people may be waiting for those > > to be addressed. Sending content free pings just adds to the mail > > volume (if they are seen at all) and if something has gone wrong > > you'll have to resend the patches anyway so [RESEND]ing with any > > comments addressed is generally a much better approach." > > This is true to some extent, but pinging after a month doesn't seem > unreasonable either. Especially if resending would mean up to 16 patches sent again.
On Tue, 25 Aug 2020, Maxime Ripard wrote: > On Mon, Aug 24, 2020 at 09:03:27AM +0100, Lee Jones wrote: > > On Mon, 24 Aug 2020, Frank Lee wrote: > > > > > ping...... > > > > "Please don't send content free pings and please allow a reasonable > > time for review. People get busy, go on holiday, attend conferences > > and so on so unless there is some reason for urgency (like critical > > bug fixes) please allow at least a couple of weeks for review. If > > there have been review comments then people may be waiting for those > > to be addressed. Sending content free pings just adds to the mail > > volume (if they are seen at all) and if something has gone wrong > > you'll have to resend the patches anyway so [RESEND]ing with any > > comments addressed is generally a much better approach." > > This is true to some extent, but pinging after a month doesn't seem > unreasonable either. Pinging is mostly a fruitless exercise. After a month, many Maintainers would have purged any un-serviced mails anyway. If a patch-set is left hanging, still requiring attention before the next version, submitting a [RESEND] is generally a better option.
On Fri, Jul 24, 2020 at 8:53 AM Frank Lee <frank@allwinnertech.com> wrote: > This patch set adds initial support for allwinner a100 soc, > which is a 64-bit tablet chip. Shall I commit the pinctrl patches (if Maxime ACKed) separately or not? Once Maxime is happy, I am usually happy too. Yours, Linus Walleij
HI Linus, On Fri, Aug 28, 2020 at 6:02 PM Linus Walleij <linus.walleij@linaro.org> wrote: > > On Fri, Jul 24, 2020 at 8:53 AM Frank Lee <frank@allwinnertech.com> wrote: > > > This patch set adds initial support for allwinner a100 soc, > > which is a 64-bit tablet chip. > > Shall I commit the pinctrl patches (if Maxime ACKed) separately > or not? Once Maxime is happy, I am usually happy too. Maxime told me he wanted you to commit the pinctrl patches. Thx, Yangtao
Hi Linus On Fri, Aug 28, 2020 at 12:02:29PM +0200, Linus Walleij wrote: > On Fri, Jul 24, 2020 at 8:53 AM Frank Lee <frank@allwinnertech.com> wrote: > > > This patch set adds initial support for allwinner a100 soc, > > which is a 64-bit tablet chip. > > Shall I commit the pinctrl patches (if Maxime ACKed) separately > or not? Once Maxime is happy, I am usually happy too. Yeah, you can apply it to your tree Thanks! Maxime
On Fri, Jul 24, 2020 at 8:53 AM Frank Lee <frank@allwinnertech.com> wrote: > dt-bindings: pinctrl: sunxi: Get rid of continual nesting > dt-bindings: pinctrl: sunxi: Add A100 pinctrl bindings > pinctrl: sunxi: add support for the Allwinner A100 pin controller These three patches (3,4,5) are now applied to the pinctrl tree. Yours, Linus Walleij
On 24/07/2020 08:52, Frank Lee wrote: > This patch set adds initial support for allwinner a100 soc, > which is a 64-bit tablet chip. Applied patches 7,8,9 Thanks -- Daniel