[PATCH v3 1/5] drivers: memory: Introduce Marvell EBU Device Bus driver

Ezequiel Garcia ezequiel.garcia at free-electrons.com
Sat Apr 6 10:50:52 EDT 2013


Hi Arnd,

On Sat, Apr 06, 2013 at 03:34:36PM +0200, Arnd Bergmann wrote:
> On Saturday 06 April 2013, Ezequiel Garcia wrote:
> > +       device-bus at d0010400 {
> > +               status = "okay";
> > +               ranges = <0 0 0xf0000000 0x1000000>; /* CS0 @addr 0xf0000000, size 0x1000000 */
> > +               #address-cells = <2>;
> > +               #size-cells = <1>;
> > +
> > +               flash at 0 {
> > +                       compatible = "cfi-flash";
> > +
> > +                       /* CS0, 16 MiB */
> > +                       reg = <0 0 0x1000000>;
> > +                       bank-width = <2>;
> > +                       #address-cells = <1>;
> > +                       #size-cells = <1>;
> > +
> > +                       /* Device Bus parameters are required */
> > +
> > +                       /* Read parameters */
> > +                       devbus,dev-width    = <1>;
> > +                       devbus,turn-off-ps  = <60000>;
> > +                       devbus,badr-skew-ps = <0>;
> > +                       devbus,acc-first-ps = <124000>;
> > +                       devbus,acc-next-ps  = <248000>;
> > +                       devbus,rd-setup-ps  = <0>;
> > +                       devbus,rd-hold-ps   = <0>;
> 
> I just read the email from Jason Gunthorpe suggesting to have a separate
> device-bus instance for each CS under it.
> 
> I think the implication of that is that all the "devbus,..." properties can
> now be moved into the parent node, which makes the split between the
> devbus node and its attached device clearer.
> 

Okey, I'll send a v4 fixing all your comments and implementing Jason's idea.

Thanks for the review!
-- 
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com



More information about the linux-arm-kernel mailing list