Hi Baruch, Baruch Siach <baruch@xxxxxxxxxx> wrote on Wed, 6 Mar 2019 12:30:39 +0200: > Hi Miquel, > > On Wed, Mar 06, 2019 at 11:21:46AM +0100, Miquel Raynal wrote: > > From: Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxx> > > > > There are two SATA ports per CP110. Each of them has a dedicated > > interrupt. Describe the real hardware by adding two SATA ports to the > > CP110 SATA node and enabling them in all the DTs including it > > (7040-db/8040-db/8040-clearfog). > > > > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxx> > > Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx> > > --- > > arch/arm64/boot/dts/marvell/armada-7040-db.dts | 7 ++++++- > > .../dts/marvell/armada-8040-clearfog-gt-8k.dts | 5 ++++- > > arch/arm64/boot/dts/marvell/armada-8040-db.dts | 14 ++++++++++++-- > > arch/arm64/boot/dts/marvell/armada-cp110.dtsi | 16 ++++++++++++++-- > > What about armada-8040-mcbin.dtsi? > I only checked final .dts files but indeed the SATA node is enabled in the mcbin .dtsi. I will wait a bit for other reviews, if there is no need to send a v3 for the SATA patches I will just resend the DT patches for Gregory in a v3. Thanks, Miquèl
