Thread (11 messages) 11 messages, 4 authors, 2020-05-18

Re: [PATCH v1 8/9] arm64: dts: actions: Add MMC controller support for S700

From: Amit Tomer <hidden>
Date: 2020-05-15 18:41:44
Also in: linux-arm-kernel

Hi,
I was wondering if we should add a SoC specific compatible here, to be
on the safe side. The BSP driver seems to differentiate between S900 and
S700, although it looks like only to cover some misplaced platform setup.

But if we later find a problem, the DTs stay the same, and the driver
can easily be fixed.

So, using "actions,s700-mmc", "actions,owl-mmc" here, adding this combo
to the binding, but leaving the driver alone for now.
I think, it can be a good idea to have this extra compatible string that may
be needed when MMC driver evolves to support more features which requires
data to be differentiated based on SoC.

Thanks
Amit
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help