Re: [U-Boot] [PATCH v5 0/8] Add support for GE PPD

2017-11-09 Thread Martyn Welch
On Thu, 2017-11-09 at 11:19 +0100, Lukasz Majewski wrote:
> Hi Martyn,
> 
> > *** Resend due to previous use of unsubscribed email address ***
> 
> Just a small remark.
> 
> You may consider using patman to get the list of u-boot community peers
> to be CC'ed for your patch. Or at least look for a custodian (in
> MAINTAINERS) responsible to IMX (NXP) SoC part of u-boot (in this case
> Stefano Babic or Fabio Estevam).
> 

Ah, OK, I ran get_maintainers and CCed anyone specifically flagged as a
maintainer (which is what I understood to be good practice from the
U-Boot wiki).

I'll bear this in mind and CC those that have made significant
contributions next time.

> The problem with sending code only to u-boot ML is that it may be
> overlooked by people, who are the best candidates to review it.
> 
> > 
> > This series adds support for the GE PPD. The initial patches in this
> > series resolve issues found during development and testing, extend
> > existing support and add new support for devices in the PPD. The final
> > patch in the series adds the board support for the PPD.
> > 
> > Ian Ray (2):
> >   ext4: recover from filesystem corruption when reading
> >   bootcount: add support for bootcounter on EXT filesystem
> > 
> > Martyn Welch (3):
> >   pwm: imx: Enable PWM support on i.MX53
> >   arm: mx5: Add more register definitions
> >   board: ge: make VPD code common
> > 
> > Nandor Han (2):
> >   imx: mxc_i2c: tweak the i2c transfer method
> >   rtc: add support for s35392a
> > 
> > Peter Senna Tschudin (1):
> >   mx53: Add Board support for GE PPD
> > 
> >  README   |   7 +
> >  arch/arm/include/asm/arch-mx5/crm_regs.h |   8 +
> >  arch/arm/include/asm/arch-mx5/imx-regs.h |  50 
> >  arch/arm/mach-imx/mx5/Kconfig|   7 +
> >  board/ge/bx50v3/Makefile |   2 +-
> >  board/ge/bx50v3/bx50v3.c |   2 +-
> >  board/ge/bx50v3/vpd_reader.c | 228 ---
> >  board/ge/bx50v3/vpd_reader.h |  25 --
> >  board/ge/common/Makefile |   7 +
> >  board/ge/common/vpd_reader.c | 197 +
> >  board/ge/common/vpd_reader.h |  17 ++
> >  board/ge/mx53ppd/Kconfig |  17 ++
> >  board/ge/mx53ppd/MAINTAINERS |   7 +
> >  board/ge/mx53ppd/Makefile|  12 +
> >  board/ge/mx53ppd/imximage.cfg|  87 ++
> >  board/ge/mx53ppd/mx53ppd.c   | 457
> > +++
> > board/ge/mx53ppd/mx53ppd_video.c | 135 +
> > board/ge/mx53ppd/ppd_gpio.h  |  96 +++
> > configs/mx53ppd_defconfig|  39 +++
> > drivers/Kconfig  |   2 +
> > drivers/bootcount/Kconfig|  58 
> > drivers/bootcount/Makefile   |   1 +
> > drivers/bootcount/bootcount_ext.c|  62 +
> > drivers/i2c/mxc_i2c.c|  25 +-
> > drivers/pwm/pwm-imx-util.c   |   2 +
> > drivers/rtc/Kconfig  |   6 +
> > drivers/rtc/Makefile |   1 +
> > drivers/rtc/s35392a.c| 365
> >  fs/ext4/ext4_common.c|
> > 10 + fs/ext4/ext4fs.c |  10 +-
> > include/configs/mx53ppd.h| 248 +
> > lib/Kconfig  |   3 + 32 files changed,
> > 1927 insertions(+), 266 deletions(-) delete mode 100644
> > board/ge/bx50v3/vpd_reader.c delete mode 100644
> > board/ge/bx50v3/vpd_reader.h create mode 100644
> > board/ge/common/Makefile create mode 100644
> > board/ge/common/vpd_reader.c create mode 100644
> > board/ge/common/vpd_reader.h create mode 100644
> > board/ge/mx53ppd/Kconfig create mode 100644
> > board/ge/mx53ppd/MAINTAINERS create mode 100644
> > board/ge/mx53ppd/Makefile create mode 100644
> > board/ge/mx53ppd/imximage.cfg create mode 100644
> > board/ge/mx53ppd/mx53ppd.c create mode 100644
> > board/ge/mx53ppd/mx53ppd_video.c create mode 100644
> > board/ge/mx53ppd/ppd_gpio.h create mode 100644
> > configs/mx53ppd_defconfig create mode 100644 drivers/bootcount/Kconfig
> >  create mode 100644 drivers/bootcount/bootcount_ext.c
> >  create mode 100644 drivers/rtc/s35392a.c
> >  create mode 100644 include/configs/mx53ppd.h
> > 
> 
> 
> 
> Best regards,
> 
> Lukasz Majewski
> 
> --
> 
> DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de


