[U-Boot] [PATCH] Add clarifications to the x86 README
Stoppa, Igor
igor.stoppa at intel.com
Wed Aug 12 15:40:01 CEST 2015
Hi Bin,
On 12 August 2015 at 16:27, Bin Meng <bmeng.cn at gmail.com> wrote:
> Hi Igor,
>
> On Wed, Aug 12, 2015 at 8:20 PM, Igor Stoppa <igor.stoppa at intel.com> wrote:
>> Explicitly list the targets supported in each section of the instructions
>> from the x86 README.
>>
>
> Nits: we should put tags in the patch/commit title, eg:
>
> x86: Add clarifications to the x86 README
ok, will fix it
[...]
>> -Intel Crown Bay specific instructions:
>> +Intel Crown Bay specific instructions (raw mode):
>
> I think we don't need add (raw mode) as you already added a section
> header above.
This was intentional. You are absolutely right that it's redundant,
but I was trying to make the doc more friendly toward someone (like
yours truly :-) who approaches it for the first time.
Reading it, sometimes I had the feeling I wasn't 100% sure of what a
certain section was referring to.
Some sections are not exactly short.
But if you still think it should be removed, I'll do so.
--
igor
More information about the U-Boot
mailing list