Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-12-15 Thread Giuseppe CAVALLARO
Hi Chris On 12/13/2011 5:35 PM, Chris Ball wrote: > Hi Peppe, > > On Tue, Dec 06 2011, Giuseppe CAVALLARO wrote: >>> I'm all for this patch so it gets visible in debugfs atleast, >>> since people seem to need to know these things, so take that as an >>> Acked-by: Linus Walleij >>> >>> Those who

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-12-13 Thread Chris Ball
Hi Peppe, On Tue, Dec 06 2011, Giuseppe CAVALLARO wrote: >> I'm all for this patch so it gets visible in debugfs atleast, >> since people seem to need to know these things, so take that as an >> Acked-by: Linus Walleij >> >> Those who want to writ userspace tools can proceed to >> implement the

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-12-05 Thread Giuseppe CAVALLARO
On 11/21/2011 11:08 AM, Linus Walleij wrote: > On Mon, Nov 21, 2011 at 8:39 AM, Giuseppe CAVALLARO > wrote: > >> Just to summarize: the patch works in debugfs context so the parsing doesn't >> break the sys rules. I agree with Chris this has to be maintained for the >> next mmc standards but IMO

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-11-21 Thread Linus Walleij
On Mon, Nov 21, 2011 at 8:39 AM, Giuseppe CAVALLARO wrote: > Just to summarize: the patch works in debugfs context so the parsing doesn't > break the sys rules. I agree with Chris this has to be maintained for the > next mmc standards but IMO we can pay this waiting for having both a > user-space

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-11-20 Thread Giuseppe CAVALLARO
Hello Linus, Chris, On 11/12/2011 2:12 AM, Linus Walleij wrote: On Tue, Oct 25, 2011 at 1:43 PM, Sebastian Rasmussen wrote: [Giuseppe] This patch enhances the debug information reported for the mmc card by parsing the extended CSD registers obviously according to all the current specification

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-11-12 Thread Linus Walleij
On Sat, Nov 12, 2011 at 2:21 AM, Chris Ball wrote: > I wonder if we should start an MMC userspace tools repository containing > code for features like the cmd passthrough ioctl, secure erase/trim/discard > requests, and debugging output of the type this patch provides.  On the > other hand, maybe

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-11-11 Thread Chris Ball
Hi Giuseppe, On Fri, Oct 28 2011, Giuseppe CAVALLARO wrote: > On 10/27/2011 4:19 PM, Sebastian Rasmussen wrote: >> Hi! >> >>> Before the patch the output from /sys/kernel/debug/mmc0/mmc0:0001/ext_csd >>> was a big number and I had many problems parsing it in real-time. >>> [...] >>> In any case,

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-11-11 Thread Linus Walleij
On Tue, Oct 25, 2011 at 1:43 PM, Sebastian Rasmussen wrote: >[Giuseppe] >> This patch enhances the debug information reported >> for the mmc card by parsing the extended CSD registers >> obviously according to all the current specifications. > > Does this belong kernel or in userspace? I'm not sur

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-10-28 Thread Giuseppe CAVALLARO
Hello Sebastian, On 10/27/2011 4:19 PM, Sebastian Rasmussen wrote: > Hi! > >> Before the patch the output from /sys/kernel/debug/mmc0/mmc0:0001/ext_csd >> was a big number and I had many problems parsing it in real-time. >> [...] >> In any case, the patch wants to help to directly get the ext_csd

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-10-27 Thread Sebastian Rasmussen
Hi! > Before the patch the output from /sys/kernel/debug/mmc0/mmc0:0001/ext_csd > was a big number and I had many problems parsing it in real-time. >[...] > In any case, the patch wants to help to directly get the ext_csd in a > human format. Yes, I realize this. However I'm unsure whether this p

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-10-27 Thread Giuseppe CAVALLARO
On 10/26/2011 1:27 PM, Giuseppe CAVALLARO wrote: > On 10/25/2011 1:43 PM, Sebastian Rasmussen wrote: [snip] >> contains a numerical customer id which leads me to believe >> that it would require some kind of list of number->name to >> make it accessible to users, something akin to the lspci >> da

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-10-26 Thread Giuseppe CAVALLARO
On 10/25/2011 1:43 PM, Sebastian Rasmussen wrote: >> This patch enhances the debug information reported >> for the mmc card by parsing the extended CSD registers >> obviously according to all the current specifications. > > Does this belong kernel or in userspace? I'm not sure, and > I'm hoping th

Re: [PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-10-25 Thread Sebastian Rasmussen
> This patch enhances the debug information reported > for the mmc card by parsing the extended CSD registers > obviously according to all the current specifications. Does this belong kernel or in userspace? I'm not sure, and I'm hoping that any of the old-timers here chime in on it. Anyway I supp

[PATCH] mmc: debugfs: parse all ext_csd via debug_fs

2011-10-24 Thread Giuseppe CAVALLARO
This patch enhances the debug information reported for the mmc card by parsing the extended CSD registers obviously according to all the current specifications. I have no HW to test eMMC 4.5 at this moment. In any case, the patch supports JEDEC Standard No. 84-B45. No issues on JESD84-A441 and old