Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753495AbdG2JJ2 (ORCPT ); Sat, 29 Jul 2017 05:09:28 -0400 Received: from wp244.webpack.hosteurope.de ([80.237.133.13]:51492 "EHLO wp244.webpack.hosteurope.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753175AbdG2JJ0 (ORCPT ); Sat, 29 Jul 2017 05:09:26 -0400 Date: Sat, 29 Jul 2017 11:09:23 +0200 (CEST) From: Marcus Wolf To: Greg Kroah-Hartman Cc: Colin King , kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org Message-ID: <1370919182.11119.1501319363782@ox.hosteurope.de> In-Reply-To: <20170729000043.GA28803@kroah.com> References: <20170718130400.25436-1-colin.king@canonical.com> <20170718130400.25436-2-colin.king@canonical.com> <294e80854a10c63451cf3e9d7ca02bce-EhVcX1pHQwdXWkQFBhENSgEKLlwACzJXX19HAVhEWENbS1kLMF52CEtUX1pBSEwcXlJRL1lQWAhaW34EVVM=-webmailer1@server01.webmailer.webmailer.hosteurope.de> <20170729000043.GA28803@kroah.com> Subject: Re: [PATCH 1/1][staging-next] staging: pi433: Make functions rf69_set_dc_cut_off_frequency_intern static MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Priority: 3 Importance: Medium X-Mailer: Open-Xchange Mailer v7.8.3-Rev29 X-Originating-Client: com.openexchange.ox.gui.dhtml X-bounce-key: webpack.hosteurope.de;marcus.wolf@wolf-entwicklungen.de;1501319366;d36be4be; X-HE-SMSGID: 1dbNkR-0003vb-Sl Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 614 Lines: 18 Hi Greg, also had a very close look to this patch. Even in your reply I can't find any problems with line wraps or other corruptions :-/ But we have alternative patches, solving these problems as well. You e.g. could use the patch [PATCH] Make functions rf69_set_bandwidth_intern and rf69_set_dc_cut_off_frequency_intern static from Colin King 21/07/2017. It's doing exactly the same, my patch should have done. But be careful - tonight you added patch staging: pi433: Make functions rf69_set_bandwidth_intern static" Clins patch includes that changes as well! Once again sorry for my crappy patches, Marcus