<font face="Default Sans Serif,Verdana,Arial,Helvetica,sans-serif" size="2"><br><br><font color="#990099">-----"openbmc" <<a target="_blank" href="mailto:openbmc-bounces+shliyi=cn.ibm.com@lists.ozlabs.org">openbmc-bounces+shliyi=cn.ibm.com@lists.ozlabs.org</a>> wrote: -----</font><br><br>>To: Patrick Williams <<a target="_blank" href="mailto:patrick@stwcx.xyz">patrick@stwcx.xyz</a>><br>>From: "Adriana Kobylak" <br>>Sent by: "openbmc" <br>>Date: 03/02/2016 06:56AM<br>>Cc: OpenBMC Maillist <<a target="_blank" href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>>, OpenBMC Patches<br>><<a target="_blank" href="mailto:openbmc-patches@stwcx.xyz">openbmc-patches@stwcx.xyz</a>><br>>Subject: Re: [PATCH openbmc] Add temporary adm1278 hwmon pmbus driver<br>>patch<br>><br>> <br>>Patrick Williams <<a target="_blank" href="mailto:patrick@stwcx.xyz">patrick@stwcx.xyz</a>> wrote on 03/01/2016 04:47:03 PM:<br>><br>>> From: Patrick Williams <<a target="_blank" href="mailto:patrick@stwcx.xyz">patrick@stwcx.xyz</a>><br>>> To: Adriana Kobylak/Austin/IBM@IBMUS<br>>> Cc: Joel Stanley <<a target="_blank" href="mailto:joel@jms.id.au">joel@jms.id.au</a>>, OpenBMC Patches <openbmc-<br>>> <a target="_blank" href="mailto:patches@stwcx.xyz">patches@stwcx.xyz</a>>, OpenBMC Maillist <<a target="_blank" href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>><br>>> Date: 03/01/2016 04:47 PM<br>>> Subject: Re: [PATCH openbmc] Add temporary adm1278 hwmon pmbus<br>>driver patch<br>>> <br>>> On Tue, Mar 01, 2016 at 03:15:12PM -0600, Adriana Kobylak wrote:<br>>> > What makes this pull request different from other openbmc changes<br>>to <br>>> > require to be sent via git-send-email before it can be reviewed?<br>>> <br>>> I suspect Joel is just asking for it to also be submitted to the<br>>Linux<br>>> tree so that its content can be reviewed there.  I agree we should<br>>not<br>>> merge patches into the openbmc tree unless it fixes a critical<br>>problem<br>>> and has also been submitted to the upstream tree.<br>>> <br>>Yi, please submit the patch to the kernel repository as well.<br>><br>>There's a requirement to deliver this driver on Friday, could we get<br>>this reviewed/merged this week in openbmc? I believe we did this<br>>before for OCC then removed the temporary patch once it was<br>>upstreamed.<br><br>OK. I will fix the kernel-coding style issue then submit path against linux kernel tree.<br><br>>> > <br>>> > <br>>> > <br>>> > <br>>> > <br><br></font><BR>