| Message ID | 20190822092643.593488-1-lkundrak@v3.sk (mailing list archive) |
|---|---|
| Headers | show |
| Series | Initial support for Marvell MMP3 SoC | expand |
On 22/08/2019 10:26, Lubomir Rintel wrote: > Hi, > > this is a second spin of a patch set that adds support for the Marvell > MMP3 processor. MMP3 is used in OLPC XO-4 laptops, Panasonic Toughpad > FZ-A1 tablet and Dell Wyse 3020 Tx0D thin clients. > > Compared to v1, there's a handful of fixes in response to reviews. Patch > 02/20 is new. Details in individual patches. > > Apart from the adjustments in mach-mmp/, the patch makes necessary > changes to the irqchip driver and adds an USB2 PHY driver. The latter > has a dependency on the mach-mmp/ changes, so it can't be submitted > separately. > > The patch set has been tested to work on Wyse Tx0D and not ruin MMP2 > support on XO-1.75. How do you want this series to be merged? I'm happy to take the irqchip related patches as well as the corresponding DT change (once reviewed) through my tree. Thanks, M.
On Thu, Aug 22, 2019 at 3:32 AM Marc Zyngier <maz@kernel.org> wrote: > > On 22/08/2019 10:26, Lubomir Rintel wrote: > > Hi, > > > > this is a second spin of a patch set that adds support for the Marvell > > MMP3 processor. MMP3 is used in OLPC XO-4 laptops, Panasonic Toughpad > > FZ-A1 tablet and Dell Wyse 3020 Tx0D thin clients. > > > > Compared to v1, there's a handful of fixes in response to reviews. Patch > > 02/20 is new. Details in individual patches. > > > > Apart from the adjustments in mach-mmp/, the patch makes necessary > > changes to the irqchip driver and adds an USB2 PHY driver. The latter > > has a dependency on the mach-mmp/ changes, so it can't be submitted > > separately. > > > > The patch set has been tested to work on Wyse Tx0D and not ruin MMP2 > > support on XO-1.75. > > How do you want this series to be merged? I'm happy to take the irqchip > related patches as well as the corresponding DT change (once reviewed) > through my tree. DT changes, unless there's lack of backwards compatibility, are best merged through the platform trees. Especially for new platforms like these where there's likely going to be nearby changes (and thus conflicts). I.e. driver changes I'm all for bringing through driver trees (including binding patches), but please leave dts/dtsi changes to the platform. -Olof
On Thu, 2019-08-22 at 11:31 +0100, Marc Zyngier wrote: > On 22/08/2019 10:26, Lubomir Rintel wrote: > > Hi, > > > > this is a second spin of a patch set that adds support for the Marvell > > MMP3 processor. MMP3 is used in OLPC XO-4 laptops, Panasonic Toughpad > > FZ-A1 tablet and Dell Wyse 3020 Tx0D thin clients. > > > > Compared to v1, there's a handful of fixes in response to reviews. Patch > > 02/20 is new. Details in individual patches. > > > > Apart from the adjustments in mach-mmp/, the patch makes necessary > > changes to the irqchip driver and adds an USB2 PHY driver. The latter > > has a dependency on the mach-mmp/ changes, so it can't be submitted > > separately. > > > > The patch set has been tested to work on Wyse Tx0D and not ruin MMP2 > > support on XO-1.75. > > How do you want this series to be merged? I'm happy to take the irqchip > related patches as well as the corresponding DT change (once reviewed) > through my tree. I was hoping for the Arm SoC tree, because there are some dependencies (MMP3 USB PHY depends on MMP3 SoC). That said, the irqchip patches are rather independent and the only downside of them going in via a different tree will be that the other tree that will lack them won't boot on MMP3 (things will compile though). I don't know if that's okay. What's typically done in cases like these? > Thanks, > > M. Thank you Lubo
On 23/08/2019 08:21, Lubomir Rintel wrote: > On Thu, 2019-08-22 at 11:31 +0100, Marc Zyngier wrote: >> On 22/08/2019 10:26, Lubomir Rintel wrote: >>> Hi, >>> >>> this is a second spin of a patch set that adds support for the Marvell >>> MMP3 processor. MMP3 is used in OLPC XO-4 laptops, Panasonic Toughpad >>> FZ-A1 tablet and Dell Wyse 3020 Tx0D thin clients. >>> >>> Compared to v1, there's a handful of fixes in response to reviews. Patch >>> 02/20 is new. Details in individual patches. >>> >>> Apart from the adjustments in mach-mmp/, the patch makes necessary >>> changes to the irqchip driver and adds an USB2 PHY driver. The latter >>> has a dependency on the mach-mmp/ changes, so it can't be submitted >>> separately. >>> >>> The patch set has been tested to work on Wyse Tx0D and not ruin MMP2 >>> support on XO-1.75. >> >> How do you want this series to be merged? I'm happy to take the irqchip >> related patches as well as the corresponding DT change (once reviewed) >> through my tree. > > I was hoping for the Arm SoC tree, because there are some dependencies > (MMP3 USB PHY depends on MMP3 SoC). > > That said, the irqchip patches are rather independent and the only > downside of them going in via a different tree will be that the other > tree that will lack them won't boot on MMP3 (things will compile > though). I don't know if that's okay. What's typically done in cases > like these? I usually take the irqchip patches that can be built standalone (without dependency on header files, for example). If you want them to go via another tree, stick my Acked-by: Marc Zyngier <maz@kernel.org> on patches #6 through #9. Thanks, M.
On Fri, 2019-08-23 at 10:42 +0100, Marc Zyngier wrote: > On 23/08/2019 08:21, Lubomir Rintel wrote: > > On Thu, 2019-08-22 at 11:31 +0100, Marc Zyngier wrote: > > > On 22/08/2019 10:26, Lubomir Rintel wrote: > > > > Hi, > > > > > > > > this is a second spin of a patch set that adds support for the Marvell > > > > MMP3 processor. MMP3 is used in OLPC XO-4 laptops, Panasonic Toughpad > > > > FZ-A1 tablet and Dell Wyse 3020 Tx0D thin clients. > > > > > > > > Compared to v1, there's a handful of fixes in response to reviews. Patch > > > > 02/20 is new. Details in individual patches. > > > > > > > > Apart from the adjustments in mach-mmp/, the patch makes necessary > > > > changes to the irqchip driver and adds an USB2 PHY driver. The latter > > > > has a dependency on the mach-mmp/ changes, so it can't be submitted > > > > separately. > > > > > > > > The patch set has been tested to work on Wyse Tx0D and not ruin MMP2 > > > > support on XO-1.75. > > > > > > How do you want this series to be merged? I'm happy to take the irqchip > > > related patches as well as the corresponding DT change (once reviewed) > > > through my tree. > > > > I was hoping for the Arm SoC tree, because there are some dependencies > > (MMP3 USB PHY depends on MMP3 SoC). > > > > That said, the irqchip patches are rather independent and the only > > downside of them going in via a different tree will be that the other > > tree that will lack them won't boot on MMP3 (things will compile > > though). I don't know if that's okay. What's typically done in cases > > like these? > > I usually take the irqchip patches that can be built standalone (without > dependency on header files, for example). If you want them to go via > another tree, stick my > > Acked-by: Marc Zyngier <maz@kernel.org> > > on patches #6 through #9. Actually, please go ahead and pick the irqchip patches into your tree. The rest of the patch set may need a couple more spins, and it will be nice if it gets shorter. Thank you Lubo
On 26/08/2019 12:59, Lubomir Rintel wrote: > On Fri, 2019-08-23 at 10:42 +0100, Marc Zyngier wrote: >> On 23/08/2019 08:21, Lubomir Rintel wrote: >>> On Thu, 2019-08-22 at 11:31 +0100, Marc Zyngier wrote: >>>> On 22/08/2019 10:26, Lubomir Rintel wrote: >>>>> Hi, >>>>> >>>>> this is a second spin of a patch set that adds support for the Marvell >>>>> MMP3 processor. MMP3 is used in OLPC XO-4 laptops, Panasonic Toughpad >>>>> FZ-A1 tablet and Dell Wyse 3020 Tx0D thin clients. >>>>> >>>>> Compared to v1, there's a handful of fixes in response to reviews. Patch >>>>> 02/20 is new. Details in individual patches. >>>>> >>>>> Apart from the adjustments in mach-mmp/, the patch makes necessary >>>>> changes to the irqchip driver and adds an USB2 PHY driver. The latter >>>>> has a dependency on the mach-mmp/ changes, so it can't be submitted >>>>> separately. >>>>> >>>>> The patch set has been tested to work on Wyse Tx0D and not ruin MMP2 >>>>> support on XO-1.75. >>>> >>>> How do you want this series to be merged? I'm happy to take the irqchip >>>> related patches as well as the corresponding DT change (once reviewed) >>>> through my tree. >>> >>> I was hoping for the Arm SoC tree, because there are some dependencies >>> (MMP3 USB PHY depends on MMP3 SoC). >>> >>> That said, the irqchip patches are rather independent and the only >>> downside of them going in via a different tree will be that the other >>> tree that will lack them won't boot on MMP3 (things will compile >>> though). I don't know if that's okay. What's typically done in cases >>> like these? >> >> I usually take the irqchip patches that can be built standalone (without >> dependency on header files, for example). If you want them to go via >> another tree, stick my >> >> Acked-by: Marc Zyngier <maz@kernel.org> >> >> on patches #6 through #9. > > Actually, please go ahead and pick the irqchip patches into your tree. > > The rest of the patch set may need a couple more spins, and it will be > nice if it gets shorter. Applied to irqchip-next. M.