> -----Original Message----- > From: Conor Dooley <conor@xxxxxxxxxx> > Sent: Friday, December 2, 2022 2:04 AM > To: JiaJie Ho <jiajie.ho@xxxxxxxxxxxxxxxx> > Cc: Conor.Dooley@xxxxxxxxxxxxx; robh+dt@xxxxxxxxxx; > herbert@xxxxxxxxxxxxxxxxxxx; linux-crypto@xxxxxxxxxxxxxxx; > kernel@xxxxxxxx; davem@xxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; > linux-kernel@xxxxxxxxxxxxxxx; linux-riscv@xxxxxxxxxxxxxxxxxxx; > krzysztof.kozlowski+dt@xxxxxxxxxx > Subject: Re: [PATCH 6/6] riscv: dts: starfive: Add crypto and DMA node for > VisionFive 2 > > On Thu, Dec 01, 2022 at 06:17:26AM +0000, JiaJie Ho wrote: > > > -----Original Message----- > > > From: Conor.Dooley@xxxxxxxxxxxxx <Conor.Dooley@xxxxxxxxxxxxx> > > In that case, the SoB block should look like: > > Co-developed-by: Huan Feng <huan.feng@xxxxxxxxxxxxxxxx> > Signed-off-by: Huan Feng <huan.feng@xxxxxxxxxxxxxxxx> > Signed-off-by: Jia Jie Ho <jiajie.ho@xxxxxxxxxxxxxxxx> > > Similarly for any other patches they may have co-developed :) > I'll add these in the v2. > If the hardware is always present, why not drop the "disabled" in jh7110.dtsi > & these two entries marking them as "okay" in the .dts? > Okay, I'll update these too. Thanks again for the suggestions. Best regards, Jia Jie
