Re: [PATCH linux dev-5.8 v2 00/11] Add NPCM7xx patches to dev-5.8

[Tomer Maimon <tmaimon77@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 3708 bytes --]

Hi Joel,

First tahnks a lot for reviewing the patches.

On Mon, 11 Jan 2021 at 02:37, Joel Stanley <joel@jms.id.au> wrote:

> Hi Tomer,
>
> On Tue, 5 Jan 2021 at 13:45, Tomer Maimon <tmaimon77@gmail.com> wrote:
> >
> > In this patch set we will like to align with relevant modifications
> > in Nuvoton OpenBMC Linux kernel 5.4.
>
> Thanks for sending the patches. I can merge them into 5.8, however I
> have a v5.10 branch that I plan on moving to imminently.
>
> >
> > Linux upstream current status:
> >         1. npcm7xx clock driver - adding read only
> >                 flag to divider clocks, Will be sent to Linux community.
> >         2. Adding NPCM ADC calibration - Will be sent to Linux vanilla,
> >                 but I am not sure it will be approved.
> >         3. Add DT restart priority and reset type support - sent to Linux
> >                 community la but havent approved yet.
> >         4. persist configuration to the pin control driver - asked by a
> costumer,
> >                 didnt sent to Linux community.
>
> Do you plan on sending it?
>
I need you advise on this one, I pretty sure that the pin
cntroller maintainer will refuse to add it
what do you think?

>
> >         5. Add HGPIO pin support to NPCM7xx pinctrl driver - will be sent
> >                 to Linux community
> >         6. JTAG master driver - will be sent to Linux community once we
> will
> >                 have BMC folder.
>
What about the JTAG driver? can you add it please? do I need to move it to
soc folder or to leave it in
misc?

>
> As you've noted, I recommend you submit them to mainline ASAP to avoid
> extra handling of patches in the openbmc tree.
>
you right, I will do my best...

>
> Cheers,
>
> Joel
>
> >
> > Changes since version 1:
> > - Address comments from Jonathan Neuschäfer: removing trailing whitespace
> >         in NPCM watchdog documentation.
> > - Adding Stanley Chu to NPCM JTAG master driver
> >
> > Tomer Maimon (11):
> >   clk: npcm7xx: add read only flag to divider clocks
> >   iio: adc: add calibration support to npcm ADC
> >   dts: npcm750: add fuse regmap support node
> >   watchdog: npcm: Add DT restart priority and reset type support
> >   dt-binding: watchdog: Add DT restart priority and reset type
> >   pinctrl: npcm7xx: Add HGPIO pin support to NPCM7xx pinctrl driver
> >   pinctrl: pinconf: add pin persist configuration
> >   pinctrl: npcm7xx: Add pin persist configuration support
> >   spi: npcm-pspi: Add full duplex support
> >   dt-binding: bmc: add NPCM7XX JTAG master documentation
> >   misc: npcm7xx-jtag-master: add NPCM7xx JTAG master driver
> >
> >  .../bindings/bmc/npcm7xx-jtag-master.txt      |  38 +
> >  .../bindings/watchdog/nuvoton,npcm-wdt.txt    |  32 +
> >  arch/arm/boot/dts/nuvoton-common-npcm7xx.dtsi |   6 +
> >  drivers/clk/clk-npcm7xx.c                     |  70 +-
> >  drivers/iio/adc/npcm_adc.c                    | 191 ++++
> >  drivers/misc/Kconfig                          |   6 +
> >  drivers/misc/Makefile                         |   1 +
> >  drivers/misc/npcm7xx-jtag-master.c            | 840 ++++++++++++++++++
> >  drivers/pinctrl/nuvoton/pinctrl-npcm7xx.c     | 130 ++-
> >  drivers/pinctrl/pinconf-generic.c             |   3 +
> >  drivers/spi/spi-npcm-pspi.c                   |  75 +-
> >  drivers/watchdog/npcm_wdt.c                   | 121 ++-
> >  12 files changed, 1418 insertions(+), 95 deletions(-)
> >  create mode 100644
> Documentation/devicetree/bindings/bmc/npcm7xx-jtag-master.txt
> >  create mode 100644 drivers/misc/npcm7xx-jtag-master.c
> >
> > --
> > 2.22.0
> >
>

cheers,

Tomer

[-- Attachment #2: Type: text/html, Size: 5105 bytes --]