Re: [PATCH] printk-formats.txt documentation update

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks for the feedback - I will truncate to 80 characters and
implement Joe's advice.

Thanks,

Andrew Murray

On 6 February 2011 00:37, Joe Perches <joe@xxxxxxxxxxx> wrote:
> On Sun, 2011-02-06 at 00:15 +0000, Andrew Murray wrote:
>> From: Andrew Murray <amurray@xxxxxxxxxxxxxx>
>> This patch updates the incomplete documentation concerning the printk
>> extended format specifiers.
>> Signed-off-by: Andrew Murray <amurray@xxxxxxxxxxxxxx>
>> ---
>> diff --git a/Documentation/printk-formats.txt b/Documentation/printk-formats.txt
> []
>> @@ -9,7 +9,42 @@ If variable is of Type, Â Â Â Â Â Â Âuse printk format specifier:
>>        size_t         Â%zu or %zx
>>        ssize_t         %zd or %zx
>>
>> -Raw pointer value SHOULD be printed with %p.
>> +Raw pointer value SHOULD be printed with %p. The kernel supports
>> +the following extended format specifiers for pointer types:
>
> I think it'd be better if each extension were individually described
> with an example output.
>
>> +
>> + Â Â General:
>> +
>> + Â Â %pF Â Â function pointer with offset, e.g. module_start
>> +   %pf   function pointer without offset, e.g. module_start+0x0/0x62 [hello]
>
> with/without inversion?
>
>> + Â Â %pS Â Â text symbol with offset
>> +   %ps   text symbol without offset
>> + Â Â %pR Â Â struct resource with decoded flags, e.g. [mem 0x0-0x1f 64 bit pref]
>> +   %pr   struct resource with raw flags, e.g. [mem 0x0-0x1f flags 0x201]
>> +
>> + Â Â MAC/FDDI addresses:
>> +
>> + Â Â %pM Â Â colon-seperated 6-byte MAC address in hex notation, e.g. 00:01:02:03:04:05
>
> spelling: separated
>
>> +   %pm   non-colon-seperated 6-byte MAC address is hex notation, e.g. 000102030405
>> + Â Â %pMF Â Â6-byte dash-separated FDDI hex notation
>> +
>> + Â Â IPv4 addresses:
>> +
>> + Â Â %pI4 Â ÂIPv4 dot-seperated decimal without leading 0's, e.g. 1.2.3.4
>> + Â Â %pi4 Â ÂIPv4 dot-seperated decimal with leading 0's, 001.002.003.004
>> + Â Â %p[Ii]4[hl] IPv4 dot-seperated decimal with/without leading 0's for
>> + Â Â Â Â Â Â host (h), little (l) or network/big (default) endian order
>> +
>> + Â Â IPv6 addresses:
>> + Â Â %pI6 Â ÂIPv6 colon-seperated network-order 16 bit hex with leading 0's,
>> + Â Â Â Â Â Â e.g. 0001:0203:...:0708
>> + Â Â %pi6 Â ÂIPv6 non-colon-seperated network-order 16 bit hex with leading 0's,
>> + Â Â Â Â Â Â e.g. 000102...0f
>> +   %pI6c  IPv6 address as described by
>> + Â Â Â Â Â Â http://tools.ietf.org/html/draft-ietf-6man-text-addr-representation-00,
>> + Â Â Â Â Â Â e.g. 1::708
>> +
>> + Â Â %pU[bBlL] 16 byte UUID/GUID in big or little endian using lower or upper case
>
> Maybe a simple 2 line table without the e.g.:
>
> Â Â Â Â%p<extension1> Âsample output1
> Â Â Â Â Â Â Â Â Â Â Â Âdescription1
> Â Â Â Â%p<extension2> Âsample output2
> Â Â Â Â Â Â Â Â Â Â Â Âdescription2
>
>
>



-- 
Andrew Murray, Embedded Linux Group
MPC Data Limited
e-mail: amurray@xxxxxxxxxxxxxx web: www.mpc-data.co.uk
tel: +44 (0) 1225 710600Â Â Â Â Â Â ÂÂ fax: +44 (0) 1225 710601
ddi: +44 (0) 1225 710665

MPC Data Limited is a company registered in England and Wales with
company number 05507446
Registered Address: County Gate, County Way, Trowbridge, Wiltshire, BA14 7FJ
VAT no: 850625238

The information in this email and in the attached documents is
confidential and may be
legally privileged. Any unauthorized review, copying, disclosure or
distribution is
prohibited and may be unlawful. It is intended solely for the
addressee. Access to this
email by anyone else is unauthorized. If you are not the intended
recipient, please
contact the sender by reply email and destroy all copies of the
original message. When
addressed to our clients any opinions or advice contained in this
email is subject to
the terms and conditions expressed in the governing contract.
--
To unsubscribe from this list: send the line "unsubscribe linux-embedded" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Gstreamer Embedded]     [Linux MMC Devel]     [U-Boot V2]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux ARM Kernel]     [Linux OMAP]     [Linux SCSI]

  Powered by Linux