On 14:59 Fri 09 Dec , Sergei Shtylyov wrote: > Hello. > > On 08-12-2011 19:23, Jean-Christophe PLAGNIOL-VILLARD wrote: > > Your subject doesn't parse. Maybe you meant "use newly introduced > smc accessor"? yeap > > >this will allow to use the pata_at91 on a single zImage > > >Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD<plagnioj@xxxxxxxxxxxx> > >Cc: Nicolas Ferre<nicolas.ferre@xxxxxxxxx> > >Cc: linux-ide@xxxxxxxxxxxxxxx > >--- > >Hi, > > > it's depends on other patch for AT91 can we apply via at91 > > >Best Regards, > >J. > > drivers/ata/pata_at91.c | 43 +++++++++++++++++++++---------------------- > > 1 files changed, 21 insertions(+), 22 deletions(-) > > >diff --git a/drivers/ata/pata_at91.c b/drivers/ata/pata_at91.c > >index 5249e6d..c8d1154 100644 > >--- a/drivers/ata/pata_at91.c > >+++ b/drivers/ata/pata_at91.c > >@@ -207,11 +207,11 @@ static void set_smc_timing(struct device *dev, struct ata_device *adev, > > { > > int ret = 0; > > int use_iordy; > >+ struct sam9_smc_config smc; > > unsigned int t6z; /* data tristate time in ns */ > > unsigned int cycle; /* SMC Cycle width in MCK ticks */ > > unsigned int setup; /* SMC Setup width in MCK ticks */ > > unsigned int pulse; /* CFIOR and CFIOW pulse width in MCK ticks */ > >- unsigned int cs_setup = 0;/* CS4 or CS5 setup width in MCK ticks */ > > unsigned int cs_pulse; /* CS4 or CS5 pulse width in MCK ticks*/ > > unsigned int tdf_cycles; /* SMC TDF MCK ticks */ > > unsigned long mck_hz; /* MCK frequency in Hz */ > >@@ -244,26 +244,25 @@ static void set_smc_timing(struct device *dev, struct ata_device *adev, > > } > > > > dev_dbg(dev, "Use IORDY=%u, TDF Cycles=%u\n", use_iordy, tdf_cycles); > >- info->mode |= AT91_SMC_TDF_(tdf_cycles); > > > > /* write SMC Setup Register */ > >- at91_sys_write(AT91_SMC_SETUP(info->cs), > >- AT91_SMC_NWESETUP_(setup) | > >- AT91_SMC_NRDSETUP_(setup) | > >- AT91_SMC_NCS_WRSETUP_(cs_setup) | > >- AT91_SMC_NCS_RDSETUP_(cs_setup)); > >+ smc.nrd_setup = setup; > >+ smc.nwe_setup = smc.nrd_setup; > >+ smc.ncs_read_setup = 0; > >+ smc.ncs_write_setup = smc.ncs_read_setup; > > Why not: > > smc.nrd_setup = smc.nwe_setup = setup; > smc.ncs_read_setup = smc.ncs_write_setup = 0; why not Best Regards, J. -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html