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 parsing should be done in kernelspace at all. If your patch is accepted then it is obvious that people think this belongs in kernelspace rather than in userspace. In addition I wonder what the rationale is behind placing the CID, SCR and CSD registers under /sysfs and EXT_CSD under debugfs. Not that you have chosen this, but do you know why they are accessible at different locations? > Maybe, there is some user-space application to do this job that I do not > know. :-( I have not seen such a program myself, however I have been asking my former colleagues to make such a program open source. I will keep you uptodate if I hear from them and they provide me with the source. >> BTW, you are parsing EXT_CSD here, but then one really >> should expand CSD, SCR, CID as well. One of those > Yes we could parse CID and CSD too. I guess in another patch. I agree. > Please consider this patch for the ext_csd entry in /sys. I guess Chris Ball is the one to convince here, not me. :) >> contains a numerical customer id which leads me to believe >> I did write an initial draft of such a userspace tool at my old >> employers over at ST-Ericsson and tried to open source it just >> before I resigned, but I don't know whether it has made it >> through the legal barrier yet. I'll let you know if I see it. > Yes let me know. I have been in contact with my former colleagues how promised to try to get it through legal. As it stands I can only hope... > Thanks for the reviewing. I'll look and fix all the points below and > resend the patch again. Cool, I noticed that you have already sent v2. :) / Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
