[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20220830083027.4iyi2s5r5bto4h2e@houat>
Date: Tue, 30 Aug 2022 10:30:27 +0200
From: Maxime Ripard <maxime@...no.tech>
To: Konstantin Ryabitsev <konstantin@...uxfoundation.org>
Cc: Noralf Trønnes <noralf@...nnes.org>,
Ben Skeggs <bskeggs@...hat.com>,
David Airlie <airlied@...ux.ie>, Chen-Yu Tsai <wens@...e.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
Jani Nikula <jani.nikula@...ux.intel.com>,
Lyude Paul <lyude@...hat.com>,
Philipp Zabel <p.zabel@...gutronix.de>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Rodrigo Vivi <rodrigo.vivi@...el.com>,
Tvrtko Ursulin <tvrtko.ursulin@...ux.intel.com>,
Jernej Skrabec <jernej.skrabec@...il.com>,
Samuel Holland <samuel@...lland.org>,
Karol Herbst <kherbst@...hat.com>,
Emma Anholt <emma@...olt.net>, Daniel Vetter <daniel@...ll.ch>,
Joonas Lahtinen <joonas.lahtinen@...ux.intel.com>,
Hans de Goede <hdegoede@...hat.com>,
linux-arm-kernel@...ts.infradead.org,
Phil Elwell <phil@...pberrypi.com>,
intel-gfx@...ts.freedesktop.org,
Dave Stevenson <dave.stevenson@...pberrypi.com>,
dri-devel@...ts.freedesktop.org, Dom Cobley <dom@...pberrypi.com>,
linux-kernel@...r.kernel.org, nouveau@...ts.freedesktop.org,
linux-sunxi@...ts.linux.dev,
Mateusz Kwiatkowski <kfyatek+publicgit@...il.com>,
Geert Uytterhoeven <geert@...ux-m68k.org>
Subject: Re: [PATCH v2 01/41] drm/tests: Order Kunit tests in Makefile
Hi Konstantin,
On Mon, Aug 29, 2022 at 03:02:42PM -0400, Konstantin Ryabitsev wrote:
> On Mon, Aug 29, 2022 at 08:46:42PM +0200, Noralf Trønnes wrote:
> > Something has gone wrong with this patchset, there are double line endings.
>
> I noticed this, too, and I think the reason is because these patches were
> generated with "b4 send -o", but actually sent using git-send-email. It's not
> a use-case I've considered (or tested) and the breakage is because when b4
> generates patches with "-o", they are written with CRLF line endings, which is
> not something git-send-email expects.
You're right
> Maxime, any reason you went this direction instead of just letting b4 send
> these patches directly?
For some reason, b4 send doesn't work for my setup and I haven't had the
time to look into why and let you know.
Whenever I used b4 send (on this series for example) I get:
$ b4 send
Converted the branch to 41 patches
Populating the To: and Cc: fields with automatically collected addresses
Invoking get_maintainer.pl for To: addresses
Invoking get_maintainer.pl for Cc: addresses
Will send the following messages:
---
To: David Airlie <airlied@...ux.ie>
Daniel Vetter <daniel@...ll.ch>
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>
Maxime Ripard <mripard@...nel.org>
Thomas Zimmermann <tzimmermann@...e.de>
Noralf Trønnes <noralf@...nnes.org>
Emma Anholt <emma@...olt.net>
Ben Skeggs <bskeggs@...hat.com>
Karol Herbst <kherbst@...hat.com>
Lyude Paul <lyude@...hat.com>
Jani Nikula <jani.nikula@...ux.intel.com>
Joonas Lahtinen <joonas.lahtinen@...ux.intel.com>
Rodrigo Vivi <rodrigo.vivi@...el.com>
Tvrtko Ursulin <tvrtko.ursulin@...ux.intel.com>
Chen-Yu Tsai <wens@...e.org>
Jernej Skrabec <jernej.skrabec@...il.com>
Samuel Holland <samuel@...lland.org>
Philipp Zabel <p.zabel@...gutronix.de>
Cc: dri-devel@...ts.freedesktop.org
linux-kernel@...r.kernel.org
nouveau@...ts.freedesktop.org
intel-gfx@...ts.freedesktop.org
linux-arm-kernel@...ts.infradead.org
linux-sunxi@...ts.linux.dev
Dom Cobley <dom@...pberrypi.com>
Maxime Ripard <maxime@...no.tech>
Geert Uytterhoeven <geert@...ux-m68k.org>
Hans de Goede <hdegoede@...hat.com>
Dave Stevenson <dave.stevenson@...pberrypi.com>
Mateusz Kwiatkowski <kfyatek+publicgit@...il.com>
Phil Elwell <phil@...pberrypi.com>
---
[PATCH v2 00/41] drm: Analog TV Improvements
[PATCH v2 01/41] drm/tests: Order Kunit tests in Makefile
[PATCH v2 02/41] drm/tests: Add Kunit Helpers
[PATCH v2 03/41] drm/atomic-helper: Rename drm_atomic_helper_connector_tv_reset to avoid ambiguity
[PATCH v2 04/41] drm/connector: Rename subconnector state variable
[PATCH v2 05/41] drm/atomic: Add TV subconnector property to get/set_property
[PATCH v2 06/41] drm/connector: Rename legacy TV property
[PATCH v2 07/41] drm/connector: Only register TV mode property if present
[PATCH v2 08/41] drm/connector: Rename drm_mode_create_tv_properties
[PATCH v2 09/41] drm/connector: Add TV standard property
[PATCH v2 10/41] drm/modes: Add a function to generate analog display modes
[PATCH v2 11/41] drm/modes: Only consider bpp and refresh before options
[PATCH v2 12/41] drm/modes: parse_cmdline: Add support for named modes containing dashes
[PATCH v2 13/41] drm/client: Add some tests for drm_connector_pick_cmdline_mode()
[PATCH v2 14/41] drm/modes: Move named modes parsing to a separate function
[PATCH v2 15/41] drm/modes: Switch to named mode descriptors
[PATCH v2 16/41] drm/modes: Fill drm_cmdline mode from named modes
[PATCH v2 17/41] drm/connector: Add pixel clock to cmdline mode
[PATCH v2 18/41] drm/connector: Add a function to lookup a TV mode by its name
[PATCH v2 19/41] drm/modes: Introduce the tv_mode property as a command-line option
[PATCH v2 20/41] drm/modes: Properly generate a drm_display_mode from a named mode
[PATCH v2 21/41] drm/modes: Introduce more named modes
[PATCH v2 22/41] drm/atomic-helper: Add a TV properties reset helper
[PATCH v2 23/41] drm/atomic-helper: Add an analog TV atomic_check implementation
[PATCH v2 24/41] drm/vc4: vec: Remove empty mode_fixup
[PATCH v2 25/41] drm/vc4: vec: Convert to atomic helpers
[PATCH v2 26/41] drm/vc4: vec: Refactor VEC TV mode setting
[PATCH v2 27/41] drm/vc4: vec: Remove redundant atomic_mode_set
[PATCH v2 28/41] drm/vc4: vec: Fix timings for VEC modes
[PATCH v2 29/41] drm/vc4: vec: Switch for common modes
[PATCH v2 30/41] drm/vc4: vec: Fix definition of PAL-M mode
[PATCH v2 31/41] drm/vc4: vec: Use TV Reset implementation
[PATCH v2 32/41] drm/vc4: vec: Convert to the new TV mode property
[PATCH v2 33/41] drm/vc4: vec: Add support for more analog TV standards
[PATCH v2 34/41] drm/sun4i: tv: Remove unused mode_valid
[PATCH v2 35/41] drm/sun4i: tv: Convert to atomic hooks
[PATCH v2 36/41] drm/sun4i: tv: Merge mode_set into atomic_enable
[PATCH v2 37/41] drm/sun4i: tv: Remove useless function
[PATCH v2 38/41] drm/sun4i: tv: Remove useless destroy function
[PATCH v2 39/41] drm/sun4i: tv: Rename error label
[PATCH v2 40/41] drm/sun4i: tv: Add missing reset assertion
[PATCH v2 41/41] drm/sun4i: tv: Convert to the new TV mode property
---
Press Enter to send or Ctrl-C to abort
[PATCH v2 00/41] drm: Analog TV Improvements
[PATCH v2 01/41] drm/tests: Order Kunit tests in Makefile
[PATCH v2 02/41] drm/tests: Add Kunit Helpers
[PATCH v2 03/41] drm/atomic-helper: Rename drm_atomic_helper_connector_tv_reset to avoid ambiguity
[PATCH v2 04/41] drm/connector: Rename subconnector state variable
[PATCH v2 05/41] drm/atomic: Add TV subconnector property to get/set_property
[PATCH v2 06/41] drm/connector: Rename legacy TV property
[PATCH v2 07/41] drm/connector: Only register TV mode property if present
[PATCH v2 08/41] drm/connector: Rename drm_mode_create_tv_properties
[PATCH v2 09/41] drm/connector: Add TV standard property
[PATCH v2 10/41] drm/modes: Add a function to generate analog display modes
[PATCH v2 11/41] drm/modes: Only consider bpp and refresh before options
[PATCH v2 12/41] drm/modes: parse_cmdline: Add support for named modes containing dashes
[PATCH v2 13/41] drm/client: Add some tests for drm_connector_pick_cmdline_mode()
[PATCH v2 14/41] drm/modes: Move named modes parsing to a separate function
[PATCH v2 15/41] drm/modes: Switch to named mode descriptors
[PATCH v2 16/41] drm/modes: Fill drm_cmdline mode from named modes
[PATCH v2 17/41] drm/connector: Add pixel clock to cmdline mode
[PATCH v2 18/41] drm/connector: Add a function to lookup a TV mode by its name
[PATCH v2 19/41] drm/modes: Introduce the tv_mode property as a command-line option
[PATCH v2 20/41] drm/modes: Properly generate a drm_display_mode from a named mode
[PATCH v2 21/41] drm/modes: Introduce more named modes
[PATCH v2 22/41] drm/atomic-helper: Add a TV properties reset helper
[PATCH v2 23/41] drm/atomic-helper: Add an analog TV atomic_check implementation
[PATCH v2 24/41] drm/vc4: vec: Remove empty mode_fixup
[PATCH v2 25/41] drm/vc4: vec: Convert to atomic helpers
[PATCH v2 26/41] drm/vc4: vec: Refactor VEC TV mode setting
[PATCH v2 27/41] drm/vc4: vec: Remove redundant atomic_mode_set
[PATCH v2 28/41] drm/vc4: vec: Fix timings for VEC modes
[PATCH v2 29/41] drm/vc4: vec: Switch for common modes
[PATCH v2 30/41] drm/vc4: vec: Fix definition of PAL-M mode
[PATCH v2 31/41] drm/vc4: vec: Use TV Reset implementation
[PATCH v2 32/41] drm/vc4: vec: Convert to the new TV mode property
[PATCH v2 33/41] drm/vc4: vec: Add support for more analog TV standards
[PATCH v2 34/41] drm/sun4i: tv: Remove unused mode_valid
[PATCH v2 35/41] drm/sun4i: tv: Convert to atomic hooks
[PATCH v2 36/41] drm/sun4i: tv: Merge mode_set into atomic_enable
[PATCH v2 37/41] drm/sun4i: tv: Remove useless function
[PATCH v2 38/41] drm/sun4i: tv: Remove useless destroy function
[PATCH v2 39/41] drm/sun4i: tv: Rename error label
[PATCH v2 40/41] drm/sun4i: tv: Add missing reset assertion
[PATCH v2 41/41] drm/sun4i: tv: Convert to the new TV mode property
Connecting to /home/max/.local/bin/msmtpq:0
Failed to configure the smtp connection:
[Errno -2] Name or service not known
msmtpq is basically a sendmail-like tool to queue the mails if the
machine is offline. It is setup in my git config as
sendemail.smtpserver.
git-send-email works just fine, but somehow b4 send fails. I think it's
due to cmd_send calling b4.get_smtp() and expecting to get some
credentials out of it?
Maxime
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists