RE: [PATCH v5 1/8] mtd: spi-nor: copy the SPI NOR commands to a new header file

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

 



Hi Marek, Shijie,

>From: Huang Shijie [mailto:b32955@xxxxxxxxxxxxx]
>
>于 2014年03月05日 11:43, Marek Vasut 写道:
>> Why didn't you keep it like that? Was there some reason for that ?
>http://marc.info/?l=linux-arm-kernel&m=138545182232220&w=4
>
>Pekon suggestted  do not touch the m25p80.c.
>So i just copy these commands to a new header.
>Anyway, it's really not an important issue for me. :)
>
My opinion was that we should not touch m25p80.c because:
(1)  As m25p80 is already working for various vendors. So unless multiple
   platforms start using SPI NOR framework, m25p80.c should remain untouched.
(2) we should not clutter header file with vendor specific op-codes as used in m25p80.
   I was of the opinion that instead of hard-coding the vendor specific info as
   MACROs, such information should be taken from DT or platform-data.

However, this was my thought during initial versions of SPI NOR patches.
But now if you think that SPI NOR is stable enough that same header info
can be reused without conflicts, then no problems.

(Apologies to Huang Shijie for re-work :-) ).

with regards, pekon
��.n��������+%������w��{.n�����{����*jg��������ݢj����G�������j:+v���w�m������w�������h�����٥





[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux