On Thursday, August 13, 2015 at 05:50:18 PM, vikasm wrote: > Hi Marek, Hi!
> On 08/12/2015 07:07 PM, Marek Vasut wrote: > > On Thursday, July 16, 2015 at 04:27:29 AM, Vikas Manocha wrote: > > > > Commit message is missing. > > Actually subject of the mail was sufficient, this patch just moves the > register configuration in init. NAK, fix the commit message. > >> Signed-off-by: Vikas Manocha <vikas.mano...@st.com> > >> --- > >> > >> Changes in v2: Rebased to master > >> > >> drivers/spi/cadence_qspi_apb.c | 9 ++------- > >> 1 file changed, 2 insertions(+), 7 deletions(-) > >> > >> diff --git a/drivers/spi/cadence_qspi_apb.c > >> b/drivers/spi/cadence_qspi_apb.c index d053407..1ae7edf 100644 > >> --- a/drivers/spi/cadence_qspi_apb.c > >> +++ b/drivers/spi/cadence_qspi_apb.c > >> @@ -534,6 +534,8 @@ void cadence_qspi_apb_controller_init(struct > >> cadence_spi_platdata *plat) > >> > >> /* Indirect mode configurations */ > >> writel((plat->sram_size/2), plat->regbase + CQSPI_REG_SRAMPARTITION); > >> > >> + writel(((u32)plat->ahbbase & CQSPI_INDIRECTTRIGGER_ADDR_MASK), > > > > You can drop the parenthesis around the first argument, they're useless. > > Also, the indent of the second arg should be fixed, I believe checkpatch > > might even complain about it. > > ok. Thanks :) Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot