On 2016/4/11 22:43, Rob Herring wrote: > On Thu, Apr 07, 2016 at 04:23:16PM +0800, Kefeng Wang wrote: >> Document the devicetree bindings for the random number generator found >> on Hisilicon Hip04 and Hip05 soc. >> >> Signed-off-by: Kefeng Wang <wangkefeng.wang@xxxxxxxxxx> >> --- >> Documentation/devicetree/bindings/rng/hisi-rng.txt | 12 ++++++++++++ >> 1 file changed, 12 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/rng/hisi-rng.txt >> >> diff --git a/Documentation/devicetree/bindings/rng/hisi-rng.txt b/Documentation/devicetree/bindings/rng/hisi-rng.txt >> new file mode 100644 >> index 0000000..72efcc1 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/rng/hisi-rng.txt >> @@ -0,0 +1,12 @@ >> +Hisilicon Random Number Generator >> + >> +Required properties: >> +- compatible : Should be "hisilicon,rng" >> +- reg : Offset and length of the register set of this block >> + >> +Example: >> + >> +rng@d1010000 { >> + compatible = "hisilicon,rng"; > > This still needs SoC specific compatible strings. It is used by Hip04/Hip05 at lease, but for new SoC, like Hip06, it maybe different, using single Soc prefix is not proper, how about hisilicon,rng-v1, or add two compatible hisilicon,hip04-rng and hisilicon,hip05-rng? What's your option? Thanks. > >> + reg = <0xd1010000 0x100>; >> +}; >> -- >> 2.6.0.GIT >> >> -- >> To unsubscribe from this list: send the line "unsubscribe devicetree" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > . > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html