Messages in this thread | | | From | "Potthuri, Sai Krishna" <> | Subject | RE: [PATCH v2 1/2] dt-bindings: mtd: spi-nor: Add reset-gpios property | Date | Thu, 1 Sep 2022 10:18:21 +0000 |
| |
Hi Krzysztof,
> -----Original Message----- > From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Sent: Thursday, September 1, 2022 1:26 PM > To: Potthuri, Sai Krishna <sai.krishna.potthuri@amd.com>; Tudor Ambarus > <tudor.ambarus@microchip.com>; Pratyush Yadav <pratyush@kernel.org>; > Michael Walle <michael@walle.cc>; Miquel Raynal > <miquel.raynal@bootlin.com>; Richard Weinberger <richard@nod.at>; > Vignesh Raghavendra <vigneshr@ti.com>; Rob Herring > <robh+dt@kernel.org>; Krzysztof Kozlowski > <krzysztof.kozlowski+dt@linaro.org> > Cc: devicetree@vger.kernel.org; linux-mtd@lists.infradead.org; linux- > kernel@vger.kernel.org; saikrishna12468@gmail.com; git (AMD-Xilinx) > <git@amd.com> > Subject: Re: [PATCH v2 1/2] dt-bindings: mtd: spi-nor: Add reset-gpios > property > > On 01/09/2022 10:29, Sai Krishna Potthuri wrote: > > SPI-NOR flashes have RESET pin which can be toggled using GPIO > > controller, for those platforms reset-gpios property can be used to > > reset the flash device. > > > > Signed-off-by: Sai Krishna Potthuri <sai.krishna.potthuri@amd.com> > > --- > > You still miss the user (DTS) of this change... Do you mean to have this property in example DTS in this yaml?
> The JEDEC spec did not mention > a reset pin. Can you provide the user? Xilinx Octal SPI (cdns,qspi-nor.yaml) is using this reset pin to reset the flash device on Versal platform. Do you want me to update the example dts in cdns,qspi-nor.yaml file to reflect this property in this patch?
Regards Sai Krishna
| |