Re: [PATCH] rtc: ds1307: handle oscillator stop flag for ds1337/ds1339/ds3231

[Ronan Dalton <Ronan.Dalton@alliedtelesis.co.nz>]

Hi Meagan,

On Mon, 2026-05-04 at 15:08 -0700, Meagan Lloyd wrote:
> > +       case ds_1337:
> > +       case ds_1339:
> > +       case ds_1341:
> > +       case ds_3231:
> > +               ret = regmap_read(ds1307->regmap,
> > DS1337_REG_STATUS, &tmp);
> > +               if (ret)
> > +                       return ret;
> > +               if (tmp & DS1337_BIT_OSF)
> > +                       return -EINVAL;
> > +               break;
> 
> If you're going to re-arrange the block to be in somewhat of an
> order,
> perhaps put it above 1338 since 1337 < 1338.

I've ordered it this way based on the first case statement in each
block. Since ds_1337 > ds_1308, I've put the block below the block
starting with ds_1308. I could instead order it based on the last case
statement in each block, if you think that's better.