<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META content="text/html; charset=utf-8" http-equiv=Content-Type><!-- flashmail style begin -->
<STYLE type=text/css>
body {border-width:0;margin:0}
img {border:0;margin:0;padding:0}
</STYLE>
<BASE target=_blank><!-- flashmail style end -->
<META name=GENERATOR content="MSHTML 11.00.10570.1001"></HEAD>
<BODY
style="BORDER-LEFT-WIDTH: 0px; FONT-SIZE: 10.5pt; FONT-FAMILY: ����������������; BORDER-RIGHT-WIDTH: 0px; BORDER-BOTTOM-WIDTH: 0px; COLOR: #000000; MARGIN: 12px; LINE-HEIGHT: 1.5; BORDER-TOP-WIDTH: 0px"
marginheight="0" marginwidth="0">
<DIV>Hi Brad,</DIV>
<DIV>Yes, eeprom driver could help the requirement partially.</DIV>
<DIV>As Ed's comments on the patch, we still need the size in the
upper level such as IPMI command handler, to differentiate the error between
writing failed and writing out of the range, and return the correct complete
code.</DIV>
<DIV><SPAN
style='FONT-SIZE: 13px; FONT-FAMILY: Roboto, -apple-system, BlinkMacSystemFont, "Segoe UI", Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol"; WHITE-SPACE: pre-wrap; WORD-SPACING: 0px; TEXT-TRANSFORM: none; FLOAT: none; FONT-WEIGHT: 400; COLOR: rgb(33,33,33); FONT-STYLE: normal; ORPHANS: 2; WIDOWS: 2; DISPLAY: inline !important; LETTER-SPACING: normal; BACKGROUND-COLOR: rgb(255,255,255); TEXT-INDENT: 0px; font-variant-ligatures: normal; font-variant-caps: normal; -webkit-text-stroke-width: 0px; text-decoration-style: initial; text-decoration-color: initial'>Thanks.</SPAN></DIV>
<DIV> </DIV>
<DIV style="FONT-SIZE: 10pt; FONT-FAMILY: Verdana; COLOR: #c0c0c0"
align=left>2020-11-26
<HR id=SignNameHR
style="BORDER-TOP: #c0c0c0 1px solid; HEIGHT: 1px; BORDER-RIGHT: 0px; WIDTH: 122px; BORDER-BOTTOM: 0px; BORDER-LEFT: 0px"
align=left>
<SPAN id=_FlashSignName>Helen.huang</SPAN> </DIV>
<HR
style="BORDER-TOP: #c0c0c0 1px solid; HEIGHT: 1px; BORDER-RIGHT: 0px; BORDER-BOTTOM: 0px; BORDER-LEFT: 0px">
<BLOCKQUOTE id=ntes-flashmail-quote
style="FONT-SIZE: 10pt; FONT-FAMILY: Verdana; PADDING-LEFT: 0px; MARGIN-LEFT: 0px">
<DIV><STRONG>发件人：</STRONG>Brad Bishop
<bradleyb@fuzziesquirrel.com></DIV>
<DIV><STRONG>发送时间：</STRONG>2020-11-25 23:36</DIV>
<DIV><STRONG>主题：</STRONG>Re: Add size property of EEPROM device</DIV>
<DIV><STRONG>收件人：</STRONG>"he.huang"<he.huang@linux.intel.com>,"ed"<ed@tanous.net>,"jae.hyun.yoo"<jae.hyun.yoo@linux.intel.com>,"vernon.mauery"<vernon.mauery@linux.intel.com>,"openbmc"<openbmc@lists.ozlabs.org></DIV>
<DIV><STRONG>抄送：</STRONG></DIV>
<DIV> </DIV>
<DIV>
<DIV>On Tue, 2020-11-24 at 15:31 +0800, he.huang wrote: </DIV>
<DIV>> The requirement comes from: </DIV>
<DIV>> When we write to the EEPROM we need the size to ensure that we won't </DIV>
<DIV>> write out of the range and return the correct complete code. </DIV>
<DIV> </DIV>
<DIV>Don't the eeprom drivers prevent this? </DIV></DIV></BLOCKQUOTE></BODY></HTML>