[U-Boot] [PATCH] MAINTAINERS: Add maintainer for TI security related files

[U-Boot] [PATCH] MAINTAINERS: Add maintainer for TI security related files

[Andrew F. Davis]

Changes involving High-Security boards should be CC'd for additional
assessment of the security implications.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
Recently some fastboot additions have been made that may effect security
on our HS boards by letting some boot methods work around our chain-of-
trust. It would be nice to get a CC for changes that directly effect
these HS boards so I have a chance to check for this kind of thing.

 MAINTAINERS | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 793ff49f68..1ea7ae013a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -434,6 +434,17 @@ S:	Maintained
 F:	drivers/spmi/
 F:	include/spmi/
 
+TI SYSTEM SECURITY
+M:	Andrew F. Davis <afd@ti.com>
+S:	Supported
+F:	arch/arm/mach-omap2/omap5/sec_entry_cpu1.S
+F:	arch/arm/mach-omap2/omap5/sec-fxns.c
+F:	arch/arm/mach-omap2/sec-common.c
+F:	arch/arm/mach-omap2/config_secure.mk
+F:	configs/am43xx_hs_evm_defconfig
+F:	configs/am57xx_hs_evm_defconfig
+F:	configs/dra7xx_hs_evm_defconfig
+
 TQ GROUP
 #M:	Martin Krause <martin.krause@tq-systems.de>
 S:	Orphaned (Since 2016-02)
-- 
2.11.0

[U-Boot] MAINTAINERS: Add maintainer for TI security related files

[Tom Rini]

On Wed, Jan 11, 2017 at 11:09:27AM -0600, Andrew F. Davis wrote:

> Changes involving High-Security boards should be CC'd for additional
> assessment of the security implications.
> 
> Signed-off-by: Andrew F. Davis <afd@ti.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170115/8c652124/attachment.sig>

[U-Boot] [PATCH] MAINTAINERS: Add maintainer for TI security related files

[Jagan Teki]

On Wed, Jan 11, 2017 at 6:09 PM, Andrew F. Davis <afd@ti.com> wrote:
> Changes involving High-Security boards should be CC'd for additional
> assessment of the security implications.
>
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> ---
> Recently some fastboot additions have been made that may effect security
> on our HS boards by letting some boot methods work around our chain-of-
> trust. It would be nice to get a CC for changes that directly effect
> these HS boards so I have a chance to check for this kind of thing.
>
>  MAINTAINERS | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 793ff49f68..1ea7ae013a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -434,6 +434,17 @@ S: Maintained
>  F:     drivers/spmi/
>  F:     include/spmi/
>
> +TI SYSTEM SECURITY
> +M:     Andrew F. Davis <afd@ti.com>
> +S:     Supported
> +F:     arch/arm/mach-omap2/omap5/sec_entry_cpu1.S
> +F:     arch/arm/mach-omap2/omap5/sec-fxns.c
> +F:     arch/arm/mach-omap2/sec-common.c
> +F:     arch/arm/mach-omap2/config_secure.mk
> +F:     configs/am43xx_hs_evm_defconfig
> +F:     configs/am57xx_hs_evm_defconfig
> +F:     configs/dra7xx_hs_evm_defconfig

I thought of commenting this but some how missed, configs/*_defconfig
files shouldn't be part of MAINTAINERS it has a separate file in
respective boards.

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

[U-Boot] [PATCH] MAINTAINERS: Add maintainer for TI security related files

[Tom Rini]

On Sun, Jan 15, 2017 at 07:50:35PM +0100, Jagan Teki wrote:
> On Wed, Jan 11, 2017 at 6:09 PM, Andrew F. Davis <afd@ti.com> wrote:
> > Changes involving High-Security boards should be CC'd for additional
> > assessment of the security implications.
> >
> > Signed-off-by: Andrew F. Davis <afd@ti.com>
> > ---
> > Recently some fastboot additions have been made that may effect security
> > on our HS boards by letting some boot methods work around our chain-of-
> > trust. It would be nice to get a CC for changes that directly effect
> > these HS boards so I have a chance to check for this kind of thing.
> >
> >  MAINTAINERS | 11 +++++++++++
> >  1 file changed, 11 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 793ff49f68..1ea7ae013a 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -434,6 +434,17 @@ S: Maintained
> >  F:     drivers/spmi/
> >  F:     include/spmi/
> >
> > +TI SYSTEM SECURITY
> > +M:     Andrew F. Davis <afd@ti.com>
> > +S:     Supported
> > +F:     arch/arm/mach-omap2/omap5/sec_entry_cpu1.S
> > +F:     arch/arm/mach-omap2/omap5/sec-fxns.c
> > +F:     arch/arm/mach-omap2/sec-common.c
> > +F:     arch/arm/mach-omap2/config_secure.mk
> > +F:     configs/am43xx_hs_evm_defconfig
> > +F:     configs/am57xx_hs_evm_defconfig
> > +F:     configs/dra7xx_hs_evm_defconfig
> 
> I thought of commenting this but some how missed, configs/*_defconfig
> files shouldn't be part of MAINTAINERS it has a separate file in
> respective boards.

I'm indifferent in cases like this.  For example, dra7xx_hs_evm has two
maintainers, Andrew (due to the security side) and Lokesh (who owns the
TI ref platforms in general).  Making Andrew edit 3 (4 with the
am335x_hs_evm_defconfig coming soon) MAINTAINERS files to add 4 lines
each won't add to the clarity of things as get_maintainer.pl should do
the correct thing and list both, today.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170115/f7195501/attachment.sig>