___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] [PATCH v5 0/8] Add support for GE PPD

2017-11-09 Thread Lukasz Majewski
Hi Martyn,

> *** Resend due to previous use of unsubscribed email address ***

Just a small remark.

You may consider using patman to get the list of u-boot community peers
to be CC'ed for your patch. Or at least look for a custodian (in
MAINTAINERS) responsible to IMX (NXP) SoC part of u-boot (in this case
Stefano Babic or Fabio Estevam).

The problem with sending code only to u-boot ML is that it may be
overlooked by people, who are the best candidates to review it.

> 
> This series adds support for the GE PPD. The initial patches in this
> series resolve issues found during development and testing, extend
> existing support and add new support for devices in the PPD. The final
> patch in the series adds the board support for the PPD.
> 
> Ian Ray (2):
>   ext4: recover from filesystem corruption when reading
>   bootcount: add support for bootcounter on EXT filesystem
> 
> Martyn Welch (3):
>   pwm: imx: Enable PWM support on i.MX53
>   arm: mx5: Add more register definitions
>   board: ge: make VPD code common
> 
> Nandor Han (2):
>   imx: mxc_i2c: tweak the i2c transfer method
>   rtc: add support for s35392a
> 
> Peter Senna Tschudin (1):
>   mx53: Add Board support for GE PPD
> 
>  README   |   7 +
>  arch/arm/include/asm/arch-mx5/crm_regs.h |   8 +
>  arch/arm/include/asm/arch-mx5/imx-regs.h |  50 
>  arch/arm/mach-imx/mx5/Kconfig|   7 +
>  board/ge/bx50v3/Makefile |   2 +-
>  board/ge/bx50v3/bx50v3.c |   2 +-
>  board/ge/bx50v3/vpd_reader.c | 228 ---
>  board/ge/bx50v3/vpd_reader.h |  25 --
>  board/ge/common/Makefile |   7 +
>  board/ge/common/vpd_reader.c | 197 +
>  board/ge/common/vpd_reader.h |  17 ++
>  board/ge/mx53ppd/Kconfig |  17 ++
>  board/ge/mx53ppd/MAINTAINERS |   7 +
>  board/ge/mx53ppd/Makefile|  12 +
>  board/ge/mx53ppd/imximage.cfg|  87 ++
>  board/ge/mx53ppd/mx53ppd.c   | 457
> +++
> board/ge/mx53ppd/mx53ppd_video.c | 135 +
> board/ge/mx53ppd/ppd_gpio.h  |  96 +++
> configs/mx53ppd_defconfig|  39 +++
> drivers/Kconfig  |   2 +
> drivers/bootcount/Kconfig|  58 
> drivers/bootcount/Makefile   |   1 +
> drivers/bootcount/bootcount_ext.c|  62 +
> drivers/i2c/mxc_i2c.c|  25 +-
> drivers/pwm/pwm-imx-util.c   |   2 +
> drivers/rtc/Kconfig  |   6 +
> drivers/rtc/Makefile |   1 +
> drivers/rtc/s35392a.c| 365
>  fs/ext4/ext4_common.c|
> 10 + fs/ext4/ext4fs.c |  10 +-
> include/configs/mx53ppd.h| 248 +
> lib/Kconfig  |   3 + 32 files changed,
> 1927 insertions(+), 266 deletions(-) delete mode 100644
> board/ge/bx50v3/vpd_reader.c delete mode 100644
> board/ge/bx50v3/vpd_reader.h create mode 100644
> board/ge/common/Makefile create mode 100644
> board/ge/common/vpd_reader.c create mode 100644
> board/ge/common/vpd_reader.h create mode 100644
> board/ge/mx53ppd/Kconfig create mode 100644
> board/ge/mx53ppd/MAINTAINERS create mode 100644
> board/ge/mx53ppd/Makefile create mode 100644
> board/ge/mx53ppd/imximage.cfg create mode 100644
> board/ge/mx53ppd/mx53ppd.c create mode 100644
> board/ge/mx53ppd/mx53ppd_video.c create mode 100644
> board/ge/mx53ppd/ppd_gpio.h create mode 100644
> configs/mx53ppd_defconfig create mode 100644 drivers/bootcount/Kconfig
>  create mode 100644 drivers/bootcount/bootcount_ext.c
>  create mode 100644 drivers/rtc/s35392a.c
>  create mode 100644 include/configs/mx53ppd.h
> 



Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de


