Re: [PATCH v5 00/16] Allwinner A100 Initial support

[Daniel Lezcano]

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

-- 
 Linaro.org │ Open source software for ARM SoCs

Follow Linaro:   Facebook |
 Twitter |
 Blog

Re: [PATCH v5 00/16] Allwinner A100 Initial support

[Linus Walleij]

On Fri, Jul 24, 2020 at 8:53 AM Frank Lee  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

Re: [PATCH v5 00/16] Allwinner A100 Initial support

[Maxime Ripard]

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  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


signature.asc
Description: PGP signature

Re: [PATCH v5 00/16] Allwinner A100 Initial support

[Frank Lee]

HI Linus,

On Fri, Aug 28, 2020 at 6:02 PM Linus Walleij  wrote:
>
> On Fri, Jul 24, 2020 at 8:53 AM Frank Lee  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

Re: [PATCH v5 00/16] Allwinner A100 Initial support

[Linus Walleij]

On Fri, Jul 24, 2020 at 8:53 AM Frank Lee  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

Re: [PATCH v5 00/16] Allwinner A100 Initial support

[Lee Jones]

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.

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH v5 00/16] Allwinner A100 Initial support

[Wolfram Sang]

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.



signature.asc
Description: PGP signature

Re: [PATCH v5 00/16] Allwinner A100 Initial support

[Maxime Ripard]

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


signature.asc
Description: PGP signature

Re: [PATCH v5 00/16] Allwinner A100 Initial support

[Lee Jones]

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."

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH v5 00/16] Allwinner A100 Initial support

[Frank Lee]

ping..

Re: [PATCH v5 00/16] Allwinner A100 Initial support

[Frank Lee]

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