public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andy Shevchenko <andriy.shevchenko@intel.com>
To: Javier Rodriguez <josejavier.rodriguez@duagon.com>
Cc: linux-kernel@vger.kernel.org,
	Jose Javier Rodriguez Barbarin
	<dev-josejavier.rodriguez@duagon.com>,
	Jorge Sanjuan Garcia <dev-jorge.sanjuangarcia@duagon.com>,
	Linus Walleij <linusw@kernel.org>,
	Bartosz Golaszewski <brgl@kernel.org>,
	linux-gpio@vger.kernel.org
Subject: Re: [PATCH 1/5] gpio: gpio-menz127: drop unneeded MODULE_ALIAS
Date: Fri, 2 Jan 2026 14:10:47 +0200	[thread overview]
Message-ID: <aVe1x3a_q4b6rSrX@smile.fi.intel.com> (raw)
In-Reply-To: <20251230215928.62258-2-josejavier.rodriguez@duagon.com>

On Tue, Dec 30, 2025 at 10:59:24PM +0100, Javier Rodriguez wrote:
>    From: Jose Javier Rodriguez Barbarin <dev-josejavier.rodriguez@duagon.com>
> 
>    The MODULE_ALIAS() is redundant since the module alias is now
>    automatically generated from the MODULE_DEVICE_TABLE().
> 
>    Remove the explicit alias.
> 
>    No functional change intended.

The patch is mangled.

>    Fixes: 1f4ea4838b13 ("mcb: Add missing modpost build support")
>    Reviewed-by: Jorge Sanjuan Garcia <dev-jorge.sanjuangarcia@duagon.com>

>    Cc: Linus Walleij <linusw@kernel.org>
>    Cc: Bartosz Golaszewski <brgl@kernel.org>
>    Cc: Andy Shevchenko <andriy.shevchenko@intel.com>
>    Cc: linux-gpio@vger.kernel.org

Please, move the Cc: list after the '---' line...

>    Signed-off-by: Jose Javier Rodriguez Barbarin
>    <dev-josejavier.rodriguez@duagon.com>
>    ---

...here. It will have the same effect on email, but will reduce a noise in the
Git history.

...

>    The information contained in this message is private and confidential, as
>    well as any enclosed document/file attached to it and is addressed
>    exclusively to its recipient. Please if you are not the intended recipient
>    and have received this message by mistake, notify us of this fact and
>    delete the message from your system. The copying, dissemination or
>    disclosure of its content to third parties without the prior written
>    consent of DUAGON IBERIA S.L. is forbidden. Otherwise, it will violate
>    current legislation. In accordance with Spanish Data Protection Law 3/2018
>    and the European Data Protection Regulation 2016/679, we remind you that
>    your data is processed by DUAGON IBERIA S.L.. You can freely exercise the
>    rights of access, opposition, rectification, cancellation or suppression,
>    revocation of consent, portability and limitation of data processing, by
>    contacting DUAGON IBERIA S.L., Ronda de Europa, 5. Tres Cantos. Madrid,
>    C.P. 28760. Spain.

This is not compatible with the Open Source workflow. Please, make sure there
is no footer in the next version.

...

Also, you missed a cover letter. Or send the patches separately as I don't see
now any evidence of the cross patch dependency.

-- 
With Best Regards,
Andy Shevchenko



  reply	other threads:[~2026-01-02 12:10 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-30 21:59 [PATCH 0/5] Remove MODULE_ALIAS from all mcb client drivers Javier Rodriguez
2025-12-30 21:59 ` [PATCH 1/5] gpio: gpio-menz127: drop unneeded MODULE_ALIAS Javier Rodriguez
2026-01-02 12:10   ` Andy Shevchenko [this message]
2026-01-07 17:17     ` Jose Javier Rodriguez Barbarin
2025-12-30 21:59 ` [PATCH 2/5] serial: men_z135_uart: " Javier Rodriguez
2025-12-30 21:59 ` [PATCH 3/5] iio: adc: men_z188_adc: " Javier Rodriguez
2025-12-31 18:00   ` Jonathan Cameron
2025-12-30 21:59 ` [PATCH 4/5] 8250_men_mcb: " Javier Rodriguez
2025-12-30 21:59 ` [PATCH 5/5] watchdog: menz069_wdt: " Javier Rodriguez
2025-12-31 14:21   ` Guenter Roeck

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aVe1x3a_q4b6rSrX@smile.fi.intel.com \
    --to=andriy.shevchenko@intel.com \
    --cc=brgl@kernel.org \
    --cc=dev-jorge.sanjuangarcia@duagon.com \
    --cc=dev-josejavier.rodriguez@duagon.com \
    --cc=josejavier.rodriguez@duagon.com \
    --cc=linusw@kernel.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox