linux-kernel - Re: [PATCH 0/3] FTM PWM adds regmap and endianness support.

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140428145615.GA28443@ulmo>
Date:	Mon, 28 Apr 2014 16:56:16 +0200
From:	Thierry Reding <thierry.reding@...il.com>
To:	"Li.Xiubo@...escale.com" <Li.Xiubo@...escale.com>
Cc:	"linux-pwm@...r.kernel.org" <linux-pwm@...r.kernel.org>,
	"broonie@...nel.org" <broonie@...nel.org>,
	"swarren@...dia.com" <swarren@...dia.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 0/3] FTM PWM adds regmap and endianness support.

On Mon, Apr 28, 2014 at 02:26:57PM +0000, Li.Xiubo@...escale.com wrote:
> 
> 
> 
> > > Xiubo Li (3):
> > >   pwm: ftm-pwm: Clean up the code.
> > >   pwm: ftm-pwm: Convert to direct regmap API usage.
> > >   pwm: ftm-pwm: Add big-endian support
> > >
> > >  drivers/pwm/pwm-fsl-ftm.c | 96 ++++++++++++++++++++++++++---------------------
> > >  1 file changed, 53 insertions(+), 43 deletions(-)
> >
> > This leaves me with only very vague idea of why this is necessary and
> > why it should be merged.
> >
> > Please describe in more detail (in both the cover-letter and each
> > individual patch) why you want me to apply these patches.
> >
> 
> Thanks very much for your reply.
> 
> Should I resend this patch series? If so, I will add some thing like the following:
> 
> The FTM PWM driver will be used in our Vybrid, LS1 and LS2+ SoCs, and on Vybrid
> and LS2 SoCs, the FTM devices are in LE mode, while on LS1 SoCs it in BE mode.
> 
> So this patch series add endianness support based on the regmap core, which has
> already support the rich endiannesses for the same device.

Yes, please resend with that description added where appropriate. I'd
like to see something like that in the cover letter and, more
importantly, in the individual patches.

Thierry

Content of type "application/pgp-signature" skipped