From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Hershberger Date: Wed, 20 Jan 2016 22:09:28 -0600 Subject: [U-Boot] [PATCH 1/2] checkpatch: ignore request to use ether_addr_copy() In-Reply-To: <20160121012241.GW3359@bill-the-cat> References: <1447430581-8805-1-git-send-email-swarren@wwwdotorg.org> <5670A37F.1020307@wwwdotorg.org> <56969621.30204@wwwdotorg.org> <569FF27B.5080606@wwwdotorg.org> <20160120210010.GZ3359@bill-the-cat> <569FF644.8040906@wwwdotorg.org> <20160121012241.GW3359@bill-the-cat> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Tom, On Wed, Jan 20, 2016 at 7:22 PM, Tom Rini wrote: > On Wed, Jan 20, 2016 at 02:04:04PM -0700, Stephen Warren wrote: >> On 01/20/2016 02:00 PM, Tom Rini wrote: >> >On Wed, Jan 20, 2016 at 01:47:55PM -0700, Stephen Warren wrote: >> >>On 01/13/2016 11:23 AM, Stephen Warren wrote: >> >>>On 12/15/2015 04:41 PM, Joe Hershberger wrote: >> >>>>Hi Stephen, >> >>>> >> >>>>On Tue, Dec 15, 2015 at 5:34 PM, Stephen Warren >> >>>> wrote: >> >>>>>On 11/23/2015 04:36 PM, Joe Hershberger wrote: >> >>>>>> >> >>>>>>On Fri, Nov 13, 2015 at 10:03 AM, Stephen Warren >> >>>>>> >> >>>>>>wrote: >> >>>>>>> >> >>>>>>>From: Stephen Warren >> >>>>>>> >> >>>>>>>The Linux kernel, from which checkpatch originates, contains function >> >>>>>>>ether_addr_copy() to copy Ethernet MAC addresses, and checkpatch warns >> >>>>>>>that it should be used in preference to memcpy() where appropriate. >> >>>>>>>U-Boot doesn't contain ether_addr_copy(), so tell checkpatch not to >> >>>>>>>issue >> >>>>>>>this warning. >> >>>>>>> >> >>>>>>>Signed-off-by: Stephen Warren >> >>>>>> >> >>>>>> >> >>>>>>Acked-by: Joe Hershberger >> >>>>> >> >>>>> >> >>>>>Were you intending someone else to apply this? I think now that Ted >> >>>>>Chen has >> >>>>>taken over the USB NIC driver (which was patch 2 in this series), >> >>>>>this patch >> >>>>>should be applied separately. >> >>>> >> >>>>It is assigned to Marek in patchwork. If he's not going to take it, >> >>>>then assign it to me and I'll pull it in. >> >>> >> >>>Marek, >> >>> >> >>>Are you taking this patch? If not, could you please re-assign it to Joe; >> >>>I don't believe I can modify patchwork myself. >> >> >> >>Marek? >> >> >> >>Tom, are you able to re-assign this patch to Joe in patchwork? >> > >> >Which? https://patchwork.ozlabs.org/patch/570547/ ? >> >> Just patch 1/2 not 2/2: >> >> https://patchwork.ozlabs.org/patch/544360/ >> checkpatch: ignore request to use ether_addr_copy() > > Ah, ok. I've taken 1/2 but if Joe beats me to it (patchwork assignments > are suggestions) that's fine too. All yours! -Joe