2021-11-04 08:25:28

by Marc Zyngier

[permalink] [raw]
Subject: Re: [PATCH v4 00/10] Add Sunplus SP7021 SoC Support

Qin,

On Thu, 04 Nov 2021 02:56:57 +0000,
Qin Jian <[email protected]> wrote:
>
> This patch series add Sunplus SP7021 SoC support.
>
> Sunplus SP7021 is an ARM Cortex A7 (4 cores) based SoC. It integrates many
> peripherals (ex: UART, I2C, SPI, SDIO, eMMC, USB, SD card and etc.) into a
> single chip. It is designed for industrial control.
>
> SP7021 consists of two chips (dies) in a package. One is called C-chip
> (computing chip). It is a 4-core ARM Cortex A7 CPU. It adopts high-level
> process (22 nm) for high performance computing. The other is called P-
> chip (peripheral chip). It has many peripherals and an ARM A926 added
> especially for real-time control. P-chip is made for customers. It adopts
> low-level process (ex: 0.11 um) to reduce cost.

That's the 3rd version of this series since Friday, two of them during
the merge window. All you are achieving is to actually *delay* the
review process (at this rate, I'll probably wait until v11 before I
take another look at it).

Documentation/process/submitting-patches.rst states it clearly:

<quote>
Wait for a minimum of one week before resubmitting
</quote>

So please leave people the time to actually do a good review job, and
take the opportunity to review your own patches before posting them
again.

Thanks,

M.

--
Without deviation from the norm, progress is not possible.


2021-11-04 08:36:21

by Qin Jian

[permalink] [raw]
Subject: 答复: [PATCH v4 00/10] Add Sunplus SP7021 SoC Support

Marc,

Got it. Thank you for reminding me.
I'm sorry for the previous mistake.


> -----?ʼ?ԭ??-----
> ??????: Marc Zyngier <[email protected]>
> ????ʱ??: 2021??11??4?? 16:23
> ?ռ???: qinjian[????] <[email protected]>
> ????: [email protected]; [email protected]; [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected]; [email protected]; [email protected];
> [email protected]; Wells Lu ?η??v <[email protected]>
> ????: Re: [PATCH v4 00/10] Add Sunplus SP7021 SoC Support
>
> Qin,
>
> On Thu, 04 Nov 2021 02:56:57 +0000,
> Qin Jian <[email protected]> wrote:
> >
> > This patch series add Sunplus SP7021 SoC support.
> >
> > Sunplus SP7021 is an ARM Cortex A7 (4 cores) based SoC. It integrates many
> > peripherals (ex: UART, I2C, SPI, SDIO, eMMC, USB, SD card and etc.) into a
> > single chip. It is designed for industrial control.
> >
> > SP7021 consists of two chips (dies) in a package. One is called C-chip
> > (computing chip). It is a 4-core ARM Cortex A7 CPU. It adopts high-level
> > process (22 nm) for high performance computing. The other is called P-
> > chip (peripheral chip). It has many peripherals and an ARM A926 added
> > especially for real-time control. P-chip is made for customers. It adopts
> > low-level process (ex: 0.11 um) to reduce cost.
>
> That's the 3rd version of this series since Friday, two of them during
> the merge window. All you are achieving is to actually *delay* the
> review process (at this rate, I'll probably wait until v11 before I
> take another look at it).
>
> Documentation/process/submitting-patches.rst states it clearly:
>
> <quote>
> Wait for a minimum of one week before resubmitting
> </quote>
>
> So please leave people the time to actually do a good review job, and
> take the opportunity to review your own patches before posting them
> again.
>
> Thanks,
>
> M.
>
> --
> Without deviation from the norm, progress is not possible.