[RFC PATCH 04/11] coresight: add CoreSight ETB driver
Mathieu Poirier
mathieu.poirier at linaro.org
Fri May 30 09:28:51 PDT 2014
On 30 May 2014 07:53, Russell King - ARM Linux <linux at arm.linux.org.uk> wrote:
> NAK for all the reasons I mentioned in the previous submission in 2012.
>
> You partially did the right thing - you read through the previous
> submission, and you said in your cover message that you had addressed
> some of the comments from that submission.
>
> What I find extremely distasteful is that you seem to have chosen to
> completely ignore my comments - you haven't mentioned them in your
> covering message at all, and you've just gone ahead and converted ETM
> and ETB to be platform devices.
>
> That gets you a NAK for that change, because you have done nothing what
> so ever to address the concerns I raised.
>From my initial reading of your assessment it wasn't clear to me that
your opinion leaned toward registering with the AMBA bus. Now that
this point has been clarified I will go back to the AMBA interface for
my next submission.
Is there anything else not AMBA releated that you'd like to see modified?
Thanks,
Mathieu
>
> Since you seem to have ignored my comments, this is as far as I'm looking
> at your submission, and you can consider the entire submission NAK'd by
> me.
>
> Thanks.
>
> --
> FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
> improving, and getting towards what was expected from it.
More information about the linux-arm-kernel
mailing list