[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core
From: |
Thomas Huth |
Subject: |
Re: [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core CPU logic |
Date: |
Sat, 25 Jan 2020 18:08:18 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 25/01/2020 11.48, Aleksandar Markovic wrote:
>
> On Friday, January 24, 2020, Philippe Mathieu-Daudé <address@hidden
> <mailto:address@hidden>> wrote:
>
> From: Michael Rolnik <address@hidden <mailto:address@hidden>>
>
> This includes:
> - CPU data structures
> - object model classes and functions
> - migration functions
> - GDB hooks
>
> I have an objection over this patch.
>
> It contains many diverse logical units squashed into a patch, and
> therefore is not in accordance to our submission giidelines.
I think you have to decide on a case by case basis. This is a new target
for a hobbyist board ... so I think you don't have to thaaat strict here.
> If we accept this patch, we will be setting a bad precedent, that may
> misled future platform contributors. additionally, this patch may be
> singled out in our countribution guidelines as the example how not to do
> a parch, which is probably not what Michael want to be exposed to.
Honestly, I'd leave that decision to the person who picks up the patch
and sends a pull request, i.e. Richard in this case, I assume.
> Splitting patches is tedius, but overall not that difficult or time
> consuming task.
OTOH hand, we've seen more than 40 iterations of the patch series
already. Michael invested a lot of time into this series, so if you make
him respin forever, he'll certainly rather lose interest at one point in
time. So I'd say if Richard is fine with the series, I'd rather not like
to see yet another iteration on the mailing list again and rather see a
pull request for these patches instead.
Thomas
- [PATCH rc2 00/25] target/avr merger, Philippe Mathieu-Daudé, 2020/01/23
- [PATCH rc2 03/25] target/avr: Add instruction translation - Registers definition, Philippe Mathieu-Daudé, 2020/01/23
- [PATCH rc2 02/25] target/avr: Add instruction helpers, Philippe Mathieu-Daudé, 2020/01/23
- [PATCH rc2 05/25] target/avr: Add instruction translation - Branch Instructions, Philippe Mathieu-Daudé, 2020/01/23
- [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core CPU logic, Philippe Mathieu-Daudé, 2020/01/23
- Re: [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core CPU logic, Aleksandar Markovic, 2020/01/25
- Re: [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core CPU logic,
Thomas Huth <=
- Re: [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core CPU logic, Joaquin de Andres, 2020/01/26
- Re: [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core CPU logic, Aleksandar Markovic, 2020/01/26
- Re: [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core CPU logic, Michael Rolnik, 2020/01/27
- Re: [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core CPU logic, Joaquin de Andres, 2020/01/27
- Re: [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core CPU logic, Michael Rolnik, 2020/01/27
- Re: [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core CPU logic, Joaquin de Andres, 2020/01/27
Re: [PATCH rc2 01/25] target/avr: Add outward facing interfaces and core CPU logic, Joaquin de Andres, 2020/01/27
[PATCH rc2 07/25] target/avr: Add instruction translation - Bit and Bit-test Instructions, Philippe Mathieu-Daudé, 2020/01/23