pgpU2U_5qdVC6.pgp
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH v5 0/8] Add support for GE PPD

2017-11-08 Thread Martyn Welch
*** Resend due to previous use of unsubscribed email address ***

This series adds support for the GE PPD. The initial patches in this
series resolve issues found during development and testing, extend
existing support and add new support for devices in the PPD. The final
patch in the series adds the board support for the PPD.

Ian Ray (2):
  ext4: recover from filesystem corruption when reading
  bootcount: add support for bootcounter on EXT filesystem

Martyn Welch (3):
  pwm: imx: Enable PWM support on i.MX53
  arm: mx5: Add more register definitions
  board: ge: make VPD code common

Nandor Han (2):
  imx: mxc_i2c: tweak the i2c transfer method
  rtc: add support for s35392a

Peter Senna Tschudin (1):
  mx53: Add Board support for GE PPD

 README   |   7 +
 arch/arm/include/asm/arch-mx5/crm_regs.h |   8 +
 arch/arm/include/asm/arch-mx5/imx-regs.h |  50 
 arch/arm/mach-imx/mx5/Kconfig|   7 +
 board/ge/bx50v3/Makefile |   2 +-
 board/ge/bx50v3/bx50v3.c |   2 +-
 board/ge/bx50v3/vpd_reader.c | 228 ---
 board/ge/bx50v3/vpd_reader.h |  25 --
 board/ge/common/Makefile |   7 +
 board/ge/common/vpd_reader.c | 197 +
 board/ge/common/vpd_reader.h |  17 ++
 board/ge/mx53ppd/Kconfig |  17 ++
 board/ge/mx53ppd/MAINTAINERS |   7 +
 board/ge/mx53ppd/Makefile|  12 +
 board/ge/mx53ppd/imximage.cfg|  87 ++
 board/ge/mx53ppd/mx53ppd.c   | 457 +++
 board/ge/mx53ppd/mx53ppd_video.c | 135 +
 board/ge/mx53ppd/ppd_gpio.h  |  96 +++
 configs/mx53ppd_defconfig|  39 +++
 drivers/Kconfig  |   2 +
 drivers/bootcount/Kconfig|  58 
 drivers/bootcount/Makefile   |   1 +
 drivers/bootcount/bootcount_ext.c|  62 +
 drivers/i2c/mxc_i2c.c|  25 +-
 drivers/pwm/pwm-imx-util.c   |   2 +
 drivers/rtc/Kconfig  |   6 +
 drivers/rtc/Makefile |   1 +
 drivers/rtc/s35392a.c| 365 
 fs/ext4/ext4_common.c|  10 +
 fs/ext4/ext4fs.c |  10 +-
 include/configs/mx53ppd.h| 248 +
 lib/Kconfig  |   3 +
 32 files changed, 1927 insertions(+), 266 deletions(-)
 delete mode 100644 board/ge/bx50v3/vpd_reader.c
 delete mode 100644 board/ge/bx50v3/vpd_reader.h
 create mode 100644 board/ge/common/Makefile
 create mode 100644 board/ge/common/vpd_reader.c
 create mode 100644 board/ge/common/vpd_reader.h
 create mode 100644 board/ge/mx53ppd/Kconfig
 create mode 100644 board/ge/mx53ppd/MAINTAINERS
 create mode 100644 board/ge/mx53ppd/Makefile
 create mode 100644 board/ge/mx53ppd/imximage.cfg
 create mode 100644 board/ge/mx53ppd/mx53ppd.c
 create mode 100644 board/ge/mx53ppd/mx53ppd_video.c
 create mode 100644 board/ge/mx53ppd/ppd_gpio.h
 create mode 100644 configs/mx53ppd_defconfig
 create mode 100644 drivers/bootcount/Kconfig
 create mode 100644 drivers/bootcount/bootcount_ext.c
 create mode 100644 drivers/rtc/s35392a.c
 create mode 100644 include/configs/mx53ppd.h

