On 26/02/20 7:02 am, Ramuthevar, Vadivel MuruganX wrote:
Hi,Old bindings does not impose a restriction that this needs to be 128
On 25/2/2020 7:00 PM, Vignesh Raghavendra wrote:
On 25/02/20 1:08 pm, Ramuthevar, Vadivel MuruganX wrote:Agreed, Thanks!
This has nothing to do with quad-mode. Its about how much SRAM amount ofcorrect me if I am wrong, the width of this field is 4bits -> 128 bytesNope, the width of this field is 8bits -> 256 bytes128 is valid, will update.+A 4GB fifo is valid? Add some constraints.
+Â cdns,fifo-depth:
+ÂÂÂ $ref: /schemas/types.yaml#/definitions/uint32
+ÂÂÂ description:
+ÂÂÂÂÂ Size of the data FIFO in words.
(based on QUAD mode) .
SRAM is present to buffer INDAC mode data. For TI platforms this is 256
bytes.
See CQSPI_REG_SRAMPARTITION definition in your datasheet.
Yes , I have gone through it , Intel and Altera SoC's SRAM(act as
FIFO)size is 128 bytes and TI has 256 .
BTW old legacy DT binding mentioned size is 128, as per your earlier
suggestion you have mention that
keep the contents from old dt bindings as it is, so shall I keep 128/256?
bytes always (Its just the example that shows this property to be set to
128)
What Rob is asking for is to add range of values that is valid for this
field and not single value. So, both 128 and 256 bytes should be allowed
as valid values for this property.