Hi Vijay, On Fri, 18 Jan 2019, at 05:38, Vijay Khemka wrote: > Memory-region and flash phandle is not a required parameter, it is > optional to describe in device tree and needed only use basis. > > Signed-off-by: Vijay Khemka <vijaykhemka@xxxxxx> > --- > Documentation/devicetree/bindings/mfd/aspeed-lpc.txt | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mfd/aspeed-lpc.txt b/ > Documentation/devicetree/bindings/mfd/aspeed-lpc.txt > index 34dd89087cff..ff0cb28903dd 100644 > --- a/Documentation/devicetree/bindings/mfd/aspeed-lpc.txt > +++ b/Documentation/devicetree/bindings/mfd/aspeed-lpc.txt > @@ -135,6 +135,10 @@ Required properties: > - clocks: contains a phandle to the syscon node describing the clocks. > There should then be one cell representing the clock to use > > +Optional properties: > +These below properties are optional and required only if one need to use it > +through ioctl. I'd just say "Optional properties". Leave the description off, as we probably shouldn't be talking about ioctls and such in the bindings. Andrew > + > - memory-region: A phandle to a reserved_memory region to be used for the LPC > to AHB mapping > > -- > 2.17.1 >
