lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aXyYi1qhaOodEZrN@MNI-190>
Date: Fri, 30 Jan 2026 12:40:19 +0100
From: Jose Javier Rodriguez Barbarin <dev-josejavier.rodriguez@...gon.com>
To: Jonathan Cameron <jic23@...nel.org>
CC: dlechner@...libre.com, nuno.sa@...log.com, andy@...nel.org,
	linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org,
	dev-jorge.sanjuangarcia@...gon.com
Subject: Re: [RESEND PATCH v2 0/1] iio: Remove MODULE_ALIAS from men_z188_adc

On Thu, Jan 29, 2026 at 06:26:44PM +0000, Jonathan Cameron wrote:
> On Thu, 29 Jan 2026 17:48:54 +0100
> Jose Javier Rodriguez Barbarin <dev-josejavier.rodriguez@...gon.com> wrote:
> 
> > On Thu, Jan 29, 2026 at 04:05:02PM +0000, Jonathan Cameron wrote:
> > > On Fri, 23 Jan 2026 11:48:04 +0100
> > > Jose Javier Rodriguez Barbarin <dev-josejavier.rodriguez@...gon.com> wrote:
> > > 
> > > Hi Jose,
> > > 
> > > When resending the first thing we should see is a clear statement here
> > > at the top of the cover letter for why?  I applied the previous posted
> > > version and not seeing any indication here on why I would consider this one.
> > > 
> > > Jonathan
> > >   
> > 
> > Hi Jonathan,
> > 
> > Thanks for your answer. First of all I would like to apologize myself.
> > I just resent the patch because I was not aware the v2 patch was already
> > applied, I came across yesterday while updating my linux-next local copy.
> > 
> > For a reason I cannot understand, I didn't receive your previous answer.
> No problem. But a resend is rarely the way to address that unless
> a rebase is needed (in which case it's a new version anyway!)
> If it's been quite a long time, just reply to the original thread
> to ask if it got missed. 
> 
> Jonathan
> 

Thanks for the info. It is good to know how to proceed in the future.

I've just followed the guideline for submitting patches.

Regards,

Javier R.

> > 
> > My bad.
> > 
> > Regards,
> > 
> > Javier R.
> > 
> > >   
> > > > The first patches I sent for fixing the autoload problem encountered
> > > > on mcb device drivers were 2. The first [1] was focused on updating the
> > > > modpost process for letting kbuild to access to the data within
> > > > MODULE_DEVICE_TABLE and the second one [2] for removing the MODULE_ALIAS
> > > > on all mcb client drivers.
> > > > 
> > > > They were rejected and Andy suggested me to split the second patch
> > > > in a per-driver basis instead of sending all drivers' changes in a
> > > > single patch once the first patch was merged.
> > > > 
> > > > The first patch is already merged on Linus's Git repository for 6.19-rc4
> > > > 
> > > > commit 1f4ea4838b13 ("mcb: Add missing modpost build support")
> > > > 
> > > > So now I am sending this patch for removing MODULE_ALIAS on the mcb driver
> > > > in this subsystem as it is no longer required. This cleanup is being sent
> > > > to each affected subsystem separately, as per the review suggestion to ease
> > > > the handling for maintainers.
> > > > 
> > > > [1] https://lore.kernel.org/all/20251127155452.42660-2-dev-josejavier.rodriguez@duagon.com/
> > > > [2] https://lore.kernel.org/all/20251127155452.42660-3-dev-josejavier.rodriguez@duagon.com/
> > > > 
> > > > changes in v2:
> > > > - Renane commit message
> > > > 
> > > > Jose Javier Rodriguez Barbarin (1):
> > > >   iio: adc: men_z188_adc: drop unneeded MODULE_ALIAS
> > > > 
> > > >  drivers/iio/adc/men_z188_adc.c | 1 -
> > > >  1 file changed, 1 deletion(-)
> > > >   
> 

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