Hello Wolfram, Thank you for the feedback! > From: Wolfram Sang <wsa@xxxxxxxxxxxxx> > Sent: 12 June 2019 10:45 > Subject: Re: [PATCH] arm64: dts: renesas: hihope-common: Add uSD and eMMC > > > > I was thinking about sending a follow-up patch to list hs400 here > > after Wolfram's patch appears in a RC, would that be okay with you? > > Do we need that? The *board* does not handle HS400, so why add the > property? The SoC the design currently comes with doesn't handle HS400, but they may replace that in the future with one that is based off R-Car M3-W version 3.0 (which does support HS400). Also, the HiHope will come in two flavours, the HiHope RZ/G2M (the flavour we are currently adding support for), and the HiHope RZ/G2N (sporting an RZ/G2N, which is based off R-Car M3-N). File arch/arm64/boot/dts/renesas/hihope-common.dtsi contains common definitions for the mother boards for both flavours (including the eMMC). My understanding is that R-Car M3-N does support HS400, hence the need for the property here. Thanks, Fab > > Similar setting would be an I2C device which can do 400kHz but the board > layout doesn't allow for such speeds, so we are limited to 100kHz. > > Or?