Hi Vinod, ? 2016/8/5 11:34, Vinod Koul ??: > On Fri, Aug 05, 2016 at 10:53:20AM +0800, Shawn Lin wrote: >> This patch adds the "arm,pl330-periph-burst" for arm-pl330 to >> support busrt mode. > > why should this be DT property. Only reason I can think of if some hw > versions support this and some won't. yes, if we want to support burst mode, both of the master(pl330) and client(several peripherals) should implement it, otherwise it will be broken when enabling. So I mentioned it on my cover letter as the reason to introduce this optional property. If people add this property and find it's ok for their platform, they could land new dt-patch to add it. But we could *not* presume that we could get all users involved in testing this patchet. I don't wanna to break any other platforms, so it's needed. Thanks. > > If all are supporting, please enable it everywhere for obvious reasons. > -- Best Regards Shawn Lin