-- 
2.1.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] [PATCH v5 0/8] Add support for GE PPD

2017-11-07 Thread Peter Robinson
On Mon, Nov 6, 2017 at 7:14 PM, Martyn Welch  wrote:
> This series adds support for the GE PPD. The initial patches in this
> series resolve issues found during development and testing, extend
> existing support and add new support for devices in the PPD. The final
> patch in the series adds the board support for the PPD.
>
> Ian Ray (2):
>   ext4: recover from filesystem corruption when reading
>   bootcount: add support for bootcounter on EXT filesystem

Shouldn't the above two patches be sent separately, they're not really
part of adding a new board.

> Martyn Welch (3):
>   pwm: imx: Enable PWM support on i.MX53
>   arm: mx5: Add more register definitions
>   board: ge: make VPD code common
>
> Nandor Han (2):
>   imx: mxc_i2c: tweak the i2c transfer method
>   rtc: add support for s35392a
>
> Peter Senna Tschudin (1):
>   mx53: Add Board support for GE PPD
>
>  README   |   7 +
>  arch/arm/include/asm/arch-mx5/crm_regs.h |   8 +
>  arch/arm/include/asm/arch-mx5/imx-regs.h |  50 
>  arch/arm/mach-imx/mx5/Kconfig|   7 +
>  board/ge/bx50v3/Makefile |   2 +-
>  board/ge/bx50v3/bx50v3.c |   2 +-
>  board/ge/bx50v3/vpd_reader.c | 228 ---
>  board/ge/bx50v3/vpd_reader.h |  25 --
>  board/ge/common/Makefile |   7 +
>  board/ge/common/vpd_reader.c | 197 +
>  board/ge/common/vpd_reader.h |  17 ++
>  board/ge/mx53ppd/Kconfig |  17 ++
>  board/ge/mx53ppd/MAINTAINERS |   7 +
>  board/ge/mx53ppd/Makefile|  12 +
>  board/ge/mx53ppd/imximage.cfg|  87 ++
>  board/ge/mx53ppd/mx53ppd.c   | 457 
> +++
>  board/ge/mx53ppd/mx53ppd_video.c | 135 +
>  board/ge/mx53ppd/ppd_gpio.h  |  96 +++
>  configs/mx53ppd_defconfig|  39 +++
>  drivers/Kconfig  |   2 +
>  drivers/bootcount/Kconfig|  58 
>  drivers/bootcount/Makefile   |   1 +
>  drivers/bootcount/bootcount_ext.c|  62 +
>  drivers/i2c/mxc_i2c.c|  25 +-
>  drivers/pwm/pwm-imx-util.c   |   2 +
>  drivers/rtc/Kconfig  |   6 +
>  drivers/rtc/Makefile |   1 +
>  drivers/rtc/s35392a.c| 365 
>  fs/ext4/ext4_common.c|  10 +
>  fs/ext4/ext4fs.c |  10 +-
>  include/configs/mx53ppd.h| 248 +
>  lib/Kconfig  |   3 +
>  32 files changed, 1927 insertions(+), 266 deletions(-)
>  delete mode 100644 board/ge/bx50v3/vpd_reader.c
>  delete mode 100644 board/ge/bx50v3/vpd_reader.h
>  create mode 100644 board/ge/common/Makefile
>  create mode 100644 board/ge/common/vpd_reader.c
>  create mode 100644 board/ge/common/vpd_reader.h
>  create mode 100644 board/ge/mx53ppd/Kconfig
>  create mode 100644 board/ge/mx53ppd/MAINTAINERS
>  create mode 100644 board/ge/mx53ppd/Makefile
>  create mode 100644 board/ge/mx53ppd/imximage.cfg
>  create mode 100644 board/ge/mx53ppd/mx53ppd.c
>  create mode 100644 board/ge/mx53ppd/mx53ppd_video.c
>  create mode 100644 board/ge/mx53ppd/ppd_gpio.h
>  create mode 100644 configs/mx53ppd_defconfig
>  create mode 100644 drivers/bootcount/Kconfig
>  create mode 100644 drivers/bootcount/bootcount_ext.c
>  create mode 100644 drivers/rtc/s35392a.c
>  create mode 100644 include/configs/mx53ppd.h
>
> --
> 2.1.4
>
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> https://lists.denx.de/listinfo/u-boot
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH v5 0/8] Add support for GE PPD

