Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1686518

Re: [PATCH v2 01/18] mfd: madera: Add register definitions for Cirrus Logic Madera codecs

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Thu, Jul 13, 2017 at 01:44:10PM +0100, Richard Fitzgerald wrote:
> On Thu, 2017-07-13 at 11:05 +0100, Mark Brown wrote:
> > On Thu, Jul 13, 2017 at 09:02:10AM +0100, Lee Jones wrote:

> > > This patch has been rejected by Linus.

> > > https://lkml.org/lkml/2017/7/7/579

> > Hrm, when I used to push the register definition patches I did elide all
> > the obviously repeated register banks like the write sequencer one that
> > Linus is calling out there.  I'm surprised by the "every single line"
> > bit though...

> Linux doesn't say what he wants us to do about it. I could manually
> strip out a few more definitions but seriously, it makes the code a lot
> harder to maintain if we can't grep it for use of registers and register
> fields. They are big chips, they have a lot of stuff in the registers.

I'd take it up with him, if you can explain why it looks very
repetitive but isn't actually that'd help...

Building up copies of the repeated IPs with macros would help too (the
AIF and especially frame control registers stick out like a sore thumb
here), as would removing the individual registers for the write
sequencer block.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v2 01/18] mfd: madera: Add register definitions for  Cirrus Logic Madera codecs Lee Jones <lee.jones@linaro.org> - 2017-07-13 10:10 +0200
  Re: [PATCH v2 01/18] mfd: madera: Add register definitions for  Cirrus Logic Madera codecs Mark Brown <broonie@kernel.org> - 2017-07-13 12:10 +0200
    Re: [PATCH v2 01/18] mfd: madera: Add register definitions for  Cirrus Logic Madera codecs Richard Fitzgerald <rf@opensource.wolfsonmicro.com> - 2017-07-13 14:50 +0200
      Re: [PATCH v2 01/18] mfd: madera: Add register definitions for  Cirrus Logic Madera codecs Mark Brown <broonie@kernel.org> - 2017-07-13 15:10 +0200
        Re: [PATCH v2 01/18] mfd: madera: Add register definitions for  Cirrus Logic Madera codecs Richard Fitzgerald <rf@opensource.wolfsonmicro.com> - 2017-07-14 14:00 +0200
          Re: [alsa-devel] [PATCH v2 01/18] mfd: madera: Add register definitions for Cirrus Logic Madera codecs Takashi Iwai <tiwai@suse.de> - 2017-07-14 14:10 +0200

csiph-web