All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] iotrace examine buffer command
@ 2018-05-22  8:26 Ramon Fried
  2018-05-22 11:56 ` Ramon Fried
  0 siblings, 1 reply; 3+ messages in thread
From: Ramon Fried @ 2018-05-22  8:26 UTC (permalink / raw)
  To: u-boot

Hi Simon.
I know that you've mentioned in the commit log that md is enough,
but I find it hard to read and I can easily add a command for
examining the buffer.
Will you accept a patch like this ?

Also, I think it will also be beneficial to add a timestamp to each
entry. what do you think ?

Thanks,
Ramon

^ permalink raw reply	[flat|nested] 3+ messages in thread
* [U-Boot] iotrace examine buffer command
  2018-05-22  8:26 [U-Boot] iotrace examine buffer command Ramon Fried
@ 2018-05-22 11:56 ` Ramon Fried
  2018-05-22 23:29   ` Simon Glass
  0 siblings, 1 reply; 3+ messages in thread
From: Ramon Fried @ 2018-05-22 11:56 UTC (permalink / raw)
  To: u-boot

In Addition to what I wrote.
I think of another improvement. to be able to limit the iotracing to a
specific region of memory.

Thanks,
Ramon


On Tue, May 22, 2018 at 11:26 AM, Ramon Fried <ramon.fried@gmail.com> wrote:
> Hi Simon.
> I know that you've mentioned in the commit log that md is enough,
> but I find it hard to read and I can easily add a command for
> examining the buffer.
> Will you accept a patch like this ?
>
> Also, I think it will also be beneficial to add a timestamp to each
> entry. what do you think ?
>
> Thanks,
> Ramon

^ permalink raw reply	[flat|nested] 3+ messages in thread
* [U-Boot] iotrace examine buffer command
  2018-05-22 11:56 ` Ramon Fried
@ 2018-05-22 23:29   ` Simon Glass
  0 siblings, 0 replies; 3+ messages in thread
From: Simon Glass @ 2018-05-22 23:29 UTC (permalink / raw)
  To: u-boot

Hi Ramon,

On 22 May 2018 at 05:56, Ramon Fried <ramon.fried@gmail.com> wrote:
>
> In Addition to what I wrote.
> I think of another improvement. to be able to limit the iotracing to a
> specific region of memory.

Both of these seem fine to me.

Regards,
Simon

>
> Thanks,
> Ramon
>
>
> On Tue, May 22, 2018 at 11:26 AM, Ramon Fried <ramon.fried@gmail.com> wrote:
> > Hi Simon.
> > I know that you've mentioned in the commit log that md is enough,
> > but I find it hard to read and I can easily add a command for
> > examining the buffer.
> > Will you accept a patch like this ?
> >
> > Also, I think it will also be beneficial to add a timestamp to each
> > entry. what do you think ?
> >
> > Thanks,
> > Ramon

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-05-22 23:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-22  8:26 [U-Boot] iotrace examine buffer command Ramon Fried
2018-05-22 11:56 ` Ramon Fried
2018-05-22 23:29   ` Simon Glass

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.