Hi Marek,

On Mon, 3 Feb 2020 at 01:28, Marek Vasut <ma...@denx.de> wrote:
>
> On 2/3/20 1:04 AM, Simon Glass wrote:
> > On Sun, 2 Feb 2020 at 10:12, Sean Anderson <sean...@gmail.com> wrote:
> >>
> >> This header is needed outside of the arm architecture by the designware wdt
> >> driver.
> >> ---
> >>  arch/arm/cpu/armv7/cache_v7.c                  | 2 +-
> >>  arch/arm/mach-davinci/spl.c                    | 2 +-
> >>  arch/arm/mach-omap2/clocks-common.c            | 2 +-
> >>  arch/arm/mach-omap2/emif-common.c              | 2 +-
> >>  arch/arm/mach-omap2/omap4/emif.c               | 2 +-
> >>  arch/arm/mach-omap2/omap5/dra7xx_iodelay.c     | 2 +-
> >>  arch/arm/mach-omap2/omap5/emif.c               | 2 +-
> >>  arch/arm/mach-omap2/omap5/hwinit.c             | 2 +-
> >>  arch/arm/mach-socfpga/spl_a10.c                | 2 +-
> >>  arch/arm/mach-socfpga/spl_agilex.c             | 2 +-
> >>  arch/arm/mach-socfpga/spl_gen5.c               | 2 +-
> >>  arch/arm/mach-socfpga/spl_s10.c                | 2 +-
> >>  drivers/watchdog/designware_wdt.c              | 4 ++--
> >>  arch/arm/include/asm/utils.h => include/log2.h | 4 ++--
> >>  14 files changed, 16 insertions(+), 16 deletions(-)
> >>  rename arch/arm/include/asm/utils.h => include/log2.h (93%)
> >
> > Reviewed-by: Simon Glass <s...@chromium.org>
>
> How could this be reviewed if it's missing basic things, like a SoB line ?

I saw you requested that on another patch. Apart from that it looks OK to me.

People should use patman :-)

Regards,
Simon

Reply via email to