Hi Rob, On Tue, Jun 14, 2022 at 2:34 PM Rob Herring <robh@xxxxxxxxxx> wrote: > > On Mon, Jun 13, 2022 at 12:56:58PM -0700, Brad Larson wrote: > > From: Brad Larson <blarson@xxxxxxx> > > > > This patch adds the reset controller functionality for the > > AMD Pensando Elba System Resource Chip. > > > > Signed-off-by: Brad Larson <blarson@xxxxxxx> > > --- > > drivers/reset/Kconfig | 9 ++ > > drivers/reset/Makefile | 1 + > > drivers/reset/reset-elbasr.c | 94 +++++++++++++++++++ > > .../reset/amd,pensando-elba-reset.h | 11 +++ > > This goes with the binding patch I must have misinterpreted an earlier request to put the bindings separately up front in the patch set. For a new driver the binding and driver should be in one patch which I'll change for the next version. > ... > > --- /dev/null > > +++ b/drivers/reset/reset-elbasr.c > > @@ -0,0 +1,94 @@ > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > Kernel code is GPL-2.0-only generally. Did something change versus earlier request for dual license? > Re: [PATCH v3 11/11] arm64: dts: Add Pensando Elba SoC support > - by Rob Herring @ 2021-10-27 21:37 UTC [8%] > > +// SPDX-License-Identifier: GPL-2.0 > Do you care about using with non-GPL OS? Dual license is preferred. Regards, Brad