Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1456911

Re: [PATCH 1/3] dt/bindings: arm-pl330: add description of arm,pl330-periph-burst

Show all headers | View raw

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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/3] dt/bindings: arm-pl330: add description of arm,pl330-periph-burst Shawn Lin <shawn.lin@rock-chips.com> - 2016-08-05 05:00 +0200
  Re: [PATCH 1/3] dt/bindings: arm-pl330: add description of  arm,pl330-periph-burst Vinod Koul <vinod.koul@intel.com> - 2016-08-05 05:30 +0200
    Re: [PATCH 1/3] dt/bindings: arm-pl330: add description of  arm,pl330-periph-burst Shawn Lin <shawn.lin@rock-chips.com> - 2016-08-05 09:30 +0200
      Re: [PATCH 1/3] dt/bindings: arm-pl330: add description of  arm,pl330-periph-burst Lars-Peter Clausen <lars@metafoo.de> - 2016-08-09 10:50 +0200
        Re: [PATCH 1/3] dt/bindings: arm-pl330: add description of  arm,pl330-periph-burst Shawn Lin <shawn.lin@rock-chips.com> - 2016-08-09 11:20 +0200

csiph-web