[flashrom] [PATCH] Add SyncMOS / Mosel Vitelic chip definitions (try 2)

[Carl-Daniel Hailfinger]

On 29.07.2010 20:23, Uwe Hermann wrote:
> On Thu, Jul 29, 2010 at 07:11:36PM +0200, Carl-Daniel Hailfinger wrote:
>   
>> On 14.07.2010 06:44, Mattias Mattsson wrote:
>>     
>>> Mosel Vitelic Corporation (MVC):
>>>
>>> Signed-off-by: Mattias Mattsson <vitplister at gmail.com>  
>>>       
>> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>>     
>
> Acked-by: Uwe Hermann <uwe at hermann-uwe.de>
>
>
> Please use both Signed-off-by's in the commit.
>   

Done. Thanks for the review, committed in r1124.


>> +#define SYNCMOS_MVC_ID		0x40	/* SyncMOS (SM) and Mosel Vitelic Corporation (MVC) */
>>     
>
> We should change this to MVI or something else in a follow-up commit.
> I cannot find MVC used in any datasheet or the vendor website. On their
> website they use the MVI abbreviation though (Mosel Vitelic Inc.).
>   

MVC is Mosel Vitelic Corporation.

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/