Re: [PATCH v2 10/17] dt-bindings: spi: add bindings for microchip mpfs spi

From
Conor.Dooley@microchip.com
Date
2021-12-20 08:05:22
Also in
linux-crypto, linux-devicetree, linux-i2c, linux-pwm, linux-riscv, linux-rtc, linux-spi, linux-usb, lkml
Reply to
Mark Brown, 2021-12-17
On 17/12/2021 11:43, Mark Brown wrote:
EXTERNAL EMAIL: Do not click links or open attachments unless you 
know the content is safe
On Fri, Dec 17, 2021 at 11:40:29AM +0000, Conor.Dooley@microchip.com 
wrote:
quoted
On 17/12/2021 11:17, Mark Brown wrote:
quoted
Why do you need this property in the DT - isn't the number of chip
selects in the IP a fixes property?
Nope! It's an IP that's intended for use in FPGAs so the number of
selects may (and does) vary based on implementation.
That doesn't explain why the number is needed in the binding - why do
you need this property in the DT?
Took another look at the IP core configuration & I think you're right 
and that property can be dropped. The register responsible for CS 
control will not be optimised away even if the CS is not used & the 
worst outcome is that nothing will happen.
Thread (56 messages) 56 messages, 8 authors, last reply 2022-01-14
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help