From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <stable-owner@vger.kernel.org>
Received: from mail.stusta.mhn.de ([141.84.69.5]:36378 "EHLO
        mail.stusta.mhn.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1729926AbeKAFtn (ORCPT
        <rfc822;stable@vger.kernel.org>); Thu, 1 Nov 2018 01:49:43 -0400
Date: Wed, 31 Oct 2018 22:49:59 +0200
From: Adrian Bunk <bunk@kernel.org>
To: Sasha Levin <sashal@kernel.org>
Cc: stable@vger.kernel.org, Alan Chiang <alanx.chiang@intel.com>,
        Andy Yeh <andy.yeh@intel.com>,
        Bartosz Golaszewski <brgl@bgdev.pl>, linux-i2c@vger.kernel.org
Subject: Re: 4.14/4.18: Please add "eeprom: at24: Add support for
 address-width property"
Message-ID: <20181031204959.GA3691@localhost>
References: <20181022083835.GA5315@localhost>
 <20181031142745.GK194472@sasha-vm>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <20181031142745.GK194472@sasha-vm>
Sender: stable-owner@vger.kernel.org
List-ID: <stable.vger.kernel.org>

On Wed, Oct 31, 2018 at 10:27:45AM -0400, Sasha Levin wrote:
> On Mon, Oct 22, 2018 at 11:38:35AM +0300, Adrian Bunk wrote:
> > Please cherry-pick the following commit to 4.14 and 4.18:
> > 
> >    commit a2b3bf4846e5eed62ea6abb096af2c950961033c
> >    Author: Alan Chiang <alanx.chiang@intel.com>
> >    Date:   Wed Jul 25 11:20:22 2018 +0800
> > 
> >    eeprom: at24: Add support for address-width property
> > 
> >    Provide a flexible way to determine the addressing bits of eeprom.
> >    Pass the addressing bits to driver through address-width property.
> > 
> >    Signed-off-by: Alan Chiang <alanx.chiang@intel.com>
> >    Signed-off-by: Andy Yeh <andy.yeh@intel.com>
> >    Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
> > 
> > 
> > Confirmed to work on 4.14 with the Identification Page
> > of an ST M24M02-DR (256 bytes but 16 bit addressing).
> 
> This patch doesn't look like stable material. The hardware never worked
> before, so this is not a regression and falls under new hardware
> enablement, but the patch is more complex than what we usually allow for
> hardware enablement (adding various IDs/quirks/etc).
> 
> Am I missing something?

IMHO it is a pretty straightforward quirk for hardware enablement.

Note that the quirk will only be used if explicitely enabled in the
device tree, for everyone else this code is a nop.

> Thanks,
> Sasha

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed