On Mon, Feb 6, 2017 at 6:20 AM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > On 02/01/2017 05:50 PM, Rob Herring wrote: >> On Mon, Jan 30, 2017 at 03:06:20PM +0100, Hans Verkuil wrote: >>> From: Hans Verkuil <hans.verkuil@xxxxxxxxx> >>> >>> Document the device tree bindings for this driver. >>> +isi: isi@f0034000 { >>> + compatible = "atmel,at91sam9g45-isi"; >>> + reg = <0xf0034000 0x4000>; >>> + interrupts = <37 IRQ_TYPE_LEVEL_HIGH 5>; >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&pinctrl_isi_data_0_7>; >>> + clocks = <&isi_clk>; >>> + clock-names = "isi_clk"; >>> + status = "ok"; >>> + port { >>> + #address-cells = <1>; >>> + #size-cells = <0>; >>> + isi_0: endpoint { >>> + reg = <0>; >> >> Drop reg. > > Is that because that is the default? Essentially, yes. You only need reg if you have more than one of something. Rob -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html