2017-11-06 Thread Martyn Welch
This series adds support for the GE PPD. The initial patches in this
series resolve issues found during development and testing, extend
existing support and add new support for devices in the PPD. The final
patch in the series adds the board support for the PPD.

Ian Ray (2):
  ext4: recover from filesystem corruption when reading
  bootcount: add support for bootcounter on EXT filesystem

Martyn Welch (3):
  pwm: imx: Enable PWM support on i.MX53
  arm: mx5: Add more register definitions
  board: ge: make VPD code common

Nandor Han (2):
  imx: mxc_i2c: tweak the i2c transfer method
  rtc: add support for s35392a

Peter Senna Tschudin (1):
  mx53: Add Board support for GE PPD

 README   |   7 +
 arch/arm/include/asm/arch-mx5/crm_regs.h |   8 +
 arch/arm/include/asm/arch-mx5/imx-regs.h |  50 
 arch/arm/mach-imx/mx5/Kconfig|   7 +
 board/ge/bx50v3/Makefile |   2 +-
 board/ge/bx50v3/bx50v3.c |   2 +-
 board/ge/bx50v3/vpd_reader.c | 228 ---
 board/ge/bx50v3/vpd_reader.h |  25 --
 board/ge/common/Makefile |   7 +
 board/ge/common/vpd_reader.c | 197 +
 board/ge/common/vpd_reader.h |  17 ++
 board/ge/mx53ppd/Kconfig |  17 ++
 board/ge/mx53ppd/MAINTAINERS |   7 +
 board/ge/mx53ppd/Makefile|  12 +
 board/ge/mx53ppd/imximage.cfg|  87 ++
 board/ge/mx53ppd/mx53ppd.c   | 457 +++
 board/ge/mx53ppd/mx53ppd_video.c | 135 +
 board/ge/mx53ppd/ppd_gpio.h  |  96 +++
 configs/mx53ppd_defconfig|  39 +++
 drivers/Kconfig  |   2 +
 drivers/bootcount/Kconfig|  58 
 drivers/bootcount/Makefile   |   1 +
 drivers/bootcount/bootcount_ext.c|  62 +
 drivers/i2c/mxc_i2c.c|  25 +-
 drivers/pwm/pwm-imx-util.c   |   2 +
 drivers/rtc/Kconfig  |   6 +
 drivers/rtc/Makefile |   1 +
 drivers/rtc/s35392a.c| 365 
 fs/ext4/ext4_common.c|  10 +
 fs/ext4/ext4fs.c |  10 +-
 include/configs/mx53ppd.h| 248 +
 lib/Kconfig  |   3 +
 32 files changed, 1927 insertions(+), 266 deletions(-)
 delete mode 100644 board/ge/bx50v3/vpd_reader.c
 delete mode 100644 board/ge/bx50v3/vpd_reader.h
 create mode 100644 board/ge/common/Makefile
 create mode 100644 board/ge/common/vpd_reader.c
 create mode 100644 board/ge/common/vpd_reader.h
 create mode 100644 board/ge/mx53ppd/Kconfig
 create mode 100644 board/ge/mx53ppd/MAINTAINERS
 create mode 100644 board/ge/mx53ppd/Makefile
 create mode 100644 board/ge/mx53ppd/imximage.cfg
 create mode 100644 board/ge/mx53ppd/mx53ppd.c
 create mode 100644 board/ge/mx53ppd/mx53ppd_video.c
 create mode 100644 board/ge/mx53ppd/ppd_gpio.h
 create mode 100644 configs/mx53ppd_defconfig
 create mode 100644 drivers/bootcount/Kconfig
 create mode 100644 drivers/bootcount/bootcount_ext.c
 create mode 100644 drivers/rtc/s35392a.c
 create mode 100644 include/configs/mx53ppd.h

-- 
2.1.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot