On Wed, 2018-08-22 at 14:45 +0300, Kalle Valo wrote: > Luca Coelho <luca@xxxxxxxxx> writes: > > > From: Mordechay Goodstein <mordechay.goodstein@xxxxxxxxx> > > > > Every rx mpdu cmd is built from cmd_hdr | 80211_hdr. The problem > > is > > that the size of cmd_hdr changes with API changes and we don't know > > where the 80211_hdr starts. > > > > By adding the size of cmd_hdr dynamically, we can ensure that we > > always > > know how to parse mpdu frames, without dependending on the API > > changes. > > > > Signed-off-by: Mordechay Goodstein <mordechay.goodstein@xxxxxxxxx> > > Signed-off-by: Luca Coelho <luciano.coelho@xxxxxxxxx> > > [...] > > > --- a/drivers/net/wireless/intel/iwlwifi/iwl-devtrace-data.h > > +++ b/drivers/net/wireless/intel/iwlwifi/iwl-devtrace-data.h > > @@ -2,6 +2,7 @@ > > * > > * Copyright(c) 2009 - 2014 Intel Corporation. All rights > > reserved. > > * Copyright(c) 2015 Intel Deutschland GmbH > > + * Copyright(c) 2018 Intel Corporation > > * > > * This program is free software; you can redistribute it and/or > > modify it > > * under the terms of version 2 of the GNU General Public License > > as > > @@ -12,10 +13,6 @@ > > * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public > > License for > > * more details. > > * > > - * You should have received a copy of the GNU General Public > > License along with > > - * this program; if not, write to the Free Software Foundation, > > Inc., > > - * 51 Franklin Street, Fifth Floor, Boston, MA 02110, USA > > - * > > * The full GNU General Public License is included in this > > distribution in the > > * file called LICENSE. > > * > > Ok, you have more of these removals in this patch so I guess it's > intentional. But IMHO it would be nicer to remove these all in one go > instead of sprinkle around in different patches. IMHO it's too much fuzz for mostly useless warning removal... We need to update the copyright when this hits, so we just do it at the same time. But you're the boss, I can submit a patch to remove just that, all at once. -- Luca.
