From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from comal.ext.ti.com (comal.ext.ti.com [198.47.26.152]) by arago-project.org (Postfix) with ESMTPS id 45D9752976 for ; Wed, 3 Sep 2014 18:12:05 +0000 (UTC) Received: from dlelxv90.itg.ti.com ([172.17.2.17]) by comal.ext.ti.com (8.13.7/8.13.7) with ESMTP id s83IC45J011865 for ; Wed, 3 Sep 2014 13:12:04 -0500 Received: from DLEE71.ent.ti.com (dlee71.ent.ti.com [157.170.170.114]) by dlelxv90.itg.ti.com (8.14.3/8.13.8) with ESMTP id s83IC4rK027820 for ; Wed, 3 Sep 2014 13:12:04 -0500 Received: from dflp33.itg.ti.com (10.64.6.16) by DLEE71.ent.ti.com (157.170.170.114) with Microsoft SMTP Server id 14.3.174.1; Wed, 3 Sep 2014 13:12:04 -0500 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by dflp33.itg.ti.com (8.14.3/8.13.8) with ESMTP id s83IC4NB017730; Wed, 3 Sep 2014 13:12:04 -0500 Date: Wed, 3 Sep 2014 14:12:03 -0400 From: Denys Dmytriyenko To: "R, Karthik" Message-ID: <20140903181203.GV18620@edge> References: <1409655283-4899-1-git-send-email-a0393906@ti.com> <8F29D6B095ED194EA1980491A5E029710C7262FB@DFLE08.ent.ti.com> <24B88602B5904F49971CAF899152529B80AEA8@DBDE04.ent.ti.com> MIME-Version: 1.0 In-Reply-To: <24B88602B5904F49971CAF899152529B80AEA8@DBDE04.ent.ti.com> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: "meta-arago@arago-project.org" , "Cooper Jr., Franklin" Subject: Re: [PATCH 00/12] Enabling WL8 R8.4 on dora X-BeenThere: meta-arago@arago-project.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Arago metadata layer for TI SDKs - OE-Core/Yocto compatible List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 03 Sep 2014 18:12:05 -0000 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Wed, Sep 03, 2014 at 12:46:11PM +0000, R, Karthik wrote: > Hi Franklin, > > Thanks for your inputs. I did some reading about these suggestions and I > fully understand your intent behind it. > I have regenerated the patches sent out the version 2 of this patch set for > review. Please review them if you find it in order. > If you need more details about the patches do let me know. > > I read a note in the git help that said that the patches generated by using > the options that make the differences minimal like the -D option will result > in patches that cannot be applied by git am. > So from a review point its fine, but I don't understand how it will get > merged into the repo? > Could you please help me understand the flow of the patches? Karthik, So, the recommendation is to use options -C, -M and --find-copies-harder (the last one applies to either -C or -M, if you don't want to specify their values directly) Please don't use -D option as it does indeed make patches un-applicable. E.g. your second set cannot be applies as is now. It may make patches with whole file being deleted smaller and easier to review though, but whole-file deletions are easy to see and skip anyway, so the small benefit doesn't outweigh the pain of not being able to apply the patch. The workflow I use is to review the patches on the list (prefer -C and -M to be used) and let our internal Patchwork capture everything. Then I apply the accepted patches directly from Patchwork the way they were submitted and Acked and push them in sets of small batches. The review process is usually given a few days on the list, unless the patch is critical and has to be pushed ASAP. -- Denys > -----Original Message----- > From: Cooper Jr., Franklin > Sent: Wednesday, September 03, 2014 3:17 AM > To: R, Karthik; meta-arago@arago-project.org > Subject: RE: [meta-arago] [PATCH 00/12] Enabling WL8 R8.4 on dora > > If you can resend using --find-copies-harder it will make it a lot easier to > review. You might also need to play with the -M option to make sure your > only sending as little deltas as possible. > > Also it would be nice in each of the patches if you add [for dora] to the > patch subject. I know you indicated that in the cover letter but it just > makes things easier and a good habit in case you don't always send cover > letters. > > I always use git send-email --annotate and manually add things like that to > each patch before I send it. A bit of a hassle for the sender but makes the > reviewer's life easier :) > > > -----Original Message----- > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago- > > bounces@arago-project.org] On Behalf Of R, Karthik > > Sent: Tuesday, September 02, 2014 5:55 AM > > To: meta-arago@arago-project.org > > Subject: [meta-arago] [PATCH 00/12] Enabling WL8 R8.4 on dora > > > > This patch set contains the set of patches that are required to get > > the latest Wilink firmware on the dora branch. > > Most of the patches here are cherry picked from the master branch. A > > few of them needed minor rework. > > > > Note that this has been tested on the GLSDK 6.10 release line for > > dra7xx-evm as the target. > > > > Denys Dmytriyenko (4): > > wl18xx-firmware: specify the branch being used for this version > > hostap-daemon: specify the branch being used for this version > > wpa-supplicant: specify the branch being used for this version > > recipes-connectivity: remove remaining unnecessary exec permissions > > > > Yaniv Machani (8): > > wpa-supplicant-wl18xx: Update recipe to ol_r8.a9.14 version. > > hostap-daemon-wl18xx: Update recipe to ol_r8.a9.14 version > > wl18xx : update to ol_r8.a9.14 > > Wl18xx : recipes update - tag changed to ol_r8.a9.14 > > wl18xx-target-scripts : Add new WiFi related scripts > > wl18xx-compat: version update > > wl18xx-wlconf: version update > > wpa-supplicant-wl18xx: configuration files update > > > > ...synchronization-between-bluetoothd-and-dr.patch | 0 > > .../0001-obexd-make-OPP-push-timeout-longer.patch | 0 > > .../packagegroup-arago-tisdk-connectivity.bb | 4 +- > > .../ti-compat-wireless-wl18xx_r8.a8.10.bb | 61 --- > > .../ti-compat-wireless-wl18xx_r8.a9.14.bb | 51 ++ > > .../wl18xx-firmware/wl18xx-firmware_r8.a8.10.bb | 30 -- > > .../wl18xx-firmware/wl18xx-firmware_r8.a9.14.bb | 31 ++ > > .../hostap/hostap-daemon-wl18xx_r8.a8.10.bb | 23 - > > .../hostap/hostap-daemon-wl18xx_r8.a9.14.bb | 24 + > > .../hostap/hostap-daemon/defconfig | 2 +- > > .../recipes-connectivity/hostap/hostap.inc | 7 +- > > .../wl18xx-target-scripts_r8.a9.14.bb | 25 + > > .../recipes-connectivity/wlconf/wlconf_r8.a8.10.bb | 41 -- > > .../recipes- connectivity/wlconf/wlconf_r8.a9.14.bb | 42 ++ > > .../wpa-supplicant/wpa- supplicant-wl18xx/defconfig | 506 ------------------- > > .../wpa-supplicant-wl18xx_r8.a8.10.bb | 24 - > > .../wpa-supplicant-wl18xx_r8.a9.14.bb | 25 + > > .../wpa-supplicant/wpa-supplicant.inc | 91 ++-- > > .../wpa-supplicant/wpa-supplicant/defaults-sane | 8 - > > .../wpa-supplicant/wpa-supplicant/defconfig | 543 > > +++++++++++++++++++++ > > .../fi.epitest.hostap.WPASupplicant.service | 4 - > > .../wpa-supplicant/fi.w1.wpa_supplicant1.service | 4 - > > .../wpa-supplicant/p2p_supplicant.conf | 3 +- > > .../wpa-supplicant/wpa_supplicant.conf | 3 +- > > .../ti-wifi-utils/ti-wifi-utils-wl18xx_r8.a8.10.bb | 31 -- > > .../ti-wifi-utils/ti-wifi- utils-wl18xx_r8.a9.14.bb | 31 ++ > > 26 files changed, 814 insertions(+), 800 deletions(-) mode change > > 100755 => > > 100644 meta-arago-distro/recipes-connectivity/bluez/bluez4/bluez4-fix- > > synchronization-between-bluetoothd-and-dr.patch > > mode change 100755 => 100644 meta-arago-distro/recipes- > > connectivity/obexd/obexd-0.34/0001-obexd-make-OPP-push-timeout- > > longer.patch > > mode change 100644 => 100755 meta-arago-distro/recipes- > > core/packagegroups/packagegroup-arago-tisdk-connectivity.bb > > delete mode 100644 > > meta-arago-extras/recipes-bsp/ti-compat-wireless/ti- > > compat-wireless-wl18xx_r8.a8.10.bb > > create mode 100644 > > meta-arago-extras/recipes-bsp/ti-compat-wireless/ti- > > compat-wireless-wl18xx_r8.a9.14.bb > > delete mode 100644 meta-arago-extras/recipes-bsp/wl18xx- > > firmware/wl18xx-firmware_r8.a8.10.bb > > create mode 100644 meta-arago-extras/recipes-bsp/wl18xx- > > firmware/wl18xx-firmware_r8.a9.14.bb > > delete mode 100644 meta-arago-extras/recipes- > > connectivity/hostap/hostap-daemon-wl18xx_r8.a8.10.bb > > create mode 100644 meta-arago-extras/recipes- > > connectivity/hostap/hostap-daemon-wl18xx_r8.a9.14.bb > > create mode 100644 meta-arago-extras/recipes-connectivity/wl18xx- > > target-scripts/wl18xx-target-scripts_r8.a9.14.bb > > delete mode 100644 meta-arago-extras/recipes- > > connectivity/wlconf/wlconf_r8.a8.10.bb > > create mode 100644 meta-arago-extras/recipes- > > connectivity/wlconf/wlconf_r8.a9.14.bb > > delete mode 100755 meta-arago-extras/recipes-connectivity/wpa- > > supplicant/wpa-supplicant-wl18xx/defconfig > > delete mode 100644 meta-arago-extras/recipes-connectivity/wpa- > > supplicant/wpa-supplicant-wl18xx_r8.a8.10.bb > > create mode 100644 meta-arago-extras/recipes-connectivity/wpa- > > supplicant/wpa-supplicant-wl18xx_r8.a9.14.bb > > delete mode 100644 meta-arago-extras/recipes-connectivity/wpa- > > supplicant/wpa-supplicant/defaults-sane > > create mode 100755 meta-arago-extras/recipes-connectivity/wpa- > > supplicant/wpa-supplicant/defconfig > > delete mode 100644 meta-arago-extras/recipes-connectivity/wpa- > > supplicant/wpa-supplicant/fi.epitest.hostap.WPASupplicant.service > > delete mode 100644 meta-arago-extras/recipes-connectivity/wpa- > > supplicant/wpa-supplicant/fi.w1.wpa_supplicant1.service > > delete mode 100644 > > meta-arago-extras/recipes-devtools/ti-wifi-utils/ti- > > wifi-utils-wl18xx_r8.a8.10.bb > > create mode 100644 > > meta-arago-extras/recipes-devtools/ti-wifi-utils/ti- > > wifi-utils-wl18xx_r8.a9.14.bb > > > > -- > > 2.0.1 > > > > _______________________________________________ > > meta-arago mailing list > > meta-arago@arago-project.org > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago > _______________________________________________ > meta-arago mailing list > meta-arago@arago-project.org > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago