On Mon, 2009-09-21 at 17:05 -0700, Prodyut Hazarika wrote: > Hi Ben, > Thanks again for your comments. > > > Same goes with the SDR register definitions. Prefix them with the SOC > > name but don't make them conditionally compiled. > > I will add the base address in the Device tree, and make all register > definitions based on offset from the base in the next version of this > patch.
That's a good idea. In fact, you can also use the dcr_read/write variants of the accessors rather than the low level mfdcri/mtdcri. This wouldn't make much of a difference unless you ever release a SoC with those same registers behind an MMIO mapping but it's cleaner. > Thanks for this comment. I will hookup ethtool with the EMAC driver, but > the MAL driver will come up with default coalesce options (as defined in > the appropriate defconfig file). The user will be able to change these > parameters as needed using ethtool. That's ok. I don't have an objection in using Kconfig to set the defaults. > I will get all the changes in place in the next version of this patch. Thanks ! BTW. If you guys are ever going to do another change to MAL, please please plase, add the -one- major missing feature that's causing all the pain and complication in the current design: Add a per-channel interrupt masking option. The lack of ability to mask the interrupt per MAL channel is what forces us to create that fake netdev structure in order to share the napi device instance between all the EMACs in the system. This is very inefficient too. We would be able to make things run a lot smoother if we could just have a napi instance per EMAC, but for that, we need per-channel interrupt masking. Cheers, Ben. _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
