On 3/30/20 11:19 AM, Christophe Kerello wrote: > > > On 3/30/20 3:24 AM, Marek Vasut wrote: >> On 3/23/20 3:58 PM, Christophe Kerello wrote: >>> The driver adds the support for the STMicroelectronics FMC2 EBI >>> controller >>> found on STM32MP SOCs. >>> >> >> On DH STM32MP1 SoM in PDK2 carrier board, >> Tested-by: Marek Vasut <marex@xxxxxxx> >> >> btw. it seems this sets BTRx DATLAT and CLKDIV to 0xf , it's "Don't >> care" in the datasheet for Muxed mode, but then it should probably be >> set to 0. > > Hi Marek, Hi, > Thanks for testing. Sure > These 2 bit fields (BTRx DATLAT and CLKDIV) are only needed for > synchronous transactions. Based on your bindings, the transaction type > is asynchronous. > CLKDIV bit fields should not be set to 0x0, as this value is reserved > for this bit field. The driver keeps the reset value when it is not > needed to update a bit field. Then I guess that's fine either way. ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/