[PATCH v2 7/7] memory: renesas-rpc-if: Add support for RZ/G2L
Wolfram Sang
wsa+renesas at sang-engineering.com
Wed Nov 3 01:41:44 PDT 2021
Hi Prabhakar,
> Oops I missed that, does the below look good?
Yes, only a minor nit.
>
> #define RPCIF_PHYADD_ADD_MD 0x00
> #define RPCIF_PHYADD_ADD_RDLSEL 0x22
> #define RPCIF_PHYADD_ADD_FDLSEL 0x24
> #define RPCIF_PHYADD_ADD_RDLMON 0x26
> #define RPCIF_PHYADD_ADD_FDLMON 0x28
>
> #define RPCIF_PHYADD_ACCEN BIT(31)
> #define RPCIF_PHYADD_RW BIT(30)
Maybe we could leave this because we don't use it? You decide.
> > + regmap_write(rpc->regmap, RPCIF_PHYWR, 0x00000030);
> > + regmap_write(rpc->regmap, RPCIF_PHYADD, 0x80000032);
> >
> For the above do you have any suggestions? As I couldn't find any
> details about it or shall I just go with magic numbers for now?
Ack. I couldn't find docs about these as well. I suggest to add a
comment where this value came from. We can ask the BSP and/or HW team
for details and update this pair incrementally.
> thanks, once we agree upon above I shall re-spin v3.
Cool, looking forward to it!
Happy hacking,
Wolfram
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-mtd/attachments/20211103/f3d92aae/attachment.sig>
More information about the linux-mtd
mailing list