Re: [PATCH v4 2/3] nvmem: add mt6323 PMIC EFUSE driver

[Andy Shevchenko]

On Mon, Sep 14, 2026 at 11:08:36PM +0100, Srinivas Kandagatla wrote:
> On 9/11/26 7:33 AM, Roman Vivchar via B4 Relay wrote:

> > Add support for the EFUSE controller found in the Mediatek MT6323 PMIC.
> > The MT6323 EFUSE stores 24 bytes of hardware-related data, such as
> > thermal sensor calibration values.

...

> > +++ b/MAINTAINERS

> >  F:	drivers/iio/adc/mt6323-auxadc.c
> >  F:	include/dt-bindings/iio/adc/mediatek,mt6323-auxadc.h
> >  
> > +MEDIATEK MT6323 PMIC NVMEM DRIVER
> > +M:	Roman Vivchar <rva333@xxxxxxxxxxxxxx>
> > +S:	Maintained
> > +F:	drivers/nvmem/mt6323-efuse.c
> 
> placement is in wrong order,

Can you elaborate on this? I do not see the issue.

> same in make file too.

> >  MEDIATEK MT6735 CLOCK & RESET DRIVERS
> >  M:	Yassine Oudjana <y.oudjana@xxxxxxxxxxxxxx>
> >  L:	linux-clk@xxxxxxxxxxxxxxx

...

> > +++ b/drivers/nvmem/Makefile

> > obj-$(CONFIG_NVMEM_MICROCHIP_OTPC)	+= nvmem-microchip-otpc.o
> >  nvmem-microchip-otpc-y			:= microchip-otpc.o
> >  obj-$(CONFIG_NVMEM_MTK_EFUSE)		+= nvmem_mtk-efuse.o
> >  nvmem_mtk-efuse-y			:= mtk-efuse.o
> > +obj-$(CONFIG_NVMEM_MT6323_EFUSE)		+= nvmem_mt6323-efuse.o
> > +nvmem_mt6323-efuse-y			:= mt6323-efuse.o

Yes, here the digits should be before letters.

> >  obj-$(CONFIG_NVMEM_MXS_OCOTP)		+= nvmem-mxs-ocotp.o
> >  nvmem-mxs-ocotp-y			:= mxs-ocotp.o
> >  obj-$(CONFIG_NVMEM_NINTENDO_OTP)	+= nvmem-nintendo-otp.o

-- 
With Best Regards,
Andy Shevchenko