[flashrom] [PATCH] Disallow the internal programmer on non-x86 for now

Vladimir 'φ-coder/phcoder' Serbinenko phcoder at gmail.com
Wed May 26 13:22:52 CEST 2010


Carl-Daniel Hailfinger wrote:
> On 26.05.2010 08:22, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
>   
>> Carl-Daniel Hailfinger wrote:
>>   
>>     
>>> The internal programmer needs correct information about flash_base and
>>> chip window top/bottom alignment on non-x86 before it can be used. Abort
>>> any internal programmer action for now until the code is fixed.
>>>
>>> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>>>
>>> +#else
>>> +	/* FIXME: Remove this unconditional abort once all PCI drivers are
>>> +	 * converted to use little-endian accesses for memory BARs.
>>> +	 */
>>> +	msg_perr("Your platform is not supported yet for the internal "
>>> +		 "programmer because it has not been converted from native "
>>> +		 "endian to little endian access yet.\n"
>>> +		 "Aborting.\n");
>>>   
>>>     
>>>       
>> Should have been "external".
>>   
>>     
>
> Are you sure? This warning is part of the internal (main onboard flash)
> programmer.
> Although this warning should be added for external PCI-based
> programmers, such a change is not part of this patch because I want to
> fix them instead of marking them as broken.
>
>   
Then I misread the code. Sorry for noise.
> Regards,
> Carl-Daniel
>
>   


-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 293 bytes
Desc: OpenPGP digital signature
URL: <http://www.flashrom.org/pipermail/flashrom/attachments/20100526/33d2249d/attachment.asc>


More information about the flashrom mailing list